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

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

Issue 2230183005: Add UMA for signin through NTP content suggestions. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Merge. Created 4 years, 4 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 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
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_NTP_CONTENT_SUGGESTIONS,
142 ACCESS_POINT_MAX, // This must be last. 143 ACCESS_POINT_MAX, // This must be last.
143 }; 144 };
144 145
145 // Enum values which enumerates all reasons to start sign in process. 146 // Enum values which enumerates all reasons to start sign in process.
146 // A Java counterpart will be generated for this enum. 147 // A Java counterpart will be generated for this enum.
147 // GENERATED_JAVA_ENUM_PACKAGE: org.chromium.chrome.browser.signin 148 // GENERATED_JAVA_ENUM_PACKAGE: org.chromium.chrome.browser.signin
148 // GENERATED_JAVA_CLASS_NAME_OVERRIDE: SigninReason 149 // GENERATED_JAVA_CLASS_NAME_OVERRIDE: SigninReason
149 enum class Reason : int { 150 enum class Reason : int {
150 REASON_SIGNIN_PRIMARY_ACCOUNT = 0, 151 REASON_SIGNIN_PRIMARY_ACCOUNT = 0,
151 REASON_ADD_SECONDARY_ACCOUNT, 152 REASON_ADD_SECONDARY_ACCOUNT,
(...skipping 251 matching lines...) Expand 10 before | Expand all | Expand 10 after
403 UMA_HISTOGRAM_ENUMERATION(name "_Periodic", sample, boundary_value); \ 404 UMA_HISTOGRAM_ENUMERATION(name "_Periodic", sample, boundary_value); \
404 break; \ 405 break; \
405 case ReportingType::ON_CHANGE: \ 406 case ReportingType::ON_CHANGE: \
406 UMA_HISTOGRAM_ENUMERATION(name "_OnChange", sample, boundary_value); \ 407 UMA_HISTOGRAM_ENUMERATION(name "_OnChange", sample, boundary_value); \
407 break; \ 408 break; \
408 } 409 }
409 410
410 } // namespace signin_metrics 411 } // namespace signin_metrics
411 412
412 #endif // COMPONENTS_SIGNIN_CORE_BROWSER_SIGNIN_METRICS_H_ 413 #endif // COMPONENTS_SIGNIN_CORE_BROWSER_SIGNIN_METRICS_H_
OLDNEW
« no previous file with comments | « chrome/browser/ui/webui/signin/inline_login_handler.cc ('k') | tools/metrics/actions/actions.xml » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698