Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(310)

Unified Diff: components/contextual_search/browser/ctr_aggregator.h

Issue 2285633004: [TTS] Record CTR by week and 28-day intervals. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@ctr
Patch Set: Added recording of impressions and getCurrentWeekNumber to invalidate the caching. Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/contextual_search/browser/ctr_aggregator.h
diff --git a/components/contextual_search/browser/ctr_aggregator.h b/components/contextual_search/browser/ctr_aggregator.h
index a799075042fa7c707380f3eb4e556edefbe1a2ed..df0403e7a2560b2c12cbdef2fd80f0c1e716740a 100644
--- a/components/contextual_search/browser/ctr_aggregator.h
+++ b/components/contextual_search/browser/ctr_aggregator.h
@@ -44,6 +44,10 @@ class CTRAggregator {
// Records an impression. Records a click if |did_click| is true.
void RecordImpression(bool did_click);
+ // Returns the week number for the current week, useful for checking when the
Theresa 2016/09/27 20:53:57 nit: "Returns the number for the current week. Use
Donn Denman 2016/09/28 00:38:43 Done.
+ // current week changes.
+ int GetCurrentWeekNumber();
+
// Returns whether we have the previous week's data for this user.
bool HasPreviousWeekData();

Powered by Google App Engine
This is Rietveld 408576698