Issue 394 - hit statistics tool data collection

lib/filterStorage.js

Index: lib/filterStorage.js
===================================================================
--- a/lib/filterStorage.js
+++ b/lib/filterStorage.js
@@ -29,6 +29,7 @@
 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.
@@ -336,6 +337,8 @@
 
     filter.hitCount++;
     filter.lastHit = Date.now();

[saroyanm, 2015/02/17 17:05:51]

@Dave here is where we are setting the lastHit for the filterObject and we use
same value for filters hit calculation. 
According to ->
http://people.mozilla.org/~jorendorff/es6-draft.html#sec-date.now 
Date.now(); should return UTC time. 
So I guess we are Okey here.

+    if (Prefs.sendstats)
+      FilterHits.increaseFilterHits(filter, wnd);    

[Thomas Greiner, 2015/02/23 18:43:05]

`FilterNotifier` allows you to listen to the "filter.hitCount" event which is
fired whenever `filter.lastHit` changes. That means that the filter hits can be
increased from within the `FilterHits` module itself.

[saroyanm, 2015/02/28 15:24:30]

I also need the window object where the match has been occurred so I can get the
Domain and we are not passing the window of the occurrence to trigger function
as one of the param with current filterClasses implementation.

[Thomas Greiner, 2015/03/06 11:29:19]

Ah, right. We also cannot add it to the `FilterNotifier.triggerListeners` call
because that one also doesn't have access to the Window. Unfortunately, seems
like we need to do it like this then.

   },
 
   /**
@@ -443,7 +446,9 @@
 
       if (sourceFile != this.sourceFile)
         this.saveToDisk();
-
+      
+      if (Prefs.sendstats)
+        FilterHits.loadFilterHitsFromDatabase();

[Thomas Greiner, 2015/02/23 18:43:05]

`FilterNotifier` allows you to listen to the "load" event which is fired
whenever filters were loaded into memory. That means that the loading of filter
hits can be triggered from within the `FilterHits` module itself.

[saroyanm, 2015/02/28 15:24:30]

Good point, done.

     }.bind(this);
 
     let explicitFile;