Chromium Code Reviews| OLD | NEW |
|---|---|
| 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 121 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 132 ACCESS_POINT_AVATAR_BUBBLE_SIGN_IN, | 132 ACCESS_POINT_AVATAR_BUBBLE_SIGN_IN, |
| 133 ACCESS_POINT_USER_MANAGER, | 133 ACCESS_POINT_USER_MANAGER, |
| 134 ACCESS_POINT_DEVICES_PAGE, | 134 ACCESS_POINT_DEVICES_PAGE, |
| 135 ACCESS_POINT_CLOUD_PRINT, | 135 ACCESS_POINT_CLOUD_PRINT, |
| 136 ACCESS_POINT_CONTENT_AREA, | 136 ACCESS_POINT_CONTENT_AREA, |
| 137 ACCESS_POINT_SIGNIN_PROMO, | 137 ACCESS_POINT_SIGNIN_PROMO, |
| 138 ACCESS_POINT_RECENT_TABS, | 138 ACCESS_POINT_RECENT_TABS, |
| 139 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. |
| 140 ACCESS_POINT_PASSWORD_BUBBLE, | 140 ACCESS_POINT_PASSWORD_BUBBLE, |
| 141 ACCESS_POINT_AUTOFILL_DROPDOWN, | 141 ACCESS_POINT_AUTOFILL_DROPDOWN, |
| 142 ACCESS_POINT_RESIGN_IN_INFOBAR, | |
|
Roger Tawa OOO till Jul 10th
2016/08/17 17:20:22
The separation with _ seems little off to me. I'd
bzanotti
2016/08/18 08:45:24
I was following the naming we've been using downst
| |
| 143 ACCESS_POINT_TAB_SWITCHER, | |
| 142 ACCESS_POINT_MAX, // This must be last. | 144 ACCESS_POINT_MAX, // This must be last. |
| 143 }; | 145 }; |
| 144 | 146 |
| 145 // Enum values which enumerates all reasons to start sign in process. | 147 // Enum values which enumerates all reasons to start sign in process. |
| 146 // A Java counterpart will be generated for this enum. | 148 // A Java counterpart will be generated for this enum. |
| 147 // GENERATED_JAVA_ENUM_PACKAGE: org.chromium.chrome.browser.signin | 149 // GENERATED_JAVA_ENUM_PACKAGE: org.chromium.chrome.browser.signin |
| 148 // GENERATED_JAVA_CLASS_NAME_OVERRIDE: SigninReason | 150 // GENERATED_JAVA_CLASS_NAME_OVERRIDE: SigninReason |
| 149 enum class Reason : int { | 151 enum class Reason : int { |
| 150 REASON_SIGNIN_PRIMARY_ACCOUNT = 0, | 152 REASON_SIGNIN_PRIMARY_ACCOUNT = 0, |
| 151 REASON_ADD_SECONDARY_ACCOUNT, | 153 REASON_ADD_SECONDARY_ACCOUNT, |
| (...skipping 251 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 403 UMA_HISTOGRAM_ENUMERATION(name "_Periodic", sample, boundary_value); \ | 405 UMA_HISTOGRAM_ENUMERATION(name "_Periodic", sample, boundary_value); \ |
| 404 break; \ | 406 break; \ |
| 405 case ReportingType::ON_CHANGE: \ | 407 case ReportingType::ON_CHANGE: \ |
| 406 UMA_HISTOGRAM_ENUMERATION(name "_OnChange", sample, boundary_value); \ | 408 UMA_HISTOGRAM_ENUMERATION(name "_OnChange", sample, boundary_value); \ |
| 407 break; \ | 409 break; \ |
| 408 } | 410 } |
| 409 | 411 |
| 410 } // namespace signin_metrics | 412 } // namespace signin_metrics |
| 411 | 413 |
| 412 #endif // COMPONENTS_SIGNIN_CORE_BROWSER_SIGNIN_METRICS_H_ | 414 #endif // COMPONENTS_SIGNIN_CORE_BROWSER_SIGNIN_METRICS_H_ |
| OLD | NEW |