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

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

Issue 813133003: New Profile metric for Auth Errors. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: ChromeSigninClient handles no ProfileManager for tests Created 5 years, 11 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 "base/time/time.h" 8 #include "base/time/time.h"
9 9
10 #include "google_apis/gaia/google_service_auth_error.h"
11
10 namespace signin_metrics { 12 namespace signin_metrics {
11 13
12 // Enum for the ways in which primary account detection is done. 14 // Enum for the ways in which primary account detection is done.
13 enum DifferentPrimaryAccounts { 15 enum DifferentPrimaryAccounts {
14 // token and cookie had same primary accounts. 16 // token and cookie had same primary accounts.
15 ACCOUNTS_SAME = 0, 17 ACCOUNTS_SAME = 0,
16 // Deprecated. Indicates different primary accounts. 18 // Deprecated. Indicates different primary accounts.
17 UNUSED_ACCOUNTS_DIFFERENT, 19 UNUSED_ACCOUNTS_DIFFERENT,
18 // No GAIA cookie present, so the primaries are considered different. 20 // No GAIA cookie present, so the primaries are considered different.
19 NO_COOKIE_PRESENT, 21 NO_COOKIE_PRESENT,
(...skipping 137 matching lines...) Expand 10 before | Expand all | Expand 10 after
157 // Track a profile signout. 159 // Track a profile signout.
158 void LogSignout(ProfileSignout metric); 160 void LogSignout(ProfileSignout metric);
159 161
160 // Tracks whether the external connection results were all fetched before 162 // Tracks whether the external connection results were all fetched before
161 // the reconcilor tried to use them with MergeSession. 163 // the reconcilor tried to use them with MergeSession.
162 // |time_to_check_connections| is the time it took to complete. 164 // |time_to_check_connections| is the time it took to complete.
163 void LogExternalCcResultFetches( 165 void LogExternalCcResultFetches(
164 bool fetches_completed, 166 bool fetches_completed,
165 const base::TimeDelta& time_to_check_connections); 167 const base::TimeDelta& time_to_check_connections);
166 168
169 // Track when the current authentication error changed.
170 void LogAuthError(GoogleServiceAuthError::State auth_error);
171
167 } // namespace signin_metrics 172 } // namespace signin_metrics
168 173
169 #endif // COMPONENTS_SIGNIN_CORE_BROWSER_SIGNIN_METRICS_H_ 174 #endif // COMPONENTS_SIGNIN_CORE_BROWSER_SIGNIN_METRICS_H_
OLDNEW
« no previous file with comments | « components/signin/core/browser/signin_error_controller.cc ('k') | components/signin/core/browser/signin_metrics.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698