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

Unified Diff: lib/ui.js

Issue 6337686776315904: Issue 394 - hit statistics tool data collection (Closed)
Patch Set: Created Oct. 14, 2014, 4:48 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
« lib/filterHits.js ('K') | « lib/filterStorage.js ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: lib/ui.js
===================================================================
--- a/lib/ui.js
+++ b/lib/ui.js
@@ -23,6 +23,7 @@
let {Policy} = require("contentPolicy");
let {FilterStorage} = require("filterStorage");
let {FilterNotifier} = require("filterNotifier");
+let {FilterHits} = require("filterHits");
let {RequestNotifier} = require("requestNotifier");
let {Filter} = require("filterClasses");
let {Subscription, SpecialSubscription, DownloadableSubscription} = require("subscriptionClasses");
@@ -1190,10 +1191,29 @@
FilterStorage.resetHitCounts();
Prefs.savestats = false;
+ if (Prefs.sendstats)
+ {
+ this.toggleSendStats();
+ return;
+ }
}
else
Prefs.savestats = true;
},
+
+ /**
+ * Toggles "Send stats" option.
+ */
+ toggleSendStats: function(window)
+ {
+ if (Prefs.sendstats)
+ {
+ FilterHits.resetFilterHits();
+ Prefs.sendstats = false;
+ }
+ else
+ Prefs.sendstats = true;
+ },
/**
* Sets the current filter subscription in a single-subscription scenario,
@@ -1534,6 +1554,7 @@
setChecked(prefix + "frameobjects", Prefs.frameobjects);
setChecked(prefix + "slowcollapse", !Prefs.fastcollapse);
setChecked(prefix + "savestats", Prefs.savestats);
+ setChecked(prefix + "sendstats", Prefs.sendstats);
let {defaultToolbarPosition, statusbarPosition} = require("appSupport");
let hasToolbar = defaultToolbarPosition;
@@ -1943,6 +1964,7 @@
["abp-command-toggleobjtabs", "command", UI.togglePref.bind(UI, "frameobjects")],
["abp-command-togglecollapse", "command", UI.togglePref.bind(UI, "fastcollapse")],
["abp-command-togglesavestats", "command", UI.toggleSaveStats.bind(UI)],
+ ["abp-command-togglesendstats", "command", UI.toggleSendStats.bind(UI)],
["abp-command-togglesync", "command", UI.toggleSync.bind(UI)],
["abp-command-toggleshowintoolbar", "command", UI.toggleToolbarIcon.bind(UI)],
["abp-command-toggleshowinstatusbar", "command", UI.togglePref.bind(UI, "showinstatusbar")],
« lib/filterHits.js ('K') | « lib/filterStorage.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld