Issue 394 - hit statistics tool data collection

lib/filterStorage.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 11 matching lines @@
 Cu.import("resource://gre/modules/Services.jsm");
 Cu.import("resource://gre/modules/FileUtils.jsm");
 Cu.import("resource://gre/modules/XPCOMUtils.jsm");
 
 let {IO} = require("io");
 let {Prefs} = require("prefs");
 let {Filter, ActiveFilter} = require("filterClasses");
 let {Subscription, SpecialSubscription, ExternalSubscription} = require("subscriptionClasses");
 let {FilterNotifier} = require("filterNotifier");
 let {Utils} = require("utils");
+let {FilterHits} = require("filterHits");
 
 /**
  * Version number of the filter storage file format.
  * @type Integer
  */
 let formatVersion = 4;
 
 /**
  * This class reads user's filters from disk, manages them in memory and writes them back.
  * @class
@@ skipping 287 matching lines @@
   increaseHitCount: function(filter, wnd)
   {
     if (!Prefs.savestats || PrivateBrowsing.enabledForWindow(wnd) ||
         PrivateBrowsing.enabled || !(filter instanceof ActiveFilter))
     {
       return;
     }
 
     filter.hitCount++;
     filter.lastHit = Date.now();
+    if (Prefs.sendstats)
+      FilterHits.increaseFilterHits(filter, wnd);    

[kzar, 2015/03/23 13:11:05]

Nit: Trailing whitespace

[saroyanm, 2015/04/07 15:23:09]

Done.

   },
 
   /**
    * Resets hit count for some filters
    * @param {Array of Filter} filters  filters to be reset, if null all filters will be reset
    */
   resetHitCounts: function(filters)
   {
     if (!filters)
     {
@@ skipping 514 matching lines @@
       Subscription.knownSubscriptions = origKnownSubscriptions;
     }
 
     // Allow events to be processed every now and then.
     // Note: IO.readFromFile() will deal with the potential reentrance here.
     this.linesProcessed++;
     if (this.linesProcessed % 1000 == 0)
       Utils.yield();
   }
 };