Issue 7284 - Move CSS escaping to createStyleSheet

test/elemHide.js

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-present eyeo GmbH
  *
  * Adblock Plus is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
  * published by the Free Software Foundation.
  *
  * Adblock Plus is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 279 matching lines @@
   test.equal(
     createStyleSheet([
       "html", "#foo", ".bar", "#foo .bar", "#foo > .bar",
       "#foo[data-bar='bar']"
     ]),
     "html, #foo, .bar, #foo .bar, #foo > .bar, #foo[data-bar='bar'] " +
     "{display: none !important;}\n",
     "Style sheet creation should work"
   );
 
-  let selectors = new Array(50000);
-  for (let index = 0; index < selectors.length; index++)
-    selectors[index] = ".s" + index;

[hub, 2019/02/12 23:55:49]

The previous version of the code produced an Array that was empty as it seems
the iterator returned nothing.

[Manish Jethani, 2019/02/13 12:27:25]

I see, `Array.prototype.map` skips empty items.

This could also be fixed by using fill()

  new Array(50000).fill().map((element, index) => ".s" + index)

+  let selectors = new Array(50000).fill().map((element, index) => ".s" + index);
 
   test.equal((createStyleSheet(selectors).match(/\n/g) || []).length,
              Math.ceil(50000 / selectorGroupSize),
              "Style sheet should be split up into rules with at most " +
              selectorGroupSize + " selectors each");
 
+  test.equal(
+    createStyleSheet([
+      "html", "#foo", ".bar", "#foo .bar", "#foo > .bar",
+      "#foo[data-bar='{foo: 1}']"
+    ]),
+    "html, #foo, .bar, #foo .bar, #foo > .bar, " +
+    "#foo[data-bar='\\7B foo: 1\\7D '] {display: none !important;}\n",
+    "Braces should be escaped"
+  );
+
   test.done();
 };
 
 exports.testRulesFromStyleSheet = function(test)
 {
   // Note: The rulesFromStyleSheet function assumes that each rule will be
   // terminated with a newline character, including the last rule. If this is
   // not the case, the function goes into an infinite loop. It should only be
   // used with the return value of the createStyleSheet function.
 
   test.deepEqual([...rulesFromStyleSheet("")], []);
   test.deepEqual([...rulesFromStyleSheet("#foo {}\n")], ["#foo {}"]);
   test.deepEqual([...rulesFromStyleSheet("#foo {}\n#bar {}\n")],
                  ["#foo {}", "#bar {}"]);
 
   test.done();
 };