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).map((element, index) => ".s" + index);
+  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)

 
   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.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();
 };