Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: lib/filterStorage.js

Issue 6337686776315904: Issue 394 - hit statistics tool data collection (Closed)
Patch Set: Rebase to changeset: 4056 Created Nov. 24, 2015, 6:44 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « lib/filterListener.js ('k') | lib/ui.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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.
@@ -323,14 +324,20 @@
/**
* Increases the hit count for a filter by one
* @param {Filter} filter
+ * @param {Window} window Window that the match originated in
*/
- increaseHitCount: function(filter)
+ increaseHitCount: function(filter, wnd)
{
if (!Prefs.savestats || !(filter instanceof ActiveFilter))
return;
filter.hitCount++;
filter.lastHit = Date.now();
+ if (Prefs.sendstats)
+ {
+ let wndLocation = Utils.getOriginWindow(wnd).location.href;
+ FilterHits.increaseFilterHits(filter, Utils.unwrapURL(wndLocation).host);
+ }
},
/**
« no previous file with comments | « lib/filterListener.js ('k') | lib/ui.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld