Issue 4676 - Replace non-standard CSSStyleSheet.addRule() call by CSSStyleSheet.insertRule()

include.preload.js

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-2016 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 562 matching lines @@
     }
 
     // Safari only allows 8192 primitive selectors to be injected at once[1], we
     // therefore chunk the inserted selectors into groups of 200 to be safe.
     // (Chrome also has a limit, larger... but we're not certain exactly what it
     //  is! Edge apparently has no such limit.)
     // [1] - https://github.com/WebKit/webkit/blob/1cb2227f6b2a1035f7bdc46e5ab69debb75fc1de/Source/WebCore/css/RuleSet.h#L68
     for (var i = 0; i < selectors.length; i += this.selectorGroupSize)
     {
       var selector = selectors.slice(i, i + this.selectorGroupSize).join(", ");
-      this.style.sheet.addRule(selector, "display: none !important;");
+      this.style.sheet.insertRule(selector + "{display: none !important;}",
+          this.style.sheet.cssRules.length);

[kzar, 2016/11/30 13:48:20]

Nit: The indentation of this line looks weird, shouldn't it be in line with the
opening parenthesis of the previous one?

[Wladimir Palant, 2016/11/30 13:52:11]

Ok, I've changed indentation.

     }
   },
 
   apply: function()
   {
     var selectors = null;
     var propertyFiltersLoaded = false;
 
     var checkLoaded = function()
     {
@@ skipping 44 matching lines @@
     checkCollapse(event.target);
   }, true);
 
   document.addEventListener("load", function(event)
   {
     var element = event.target;
     if (/^i?frame$/.test(element.localName))
       checkCollapse(element);
   }, true);
 }