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

Unified Diff: chrome/browser/sync/sync_prefs_unittest.cc

Issue 59793008: sync: don't allow NULL profile in ProfileSyncService and friends (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix test Created 7 years, 1 month 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
« no previous file with comments | « chrome/browser/sync/sync_prefs.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/sync/sync_prefs_unittest.cc
diff --git a/chrome/browser/sync/sync_prefs_unittest.cc b/chrome/browser/sync/sync_prefs_unittest.cc
index 41d2d5609f0570f88cbaa0f045201b2c042d27ab..357ae27efbaa471d7ecb6043f403c8ee4ef8be7f 100644
--- a/chrome/browser/sync/sync_prefs_unittest.cc
+++ b/chrome/browser/sync/sync_prefs_unittest.cc
@@ -232,19 +232,6 @@ TEST_F(SyncPrefsTest, ClearPreferences) {
EXPECT_TRUE(sync_prefs.GetEncryptionBootstrapToken().empty());
}
-TEST_F(SyncPrefsTest, NullPrefService) {
- SyncPrefs sync_prefs(NULL);
-
- EXPECT_FALSE(sync_prefs.HasSyncSetupCompleted());
- EXPECT_FALSE(sync_prefs.IsStartSuppressed());
- EXPECT_EQ(base::Time(), sync_prefs.GetLastSyncedTime());
- EXPECT_FALSE(sync_prefs.HasKeepEverythingSynced());
- const syncer::ModelTypeSet user_types = syncer::UserTypes();
- EXPECT_TRUE(sync_prefs.GetPreferredDataTypes(user_types).Empty());
- EXPECT_FALSE(sync_prefs.IsManaged());
- EXPECT_TRUE(sync_prefs.GetEncryptionBootstrapToken().empty());
-}
-
} // namespace
} // namespace browser_sync
« no previous file with comments | « chrome/browser/sync/sync_prefs.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698