Issue 3998 - cssProperties.js content script shouldn't assume that messaging will automatically kno…

chrome/content/cssProperties.js

 function splitSelector(selector)
 {
   if (selector.indexOf(",") == -1)
     return [selector];
 
   var selectors = [];
   var start = 0;
   var level = 0;
   var sep = "";
 
@@ skipping 18 matching lines @@
         selectors.push(selector.substring(start, i));
         start = i + 1;
       }
     }
   }
 
   selectors.push(selector.substring(start));
   return selectors;
 }
 
-function CSSPropertyFilters(window, addSelectorsFunc) {
+function CSSPropertyFilters(window, getFiltersFunc, addSelectorsFunc)
+{
   this.window = window;
+  this.getFiltersFunc = getFiltersFunc;
   this.addSelectorsFunc = addSelectorsFunc;
 }
 
 CSSPropertyFilters.prototype = {
   stringifyStyle: function(style)
   {
     var styles = [];
     for (var i = 0; i < style.length; i++)
     {
       var property = style.item(i);
@@ skipping 64 matching lines @@
 
   onLoad: function(event)
   {
     var stylesheet = event.target.sheet;
     if (stylesheet)
       this.addSelectors([stylesheet]);
   },
 
   load: function(callback)
   {
-    ext.backgroundPage.sendMessage(
-      {
-        type: "filters.get",
-        what: "cssproperties"
-      },
-      function(patterns)
-      {
-        this.patterns = patterns;
-        callback();
-      }.bind(this)
-    );
+    this.getFiltersFunc(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);
     }
   }
 };