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

Unified Diff: components/ntp_snippets/content_suggestions_metrics.cc

Issue 2686063003: [remote suggestions] Attach the fetch time to RemoteSnippets, ContentSnippets and SnippetArticle (Closed)
Patch Set: rebase Created 3 years, 10 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/ntp_snippets/content_suggestions_metrics.cc
diff --git a/components/ntp_snippets/content_suggestions_metrics.cc b/components/ntp_snippets/content_suggestions_metrics.cc
index 38860864bc854b192923f251d5c8419d9b12ad1b..95a1381cc08ef963bcdad3cefe72076adce86337 100644
--- a/components/ntp_snippets/content_suggestions_metrics.cc
+++ b/components/ntp_snippets/content_suggestions_metrics.cc
@@ -223,8 +223,8 @@ void OnSuggestionShown(int global_position,
Category category,
int position_in_category,
base::Time publish_date,
- base::Time last_background_fetch_time,
- float score) {
+ float score,
+ base::Time fetch_date) {
UMA_HISTOGRAM_EXACT_LINEAR(kHistogramShown, global_position,
kMaxSuggestionsTotal);
LogCategoryHistogramPosition(kHistogramShown, category, position_in_category,
@@ -247,8 +247,8 @@ void OnSuggestionShown(int global_position,
// suggestions.
UMA_HISTOGRAM_CUSTOM_TIMES(
kHistogramContentSuggestionsTimeSinceLastBackgroundFetch,
- base::Time::Now() - last_background_fetch_time,
- base::TimeDelta::FromSeconds(1), base::TimeDelta::FromDays(7),
+ base::Time::Now() - fetch_date, base::TimeDelta::FromSeconds(1),
+ base::TimeDelta::FromDays(7),
/*bucket_count=*/100);
}
}
« no previous file with comments | « components/ntp_snippets/content_suggestions_metrics.h ('k') | components/ntp_snippets/content_suggestions_metrics_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698