Noissue - Merge current tip to Edge bookmark

qunit/tests/cssEscaping.js

 "use strict";
 
+(function()
 {
   const {Filter, ElemHideFilter} = require("filterClasses");
   const {escapeCSS, quoteCSS} = require("filterComposer");
 
   module("CSS escaping");
 
   test("CSS escaping", () =>
   {
     function testSelector(opts)
     {
@@ skipping 23 matching lines @@
         for (let attr in opts.attributes)
           element.setAttribute(attr, opts.attributes[attr]);
 
         doc.documentElement.appendChild(element);
       }
 
       let foundElement = doc.querySelector(opts.selector);
       let filter = Filter.fromText("##" + opts.selector);
 
       if (!(filter instanceof ElemHideFilter))
-      {
         ok(false, opts.selector + " (not allowed in elemhide filters)");
-      }
+      else if (mustMatch)
+        equal(foundElement, element, opts.selector);
       else
-      {
-        if (mustMatch)
-          equal(foundElement, element, opts.selector);
-        else
-          ok(true, opts.selector);
-      }
+        ok(true, opts.selector);
     }
 
     function testEscape(s)
     {
       testSelector({
         selector: escapeCSS(s),
         tagName: s
       });
 
       testSelector({
@@ skipping 26 matching lines @@
 
       // Some characters are only escaped when in the first positon,
       // so we still have to make sure that everything is correctly escaped
       // in subsequent positions.
       testEscape("x" + chr);
 
       // Leading dashes must be escaped, when followed by certain characters.
       testEscape("-" + chr);
     }
 
-    // Test some non-ASCII characters. However, those shouldn't require escaping.
+    // Test some non-ASCII characters. However, those shouldn't
+    // require escaping.
     testEscape("\uD83D\uDE3B\u2665\u00E4");
   });
-}
+}());