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

Unified Diff: chrome/browser/policy/cloud/user_policy_signin_service_unittest.cc

Issue 338993007: Track when and how a profile is signed out via UMA. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: SignOut() param in Android tests 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/policy/cloud/user_policy_signin_service_unittest.cc
diff --git a/chrome/browser/policy/cloud/user_policy_signin_service_unittest.cc b/chrome/browser/policy/cloud/user_policy_signin_service_unittest.cc
index 5a9d019dfe63f509ccf854eae3c30b58ea3167d5..d78256e12940486a3e209235b3bfd8967db14ab0 100644
--- a/chrome/browser/policy/cloud/user_policy_signin_service_unittest.cc
+++ b/chrome/browser/policy/cloud/user_policy_signin_service_unittest.cc
@@ -88,7 +88,7 @@ class SigninManagerFake : public FakeSigninManager {
void ForceSignOut() {
// Allow signing out now.
prohibit_signout_ = false;
- SignOut();
+ SignOut(signin_metrics::SIGNOUT_TEST);
}
static KeyedService* Build(content::BrowserContext* profile) {
@@ -576,7 +576,8 @@ TEST_F(UserPolicySigninServiceSignedInTest, SignOutAfterInit) {
EXPECT_CALL(*mock_store_, Clear());
// Now sign out.
- SigninManagerFactory::GetForProfile(profile_.get())->SignOut();
+ SigninManagerFactory::GetForProfile(profile_.get())->SignOut(
+ signin_metrics::SIGNOUT_TEST);
// UserCloudPolicyManager should be shut down.
ASSERT_FALSE(manager_->core()->service());

Powered by Google App Engine
This is Rietveld 408576698