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

Side by Side Diff: chrome/browser/engagement/site_engagement_metrics.cc

Issue 1368533004: Add UMA metrics to the site engagement service. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@time-on-site
Patch Set: Refactor addressing reviewer comments Created 5 years, 2 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 unified diff | Download patch
OLDNEW
(Empty)
1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "chrome/browser/engagement/site_engagement_metrics.h"
6
7 #include "base/metrics/histogram_macros.h"
8 #include "base/metrics/sparse_histogram.h"
9
10 namespace {
11
12 const char kTotalEngagementHistogram[] =
13 "SiteEngagementService.TotalEngagement";
14
15 const char kTotalOriginsHistogram[] = "SiteEngagementService.OriginsEngaged";
16
17 const char kEngagementByOriginHistogram[] =
18 "SiteEngagementService.EngagementByOrigin";
19
20 const char kOriginsWithMaxEngagementHistogram[] =
21 "SiteEngagementService.OriginsWithMaxEngagement";
22
23 const char kOriginsWithMaxDailyEngagementHistogram[] =
24 "SiteEngagementService.OriginsWithMaxDailyEngagement";
25
26 const char kEngagementTypeHistogram[] = "SiteEngagementService.EngagementType";
27
28 } // namespace
29
30 void SiteEngagementMetrics::RecordTotalSiteEngagement(
31 double total_engagement) {
32 UMA_HISTOGRAM_COUNTS_10000(kTotalEngagementHistogram, total_engagement);
33 }
34
35 void SiteEngagementMetrics::RecordTotalOriginsEngaged(int num_origins) {
36 UMA_HISTOGRAM_COUNTS_10000(kTotalOriginsHistogram, num_origins);
37 }
38
39 void SiteEngagementMetrics::RecordEngagementByOrigin(
40 std::map<GURL, double> score_map) {
41 for (const auto& value: score_map) {
42 UMA_HISTOGRAM_COUNTS_100(kEngagementByOriginHistogram, value.second);
43 }
44 }
45
46 void SiteEngagementMetrics::RecordOriginsWithMaxEngagement(int total_origins) {
47 UMA_HISTOGRAM_COUNTS_100(kOriginsWithMaxEngagementHistogram, total_origins);
48 }
49
50 void SiteEngagementMetrics::RecordOriginsWithMaxDailyEngagement(
51 int total_origins) {
52 UMA_HISTOGRAM_COUNTS_100(kOriginsWithMaxDailyEngagementHistogram,
53 total_origins);
54 }
55
56 void SiteEngagementMetrics::RecordEngagement(EngagementType type) {
57 switch (type) {
58 case ENGAGEMENT_NAVIGATION:
59 UMA_HISTOGRAM_SPARSE_SLOWLY(kEngagementTypeHistogram,
60 ENGAGEMENT_NAVIGATION);
61 break;
62 case ENGAGEMENT_KEYPRESS:
63 UMA_HISTOGRAM_SPARSE_SLOWLY(kEngagementTypeHistogram,
64 ENGAGEMENT_KEYPRESS);
65 break;
66 case ENGAGEMENT_MOUSE:
67 UMA_HISTOGRAM_SPARSE_SLOWLY(kEngagementTypeHistogram, ENGAGEMENT_MOUSE);
68 break;
69 default:
70 NOTREACHED() << "Invalid type passed to RecordUserInput().";
71 break;
72 }
73 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698