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

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

Issue 270673003: UMA metrics - number of accounts per profile and account reconciler. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Signin Metrics becomes a namespace; formatting nits Created 6 years, 7 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 | Annotate | Revision Log
OLDNEW
(Empty)
1 // Copyright (c) 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #ifndef COMPONENTS_SIGNIN_CORE_BROWSER_SIGNIN_METRICS_H_
6 #define COMPONENTS_SIGNIN_CORE_BROWSER_SIGNIN_METRICS_H_
7
8 #include <stddef.h>
Alexei Svitkine (slow) 2014/05/08 14:39:29 Nit: I don't think this is needed.
Mike Lerman 2014/05/08 18:23:14 Done.
9
10 namespace signin_metrics {
Alexei Svitkine (slow) 2014/05/08 14:39:29 Nit: Add an empty line after this.
Mike Lerman 2014/05/08 18:23:14 Done.
11 // Log to UMA histograms and UserCounts stats about a single execution of the
12 // AccountReconciler.
13 // |total_number_accounts| - How many accounts are in the browser for this
14 // profile.
15 // |count_added_to_cookie_jar| - How many accounts were in the browser but not
16 // the cookie jar.
17 // |count_added_to_token| - How may accounts were in the cookie jar but not in
18 // the browser.
19 // |primary_accounts_same| - False if the primary account for the cookie jar
20 // and the token service were different; else true.
21 // |is_first_reconcile| - True if these stats are from the first execution of
22 // the AccountReconcilor.
23 void LogSigninAccountReconciliation(int total_number_accounts,
Alexei Svitkine (slow) 2014/05/08 14:39:29 Nit: Don't indent in a namespace.
Mike Lerman 2014/05/08 18:23:14 Done.
24 int count_added_to_cookie_jar,
25 int count_added_to_token,
26 bool primary_accounts_same,
27 bool is_first_reconcile);
28
29 } // namespace signin_metrics
30
31
32 #endif // COMPONENTS_SIGNIN_CORE_BROWSER_SIGNIN_METRICS_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698