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

Side by Side Diff: chrome/browser/metrics/signin_status_metrics_provider.h

Issue 554863002: Handles unknown signin status and collect more stats. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Update tests. 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
« no previous file with comments | « no previous file | chrome/browser/metrics/signin_status_metrics_provider.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 CHROME_BROWSER_METRICS_SIGNIN_STATUS_METRICS_PROVIDER_H_ 5 #ifndef CHROME_BROWSER_METRICS_SIGNIN_STATUS_METRICS_PROVIDER_H_
6 #define CHROME_BROWSER_METRICS_SIGNIN_STATUS_METRICS_PROVIDER_H_ 6 #define CHROME_BROWSER_METRICS_SIGNIN_STATUS_METRICS_PROVIDER_H_
7 7
8 #include <string> 8 #include <string>
9 9
10 #include "base/gtest_prod_util.h" 10 #include "base/gtest_prod_util.h"
(...skipping 43 matching lines...) Expand 10 before | Expand all | Expand 10 after
54 FRIEND_TEST_ALL_PREFIXES(SigninStatusMetricsProvider, GoogleSignedOut); 54 FRIEND_TEST_ALL_PREFIXES(SigninStatusMetricsProvider, GoogleSignedOut);
55 55
56 // Possible sign-in status of all opened profiles during one UMA session. For 56 // Possible sign-in status of all opened profiles during one UMA session. For
57 // MIXED_SIGNIN_STATUS, at least one signed-in profile and at least one 57 // MIXED_SIGNIN_STATUS, at least one signed-in profile and at least one
58 // unsigned-in profile were opened between two UMA log uploads. 58 // unsigned-in profile were opened between two UMA log uploads.
59 enum ProfilesSigninStatus { 59 enum ProfilesSigninStatus {
60 ALL_PROFILES_SIGNED_IN, 60 ALL_PROFILES_SIGNED_IN,
61 ALL_PROFILES_NOT_SIGNED_IN, 61 ALL_PROFILES_NOT_SIGNED_IN,
62 MIXED_SIGNIN_STATUS, 62 MIXED_SIGNIN_STATUS,
63 UNKNOWN_SIGNIN_STATUS, 63 UNKNOWN_SIGNIN_STATUS,
64 ERROR_GETTING_SIGNIN_STATUS,
64 SIGNIN_STATUS_MAX, 65 SIGNIN_STATUS_MAX,
65 }; 66 };
66 67
67 // chrome::BrowserListObserver: 68 // chrome::BrowserListObserver:
68 // This will never be called on Android. 69 // This will never be called on Android.
69 virtual void OnBrowserAdded(Browser* browser) OVERRIDE; 70 virtual void OnBrowserAdded(Browser* browser) OVERRIDE;
70 71
71 // SigninManagerFactory::Observer: 72 // SigninManagerFactory::Observer:
72 virtual void SigninManagerCreated(SigninManagerBase* manager) OVERRIDE; 73 virtual void SigninManagerCreated(SigninManagerBase* manager) OVERRIDE;
73 virtual void SigninManagerShutdown(SigninManagerBase* manager) OVERRIDE; 74 virtual void SigninManagerShutdown(SigninManagerBase* manager) OVERRIDE;
(...skipping 34 matching lines...) Expand 10 before | Expand all | Expand 10 after
108 109
109 // Whether the instance is for testing or not. 110 // Whether the instance is for testing or not.
110 bool is_test_; 111 bool is_test_;
111 112
112 base::WeakPtrFactory<SigninStatusMetricsProvider> weak_ptr_factory_; 113 base::WeakPtrFactory<SigninStatusMetricsProvider> weak_ptr_factory_;
113 114
114 DISALLOW_COPY_AND_ASSIGN(SigninStatusMetricsProvider); 115 DISALLOW_COPY_AND_ASSIGN(SigninStatusMetricsProvider);
115 }; 116 };
116 117
117 #endif // CHROME_BROWSER_METRICS_SIGNIN_STATUS_METRICS_PROVIDER_H_ 118 #endif // CHROME_BROWSER_METRICS_SIGNIN_STATUS_METRICS_PROVIDER_H_
OLDNEW
« no previous file with comments | « no previous file | chrome/browser/metrics/signin_status_metrics_provider.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698