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

Side by Side Diff: components/signin/core/browser/signin_metrics.h

Issue 1578433002: Implementation of newly designed sign in related histograms for Android (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: import RecordHistogram in SigninManager Created 4 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 unified diff | Download patch
« no previous file with comments | « chrome/chrome.gyp ('k') | tools/android/eclipse/.classpath » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 COMPONENTS_SIGNIN_CORE_BROWSER_SIGNIN_METRICS_H_ 5 #ifndef COMPONENTS_SIGNIN_CORE_BROWSER_SIGNIN_METRICS_H_
6 #define COMPONENTS_SIGNIN_CORE_BROWSER_SIGNIN_METRICS_H_ 6 #define COMPONENTS_SIGNIN_CORE_BROWSER_SIGNIN_METRICS_H_
7 7
8 #include <limits.h> 8 #include <limits.h>
9 9
10 #include "base/time/time.h" 10 #include "base/time/time.h"
(...skipping 97 matching lines...) Expand 10 before | Expand all | Expand 10 after
108 enum Source { 108 enum Source {
109 SOURCE_START_PAGE = 0, // This must be first. 109 SOURCE_START_PAGE = 0, // This must be first.
110 SOURCE_SETTINGS = 3, 110 SOURCE_SETTINGS = 3,
111 SOURCE_OTHERS = 13, 111 SOURCE_OTHERS = 13,
112 }; 112 };
113 113
114 // Enum values which enumerates all access points where sign in could be 114 // Enum values which enumerates all access points where sign in could be
115 // initiated. Not all of them exist on all platforms. They are used with 115 // initiated. Not all of them exist on all platforms. They are used with
116 // "Signin.SigninStartedAccessPoint" and "Signin.SigninCompletedAccessPoint" 116 // "Signin.SigninStartedAccessPoint" and "Signin.SigninCompletedAccessPoint"
117 // histograms. 117 // histograms.
118 // A Java counterpart will be generated for this enum.
119 // GENERATED_JAVA_ENUM_PACKAGE: org.chromium.chrome.browser.signin
120 // GENERATED_JAVA_CLASS_NAME_OVERRIDE: SigninAccessPoint
118 enum class AccessPoint : int { 121 enum class AccessPoint : int {
119 ACCESS_POINT_START_PAGE = 0, 122 ACCESS_POINT_START_PAGE = 0,
120 ACCESS_POINT_NTP_LINK, 123 ACCESS_POINT_NTP_LINK,
121 ACCESS_POINT_MENU, 124 ACCESS_POINT_MENU,
122 ACCESS_POINT_SETTINGS, 125 ACCESS_POINT_SETTINGS,
123 ACCESS_POINT_SUPERVISED_USER, 126 ACCESS_POINT_SUPERVISED_USER,
124 ACCESS_POINT_EXTENSION_INSTALL_BUBBLE, 127 ACCESS_POINT_EXTENSION_INSTALL_BUBBLE,
125 ACCESS_POINT_EXTENSIONS, 128 ACCESS_POINT_EXTENSIONS,
126 ACCESS_POINT_APPS_PAGE_LINK, 129 ACCESS_POINT_APPS_PAGE_LINK,
127 ACCESS_POINT_BOOKMARK_BUBBLE, 130 ACCESS_POINT_BOOKMARK_BUBBLE,
128 ACCESS_POINT_BOOKMARK_MANAGER, 131 ACCESS_POINT_BOOKMARK_MANAGER,
129 ACCESS_POINT_AVATAR_BUBBLE_SIGN_IN, 132 ACCESS_POINT_AVATAR_BUBBLE_SIGN_IN,
130 ACCESS_POINT_USER_MANAGER, 133 ACCESS_POINT_USER_MANAGER,
131 ACCESS_POINT_DEVICES_PAGE, 134 ACCESS_POINT_DEVICES_PAGE,
132 ACCESS_POINT_CLOUD_PRINT, 135 ACCESS_POINT_CLOUD_PRINT,
133 ACCESS_POINT_CONTENT_AREA, 136 ACCESS_POINT_CONTENT_AREA,
134 ACCESS_POINT_SIGNIN_PROMO, 137 ACCESS_POINT_SIGNIN_PROMO,
135 ACCESS_POINT_RECENT_TABS, 138 ACCESS_POINT_RECENT_TABS,
136 ACCESS_POINT_UNKNOWN, // This should never have been used to get signin URL. 139 ACCESS_POINT_UNKNOWN, // This should never have been used to get signin URL.
137 ACCESS_POINT_MAX, // This must be last. 140 ACCESS_POINT_MAX, // This must be last.
138 }; 141 };
139 142
140 // Enum values which enumerates all reasons to start sign in process. 143 // Enum values which enumerates all reasons to start sign in process.
144 // A Java counterpart will be generated for this enum.
145 // GENERATED_JAVA_ENUM_PACKAGE: org.chromium.chrome.browser.signin
146 // GENERATED_JAVA_CLASS_NAME_OVERRIDE: SigninReason
141 enum class Reason : int { 147 enum class Reason : int {
142 REASON_SIGNIN_PRIMARY_ACCOUNT = 0, 148 REASON_SIGNIN_PRIMARY_ACCOUNT = 0,
143 REASON_ADD_SECONDARY_ACCOUNT, 149 REASON_ADD_SECONDARY_ACCOUNT,
144 REASON_REAUTHENTICATION, 150 REASON_REAUTHENTICATION,
145 REASON_UNLOCK, 151 REASON_UNLOCK,
146 REASON_UNKNOWN_REASON, // This should never have been used to get signin URL. 152 REASON_UNKNOWN_REASON, // This should never have been used to get signin URL.
147 REASON_MAX, // This must be last. 153 REASON_MAX, // This must be last.
148 }; 154 };
149 155
150 // Enum values used for use with the "Signin.Reauth" histogram. 156 // Enum values used for use with the "Signin.Reauth" histogram.
(...skipping 152 matching lines...) Expand 10 before | Expand all | Expand 10 after
303 // be shown a different set of accounts in the content-area and the settings UI. 309 // be shown a different set of accounts in the content-area and the settings UI.
304 void LogAccountReconcilorStateOnGaiaResponse(AccountReconcilorState state); 310 void LogAccountReconcilorStateOnGaiaResponse(AccountReconcilorState state);
305 311
306 // Records the AccountEquality metric when an investigator compares the current 312 // Records the AccountEquality metric when an investigator compares the current
307 // and previous id/emails during a signin. 313 // and previous id/emails during a signin.
308 void LogAccountEquality(AccountEquality equality); 314 void LogAccountEquality(AccountEquality equality);
309 315
310 } // namespace signin_metrics 316 } // namespace signin_metrics
311 317
312 #endif // COMPONENTS_SIGNIN_CORE_BROWSER_SIGNIN_METRICS_H_ 318 #endif // COMPONENTS_SIGNIN_CORE_BROWSER_SIGNIN_METRICS_H_
OLDNEW
« no previous file with comments | « chrome/chrome.gyp ('k') | tools/android/eclipse/.classpath » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698