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

Side by Side Diff: chrome/browser/ntp_snippets/ntp_snippets_metrics.h

Issue 2790183002: Add UMA for the content suggestions settings (Closed)
Patch Set: rename methods to remove service mention Created 3 years, 8 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
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_NTP_SNIPPETS_NTP_SNIPPETS_METRICS_H_ 5 #ifndef CHROME_BROWSER_NTP_SNIPPETS_NTP_SNIPPETS_METRICS_H_
6 #define CHROME_BROWSER_NTP_SNIPPETS_NTP_SNIPPETS_METRICS_H_ 6 #define CHROME_BROWSER_NTP_SNIPPETS_NTP_SNIPPETS_METRICS_H_
7 7
8 enum ContentSuggestionsNotificationImpression { 8 enum ContentSuggestionsNotificationImpression {
9 CONTENT_SUGGESTIONS_ARTICLE = 0, // Server-provided "articles" category. 9 CONTENT_SUGGESTIONS_ARTICLE = 0, // Server-provided "articles" category.
10 CONTENT_SUGGESTIONS_NONARTICLE, // Anything else. 10 CONTENT_SUGGESTIONS_NONARTICLE, // Anything else.
11 11
12 MAX_CONTENT_SUGGESTIONS_NOTIFICATION_IMPRESSION 12 MAX_CONTENT_SUGGESTIONS_NOTIFICATION_IMPRESSION
13 }; 13 };
14 14
15 // GENERATED_JAVA_ENUM_PACKAGE: org.chromium.chrome.browser.ntp.snippets 15 // GENERATED_JAVA_ENUM_PACKAGE: org.chromium.chrome.browser.ntp.snippets
16 // GENERATED_JAVA_PREFIX_TO_STRIP: CONTENT_SUGGESTIONS_
Bernhard Bauer 2017/04/10 23:16:03 That makes me wonder whether it would make sense t
dgn 2017/04/10 23:31:29 Definitely, I was thinking about that. But I'd do
16 enum ContentSuggestionsNotificationAction { 17 enum ContentSuggestionsNotificationAction {
17 CONTENT_SUGGESTIONS_TAP = 0, // User tapped notification to open article. 18 CONTENT_SUGGESTIONS_TAP = 0, // User tapped notification to open article.
18 CONTENT_SUGGESTIONS_DISMISSAL, // User swiped notification to dismiss it. 19 CONTENT_SUGGESTIONS_DISMISSAL, // User swiped notification to dismiss it.
19 20
20 CONTENT_SUGGESTIONS_HIDE_DEADLINE, // notification_extra().deadline passed. 21 CONTENT_SUGGESTIONS_HIDE_DEADLINE, // notification_extra().deadline passed.
21 CONTENT_SUGGESTIONS_HIDE_EXPIRY, // NTP no longer shows notified article. 22 CONTENT_SUGGESTIONS_HIDE_EXPIRY, // NTP no longer shows notified article.
22 CONTENT_SUGGESTIONS_HIDE_FRONTMOST, // Chrome became the frontmost app. 23 CONTENT_SUGGESTIONS_HIDE_FRONTMOST, // Chrome became the frontmost app.
23 CONTENT_SUGGESTIONS_HIDE_DISABLED, // NTP no longer shows whole category. 24 CONTENT_SUGGESTIONS_HIDE_DISABLED, // NTP no longer shows whole category.
24 CONTENT_SUGGESTIONS_HIDE_SHUTDOWN, // Content sugg service is shutting down. 25 CONTENT_SUGGESTIONS_HIDE_SHUTDOWN, // Content sugg service is shutting down.
25 26
27 CONTENT_SUGGESTIONS_OPEN_SETTINGS, // User opened settings from notification.
28
26 MAX_CONTENT_SUGGESTIONS_NOTIFICATION_ACTION 29 MAX_CONTENT_SUGGESTIONS_NOTIFICATION_ACTION
27 }; 30 };
28 31
32 // GENERATED_JAVA_ENUM_PACKAGE: org.chromium.chrome.browser.ntp.snippets
33 // GENERATED_JAVA_PREFIX_TO_STRIP: CONTENT_SUGGESTIONS_
29 enum ContentSuggestionsNotificationOptOut { 34 enum ContentSuggestionsNotificationOptOut {
30 CONTENT_SUGGESTIONS_IMPLICIT = 0, // User ignored notifications. 35 CONTENT_SUGGESTIONS_IMPLICIT = 0, // User ignored notifications.
31 CONTENT_SUGGESTIONS_EXPLICIT, // User explicitly opted-out. 36 CONTENT_SUGGESTIONS_EXPLICIT, // User explicitly opted-out.
32 37
33 MAX_CONTENT_SUGGESTIONS_NOTIFICATION_OPT_OUT 38 MAX_CONTENT_SUGGESTIONS_NOTIFICATION_OPT_OUT
34 }; 39 };
35 40
36 void RecordContentSuggestionsNotificationImpression( 41 void RecordContentSuggestionsNotificationImpression(
37 ContentSuggestionsNotificationImpression what); 42 ContentSuggestionsNotificationImpression what);
38 void RecordContentSuggestionsNotificationAction( 43 void RecordContentSuggestionsNotificationAction(
39 ContentSuggestionsNotificationAction what); 44 ContentSuggestionsNotificationAction what);
40 void RecordContentSuggestionsNotificationOptOut( 45 void RecordContentSuggestionsNotificationOptOut(
41 ContentSuggestionsNotificationOptOut what); 46 ContentSuggestionsNotificationOptOut what);
42 47
43 #endif // CHROME_BROWSER_NTP_SNIPPETS_NTP_SNIPPETS_METRICS_H_ 48 #endif // CHROME_BROWSER_NTP_SNIPPETS_NTP_SNIPPETS_METRICS_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698