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

Unified Diff: chrome/browser/sync/profile_sync_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/sync/profile_sync_service_unittest.cc
diff --git a/chrome/browser/sync/profile_sync_service_unittest.cc b/chrome/browser/sync/profile_sync_service_unittest.cc
index 8e80aa22ca2902cdbe88faad410c11f18cb33914..44361a292e81790ecb4ac889dc48e86c7e3e5ac0 100644
--- a/chrome/browser/sync/profile_sync_service_unittest.cc
+++ b/chrome/browser/sync/profile_sync_service_unittest.cc
@@ -208,7 +208,8 @@ class ProfileSyncServiceTest : public ::testing::Test {
#if defined(OS_WIN) || defined(OS_MACOSX) || (defined(OS_LINUX) && !defined(OS_CHROMEOS))
void CreateServiceWithoutSignIn() {
CreateService(browser_sync::MANUAL_START);
- SigninManagerFactory::GetForProfile(profile())->SignOut();
+ SigninManagerFactory::GetForProfile(profile())->SignOut(
+ signin_metrics::SIGNOUT_TEST);
service()->SetBackupStartDelayForTest(
base::TimeDelta::FromMilliseconds(100));
}
@@ -430,7 +431,8 @@ TEST_F(ProfileSyncServiceTest, EnableSyncAndSignOutDesktop) {
sync_driver::prefs::kSyncSuppressStart));
EXPECT_EQ(ProfileSyncService::SYNC, service()->backend_mode());
- SigninManagerFactory::GetForProfile(profile())->SignOut();
+ SigninManagerFactory::GetForProfile(profile())->SignOut(
+ signin_metrics::SIGNOUT_TEST);
EXPECT_TRUE(service()->sync_initialized());
EXPECT_EQ(ProfileSyncService::BACKUP, service()->backend_mode());
}
@@ -446,7 +448,8 @@ TEST_F(ProfileSyncServiceTest, EnableSyncAndSignOut) {
EXPECT_FALSE(profile()->GetPrefs()->GetBoolean(
sync_driver::prefs::kSyncSuppressStart));
- SigninManagerFactory::GetForProfile(profile())->SignOut();
+ SigninManagerFactory::GetForProfile(profile())->SignOut(
+ signin_metrics::SIGNOUT_TEST);
EXPECT_FALSE(service()->sync_initialized());
}
#endif // !defined(OS_CHROMEOS)
« no previous file with comments | « chrome/browser/sync/profile_sync_service.cc ('k') | chrome/browser/ui/sync/one_click_signin_sync_starter.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698