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

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

Issue 590113004: Handle account removal correctly on all platforms. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Modify metrics Created 6 years, 3 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 namespace signin_metrics { 10 namespace signin_metrics {
(...skipping 35 matching lines...) Expand 10 before | Expand all | Expand 10 after
46 46
47 // Keep this as the last enum. 47 // Keep this as the last enum.
48 NUM_PROFILE_SIGNOUT_METRICS, 48 NUM_PROFILE_SIGNOUT_METRICS,
49 }; 49 };
50 50
51 // Log to UMA histograms and UserCounts stats about a single execution of the 51 // Log to UMA histograms and UserCounts stats about a single execution of the
52 // AccountReconciler. 52 // AccountReconciler.
53 // |total_number_accounts| - How many accounts are in the browser for this 53 // |total_number_accounts| - How many accounts are in the browser for this
54 // profile. 54 // profile.
55 // |count_added_to_cookie_jar| - How many accounts were in the browser but not 55 // |count_added_to_cookie_jar| - How many accounts were in the browser but not
56 // the cookie jar. 56 // in the cookie jar.
Mike Lerman 2014/09/23 14:27:47 Nit: add a space to left-align the comments
Roger Tawa OOO till Jul 10th 2014/09/23 21:25:59 Done.
57 // |count_added_to_token| - How may accounts were in the cookie jar but not in 57 // |count_removed_from_cookie_jar| - How many accounts were in the cookie jar
58 // the browser. 58 // but not in the browser.
59 // |primary_accounts_same| - False if the primary account for the cookie jar 59 // |primary_accounts_same| - False if the primary account for the cookie jar
60 // and the token service were different; else true. 60 // and the token service were different; else true.
61 // |is_first_reconcile| - True if these stats are from the first execution of 61 // |is_first_reconcile| - True if these stats are from the first execution of
62 // the AccountReconcilor. 62 // the AccountReconcilor.
63 // |pre_count_gaia_cookies| - How many GAIA cookies were present before 63 // |pre_count_gaia_cookies| - How many GAIA cookies were present before
64 // the AccountReconcilor began modifying the state. 64 // the AccountReconcilor began modifying the state.
65 void LogSigninAccountReconciliation(int total_number_accounts, 65 void LogSigninAccountReconciliation(int total_number_accounts,
66 int count_added_to_cookie_jar, 66 int count_added_to_cookie_jar,
67 int count_added_to_token, 67 int count_removed_from_cookie_jar,
68 bool primary_accounts_same, 68 bool primary_accounts_same,
69 bool is_first_reconcile, 69 bool is_first_reconcile,
70 int pre_count_gaia_cookies); 70 int pre_count_gaia_cookies);
71 71
72 // Track a successful signin. 72 // Track a successful signin.
73 void LogSigninAddAccount(); 73 void LogSigninAddAccount();
74 74
75 // Track a successful signin of a profile. 75 // Track a successful signin of a profile.
76 void LogSigninProfile(bool is_first_run, base::Time install_date); 76 void LogSigninProfile(bool is_first_run, base::Time install_date);
77 77
78 // Track a profile signout. 78 // Track a profile signout.
79 void LogSignout(ProfileSignout metric); 79 void LogSignout(ProfileSignout metric);
80 80
81 // Tracks whether the external connection results were all fetched before 81 // Tracks whether the external connection results were all fetched before
82 // the reconcilor tried to use them with MergeSession. 82 // the reconcilor tried to use them with MergeSession.
83 void LogExternalCcResultFetches(bool fetches_completed); 83 void LogExternalCcResultFetches(bool fetches_completed);
84 84
85 } // namespace signin_metrics 85 } // namespace signin_metrics
86 86
87 #endif // COMPONENTS_SIGNIN_CORE_BROWSER_SIGNIN_METRICS_H_ 87 #endif // COMPONENTS_SIGNIN_CORE_BROWSER_SIGNIN_METRICS_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698