Issue 2392/2393 - Created container for CSS property filters

lib/filterListener.js

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-2015 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 10 matching lines @@
 
 Cu.import("resource://gre/modules/XPCOMUtils.jsm");
 Cu.import("resource://gre/modules/Services.jsm");
 
 let {FilterStorage} = require("filterStorage");
 let {FilterNotifier} = require("filterNotifier");
 let {ElemHide} = require("elemHide");
 let {CSSRules} = require("cssRules");
 let {defaultMatcher} = require("matcher");
 let {ActiveFilter, RegExpFilter, ElemHideBase, CSSPropertyFilter} =
-  require("filterClasses");

[Wladimir Palant, 2015/11/05 12:46:19]

Nit: While I couldn't find anything in our style guide, I think that we usually
use double indentation in cases like this.

[Thomas Greiner, 2015/11/25 11:20:00]

Done. I looked through our code and unfortunately, it's quite inconsistent
throughout. Anyway, double indentation makes more sense, true.

+    require("filterClasses");
 let {Prefs} = require("prefs");
 
 /**
  * Value of the FilterListener.batchMode property.
  * @type Boolean
  */
 let batchMode = false;
 
 /**
  * Increases on filter changes, filters will be saved if it exceeds 1.
@@ skipping 236 matching lines @@
     ElemHide.clear();
     CSSRules.clear();
     for (let subscription of FilterStorage.subscriptions)
       if (!subscription.disabled)
         subscription.filters.forEach(addFilter);
     flushElemHide();
   }
   else if (action == "save")
     isDirty = 0;
 }