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

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

Issue 338573002: Differentiate primary account compares on GAIA cookie presence. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 6 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 namespace signin_metrics { 8 namespace signin_metrics {
9 9
10 // Log to UMA histograms and UserCounts stats about a single execution of the 10 // Log to UMA histograms and UserCounts stats about a single execution of the
11 // AccountReconciler. 11 // AccountReconciler.
12 // |total_number_accounts| - How many accounts are in the browser for this 12 // |total_number_accounts| - How many accounts are in the browser for this
13 // profile. 13 // profile.
14 // |count_added_to_cookie_jar| - How many accounts were in the browser but not 14 // |count_added_to_cookie_jar| - How many accounts were in the browser but not
15 // the cookie jar. 15 // the cookie jar.
16 // |count_added_to_token| - How may accounts were in the cookie jar but not in 16 // |count_added_to_token| - How may accounts were in the cookie jar but not in
17 // the browser. 17 // the browser.
18 // |primary_accounts_same| - False if the primary account for the cookie jar 18 // |primary_accounts_same| - False if the primary account for the cookie jar
19 // and the token service were different; else true. 19 // and the token service were different; else true.
20 // |is_first_reconcile| - True if these stats are from the first execution of 20 // |is_first_reconcile| - True if these stats are from the first execution of
21 // the AccountReconcilor. 21 // the AccountReconcilor.
22 void LogSigninAccountReconciliation(int total_number_accounts, 22 void LogSigninAccountReconciliation(int total_number_accounts,
23 int count_added_to_cookie_jar, 23 int count_added_to_cookie_jar,
24 int count_added_to_token, 24 int count_added_to_token,
25 bool primary_accounts_same, 25 bool primary_accounts_same,
26 bool is_first_reconcile); 26 bool is_first_reconcile,
27 int pre_count_gaia_cookies);
Alexei Svitkine (slow) 2014/06/17 19:43:04 Document this param in the comment.
27 28
28 // Track a successful signin. 29 // Track a successful signin.
29 void LogSigninAddAccount(); 30 void LogSigninAddAccount();
30 31
31 } // namespace signin_metrics 32 } // namespace signin_metrics
32 33
33 #endif // COMPONENTS_SIGNIN_CORE_BROWSER_SIGNIN_METRICS_H_ 34 #endif // COMPONENTS_SIGNIN_CORE_BROWSER_SIGNIN_METRICS_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698