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

Side by Side Diff: chrome/browser/banners/app_banner_metrics.h

Issue 2122783002: Add metrics for app banner preventDefault() and prompt(). (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address nits Created 4 years, 5 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 2015 The Chromium Authors. All rights reserved. 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 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_BANNERS_APP_BANNER_METRICS_H_ 5 #ifndef CHROME_BROWSER_BANNERS_APP_BANNER_METRICS_H_
6 #define CHROME_BROWSER_BANNERS_APP_BANNER_METRICS_H_ 6 #define CHROME_BROWSER_BANNERS_APP_BANNER_METRICS_H_
7 7
8 namespace banners { 8 namespace banners {
9 9
10 // Keep in sync with the values defined in histograms.xml. 10 // This enum backs a UMA histogram, so it should be treated as append-only.
11 enum DisplayEvent { 11 enum DisplayEvent {
12 DISPLAY_EVENT_MIN = 0, 12 DISPLAY_EVENT_MIN = 0,
13 DISPLAY_EVENT_BANNER_REQUESTED = 1, 13 DISPLAY_EVENT_BANNER_REQUESTED = 1,
14 DISPLAY_EVENT_BLOCKED_PREVIOUSLY = 2, 14 DISPLAY_EVENT_BLOCKED_PREVIOUSLY = 2,
15 DISPLAY_EVENT_PROMOTED_TOO_MANY_OTHERS = 3, 15 DISPLAY_EVENT_PROMOTED_TOO_MANY_OTHERS = 3,
16 DISPLAY_EVENT_CREATED = 4, 16 DISPLAY_EVENT_CREATED = 4,
17 DISPLAY_EVENT_INSTALLED_PREVIOUSLY = 5, 17 DISPLAY_EVENT_INSTALLED_PREVIOUSLY = 5,
18 DISPLAY_EVENT_IGNORED_PREVIOUSLY = 6, 18 DISPLAY_EVENT_IGNORED_PREVIOUSLY = 6,
19 DISPLAY_EVENT_LACKS_SERVICE_WORKER = 7, 19 DISPLAY_EVENT_LACKS_SERVICE_WORKER = 7,
20 DISPLAY_EVENT_NOT_VISITED_ENOUGH = 8, 20 DISPLAY_EVENT_NOT_VISITED_ENOUGH = 8,
21 DISPLAY_EVENT_NATIVE_APP_BANNER_REQUESTED = 9, 21 DISPLAY_EVENT_NATIVE_APP_BANNER_REQUESTED = 9,
22 DISPLAY_EVENT_WEB_APP_BANNER_REQUESTED = 10, 22 DISPLAY_EVENT_WEB_APP_BANNER_REQUESTED = 10,
23 DISPLAY_EVENT_NATIVE_APP_BANNER_CREATED = 11, 23 DISPLAY_EVENT_NATIVE_APP_BANNER_CREATED = 11,
24 DISPLAY_EVENT_WEB_APP_BANNER_CREATED = 12, 24 DISPLAY_EVENT_WEB_APP_BANNER_CREATED = 12,
25 DISPLAY_EVENT_MAX = 13, 25 DISPLAY_EVENT_MAX = 13,
26 }; 26 };
27 27
28 // This enum backs a UMA histogram, so it should be treated as append-only.
28 enum InstallEvent { 29 enum InstallEvent {
29 INSTALL_EVENT_MIN = 20, 30 INSTALL_EVENT_MIN = 20,
30 INSTALL_EVENT_NATIVE_APP_INSTALL_TRIGGERED = 21, 31 INSTALL_EVENT_NATIVE_APP_INSTALL_TRIGGERED = 21,
31 INSTALL_EVENT_NATIVE_APP_INSTALL_STARTED = 22, 32 INSTALL_EVENT_NATIVE_APP_INSTALL_STARTED = 22,
32 INSTALL_EVENT_NATIVE_APP_INSTALL_COMPLETED = 23, 33 INSTALL_EVENT_NATIVE_APP_INSTALL_COMPLETED = 23,
33 INSTALL_EVENT_WEB_APP_INSTALLED = 24, 34 INSTALL_EVENT_WEB_APP_INSTALLED = 24,
34 INSTALL_EVENT_MAX = 25, 35 INSTALL_EVENT_MAX = 25,
35 }; 36 };
36 37
38 // This enum backs a UMA histogram, so it should be treated as append-only.
37 enum DismissEvent { 39 enum DismissEvent {
38 DISMISS_EVENT_MIN = 40, 40 DISMISS_EVENT_MIN = 40,
39 DISMISS_EVENT_ERROR = 41, 41 DISMISS_EVENT_ERROR = 41,
40 DISMISS_EVENT_APP_OPEN = 42, 42 DISMISS_EVENT_APP_OPEN = 42,
41 DISMISS_EVENT_BANNER_CLICK = 43, 43 DISMISS_EVENT_BANNER_CLICK = 43,
42 DISMISS_EVENT_BANNER_SWIPE = 44, 44 DISMISS_EVENT_BANNER_SWIPE = 44,
43 DISMISS_EVENT_CLOSE_BUTTON = 45, 45 DISMISS_EVENT_CLOSE_BUTTON = 45,
44 DISMISS_EVENT_INSTALL_TIMEOUT = 46, 46 DISMISS_EVENT_INSTALL_TIMEOUT = 46,
45 DISMISS_EVENT_DISMISSED = 47, 47 DISMISS_EVENT_DISMISSED = 47,
46 DISMISS_EVENT_MAX = 48, 48 DISMISS_EVENT_MAX = 48,
47 }; 49 };
48 50
51 // This enum backs a UMA histogram, so it should be treated as append-only.
49 enum UserResponse { 52 enum UserResponse {
50 USER_RESPONSE_MIN = 0, 53 USER_RESPONSE_MIN = 0,
51 USER_RESPONSE_NATIVE_APP_ACCEPTED = 1, 54 USER_RESPONSE_NATIVE_APP_ACCEPTED = 1,
52 USER_RESPONSE_WEB_APP_ACCEPTED = 2, 55 USER_RESPONSE_WEB_APP_ACCEPTED = 2,
53 USER_RESPONSE_NATIVE_APP_DISMISSED = 3, 56 USER_RESPONSE_NATIVE_APP_DISMISSED = 3,
54 USER_RESPONSE_WEB_APP_DISMISSED = 4, 57 USER_RESPONSE_WEB_APP_DISMISSED = 4,
55 USER_RESPONSE_NATIVE_APP_IGNORED = 5, 58 USER_RESPONSE_NATIVE_APP_IGNORED = 5,
56 USER_RESPONSE_WEB_APP_IGNORED = 6, 59 USER_RESPONSE_WEB_APP_IGNORED = 6,
57 USER_RESPONSE_MAX = 7, 60 USER_RESPONSE_MAX = 7,
58 }; 61 };
59 62
63 // This enum backs a UMA histogram, so it should be treated as append-only.
64 enum BeforeInstallEvent {
65 BEFORE_INSTALL_EVENT_MIN = 0,
66 BEFORE_INSTALL_EVENT_CREATED = 1,
67 BEFORE_INSTALL_EVENT_COMPLETE = 2,
68 BEFORE_INSTALL_EVENT_NO_ACTION = 3,
69 BEFORE_INSTALL_EVENT_PREVENT_DEFAULT_CALLED = 4,
70 BEFORE_INSTALL_EVENT_PROMPT_CALLED_AFTER_PREVENT_DEFAULT = 5,
71 BEFORE_INSTALL_EVENT_PROMPT_NOT_CALLED_AFTER_PREVENT_DEFAULT = 6,
72 BEFORE_INSTALL_EVENT_MAX = 7,
73 };
74
60 extern const char kDismissEventHistogram[]; 75 extern const char kDismissEventHistogram[];
61 extern const char kDisplayEventHistogram[]; 76 extern const char kDisplayEventHistogram[];
62 extern const char kInstallEventHistogram[]; 77 extern const char kInstallEventHistogram[];
63 extern const char kMinutesHistogram[]; 78 extern const char kMinutesHistogram[];
64 extern const char kUserResponseHistogram[]; 79 extern const char kUserResponseHistogram[];
80 extern const char kBeforeInstallEventHistogram[];
65 81
66 void TrackDismissEvent(int event); 82 void TrackDismissEvent(int event);
67 void TrackDisplayEvent(int event); 83 void TrackDisplayEvent(int event);
68 void TrackInstallEvent(int event); 84 void TrackInstallEvent(int event);
69 void TrackMinutesFromFirstVisitToBannerShown(int minutes); 85 void TrackMinutesFromFirstVisitToBannerShown(int minutes);
70 void TrackUserResponse(int event); 86 void TrackUserResponse(int event);
87 void TrackBeforeInstallEvent(int event);
71 88
72 }; // namespace banners 89 }; // namespace banners
73 90
74 #endif // CHROME_BROWSER_BANNERS_APP_BANNER_METRICS_H_ 91 #endif // CHROME_BROWSER_BANNERS_APP_BANNER_METRICS_H_
OLDNEW
« no previous file with comments | « chrome/browser/banners/app_banner_data_fetcher.cc ('k') | chrome/browser/banners/app_banner_metrics.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698