Issue 5695 - Use tabs.insertCSS if extensionTypes.CSSOrigin exists

include.preload.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 423 matching lines @@
       let selector = preparedSelectors.slice(
         i, i + this.selectorGroupSize
       ).join(", ");
       this.style.sheet.insertRule(selector + "{display: none !important;}",
                                   this.style.sheet.cssRules.length);
     }
   },
 
   addSelectors(selectors, filters)
   {
-    if (!selectors || selectors.length == 0)

[Manish Jethani, 2017/09/17 13:51:37]

This is no longer needed since we're checking this before calling this method.
It was never needed for the other case where this method is called for the
emulation filters.

+    if (selectors.length == 0)
       return;
 
     if (this.inject)
     {
       // Insert the style rules inline if we have been instructed by the
       // background page to do so. This is usually the case, except on platforms
       // that do support user stylesheets via the chrome.tabs.insertCSS API
       // (Firefox 53 onwards for now and possibly Chrome in the near future).
       // Once all supported platforms have implemented this API, we can remove
       // the code below. See issue #5090.
@@ skipping 37 matching lines @@
         this.tracer.disconnect();
       this.tracer = null;
 
       if (this.style && this.style.parentElement)
         this.style.parentElement.removeChild(this.style);
       this.style = null;
 
       if (response.trace)
         this.tracer = new ElementHidingTracer();
 
-      this.inject = !!response.inject;
+      this.inject = response.inject;
 
       if (this.inject)
         this.addSelectors(response.selectors);
       else if (this.tracer)
         this.tracer.addSelectors(response.selectors);
 
       this.elemHideEmulation.apply();
     });
   }
 };
@@ skipping 10 matching lines @@
     checkCollapse(event.target);
   }, true);
 
   document.addEventListener("load", event =>
   {
     let element = event.target;
     if (/^i?frame$/.test(element.localName))
       checkCollapse(element);
   }, true);
 }