Issue 2395 - Added content script for CSS property filters

chrome/content/cssProperties.js

-function CSSPropertyFilters(window) {
+function CSSPropertyFilters(window, addSelectorsFunc) {
   this.window = window;
+  this.addSelectorsFunc = addSelectorsFunc;
   this.load(this.apply.bind(this));
 }
 
 CSSPropertyFilters.prototype = {
   stringifyStyle: function(style)
   {
     var styles = [];
     for (var i = 0; i < style.length; i++)
     {
       var property = style.item(i);
@@ skipping 30 matching lines @@
           selectors.push(pattern.prefix + rule.selectorText + pattern.suffix);
       }
     }
   },
 
   addSelectors: function(stylesheets)
   {
     var selectors = [];
     for (var i = 0; i < stylesheets.length; i++)
       this.findSelectors(stylesheets[i], selectors);
-    addElemHideSelectors(selectors);
+    this.addSelectorsFunc(selectors);
   },
 
   onLoad: function(event)
   {
     var stylesheet = event.target.sheet;
     if (stylesheet)
       this.addSelectors([stylesheet]);
   },
 
   load: function(callback)
   {
     ext.backgroundPage.sendMessage(
       {
         type: "filters.get",
         what: "cssproperties"

[kzar, 2015/11/02 20:30:56]

We need to pass the domain here as well, according to #2396

[Wladimir Palant, 2015/11/05 13:10:04]

Actually, I don't think we do - sender.frame.url should do.

[kzar, 2015/11/05 14:35:32]

OK I've updated the description for #2396 accordingly.

       },
       function(patterns)
       {
         this.patterns = patterns;
         callback();
       }.bind(this)
     );
   },
 
   apply: function()
   {
     if (this.patterns.length > 0)
     {
       var document = this.window.document;
       this.addSelectors(document.styleSheets);
       document.addEventListener("load", this.onLoad.bind(this), true);
     }
   }
 };
-
-if (typeof window == "object")
-  new CSSPropertyFilters(window);