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 dd477bcfee8b33247c8f20617ae1e54b7b775707..4bfe2dcd39b54184f91d6d244b5470a3203bda7f 100644 |
--- a/chrome/browser/sync/sync_prefs_unittest.cc |
+++ b/chrome/browser/sync/sync_prefs_unittest.cc |
@@ -26,16 +26,9 @@ class SyncPrefsTest : public testing::Test { |
MessageLoop loop_; |
}; |
-// Get all types with a user-facing component. |
-syncer::ModelTypeSet GetNonPassiveTypes() { |
- syncer::ModelTypeSet non_passive_types = syncer::ModelTypeSet::All(); |
- non_passive_types.Remove(syncer::NIGORI); |
- return non_passive_types; |
-} |
- |
// Returns all types visible from the setup UI. |
syncer::ModelTypeSet GetUserVisibleTypes() { |
- syncer::ModelTypeSet user_visible_types(GetNonPassiveTypes()); |
+ syncer::ModelTypeSet user_visible_types(syncer::UserTypes()); |
user_visible_types.Remove(syncer::APP_NOTIFICATIONS); |
user_visible_types.Remove(syncer::APP_SETTINGS); |
user_visible_types.Remove(syncer::AUTOFILL_PROFILE); |
@@ -78,17 +71,17 @@ TEST_F(SyncPrefsTest, PreferredTypesKeepEverythingSynced) { |
EXPECT_TRUE(sync_prefs.HasKeepEverythingSynced()); |
- const syncer::ModelTypeSet non_passive_types = GetNonPassiveTypes(); |
- EXPECT_TRUE(non_passive_types.Equals( |
- sync_prefs.GetPreferredDataTypes(non_passive_types))); |
+ const syncer::ModelTypeSet user_types = syncer::UserTypes(); |
+ EXPECT_TRUE(user_types.Equals( |
+ sync_prefs.GetPreferredDataTypes(user_types))); |
const syncer::ModelTypeSet user_visible_types = GetUserVisibleTypes(); |
for (syncer::ModelTypeSet::Iterator it = user_visible_types.First(); |
it.Good(); it.Inc()) { |
syncer::ModelTypeSet preferred_types; |
preferred_types.Put(it.Get()); |
- sync_prefs.SetPreferredDataTypes(non_passive_types, preferred_types); |
- EXPECT_TRUE(non_passive_types.Equals( |
- sync_prefs.GetPreferredDataTypes(non_passive_types))); |
+ sync_prefs.SetPreferredDataTypes(user_types, preferred_types); |
+ EXPECT_TRUE(user_types.Equals( |
+ sync_prefs.GetPreferredDataTypes(user_types))); |
} |
} |
@@ -97,9 +90,9 @@ TEST_F(SyncPrefsTest, PreferredTypesNotKeepEverythingSynced) { |
sync_prefs.SetKeepEverythingSynced(false); |
- const syncer::ModelTypeSet non_passive_types = GetNonPassiveTypes(); |
- EXPECT_TRUE(non_passive_types.Equals( |
- sync_prefs.GetPreferredDataTypes(non_passive_types))); |
+ const syncer::ModelTypeSet user_types = syncer::UserTypes(); |
+ EXPECT_TRUE(user_types.Equals( |
+ sync_prefs.GetPreferredDataTypes(user_types))); |
const syncer::ModelTypeSet user_visible_types = GetUserVisibleTypes(); |
for (syncer::ModelTypeSet::Iterator it = user_visible_types.First(); |
it.Good(); it.Inc()) { |
@@ -119,9 +112,9 @@ TEST_F(SyncPrefsTest, PreferredTypesNotKeepEverythingSynced) { |
if (it.Get() == syncer::EXTENSIONS) { |
expected_preferred_types.Put(syncer::EXTENSION_SETTINGS); |
} |
- sync_prefs.SetPreferredDataTypes(non_passive_types, preferred_types); |
+ sync_prefs.SetPreferredDataTypes(user_types, preferred_types); |
EXPECT_TRUE(expected_preferred_types.Equals( |
- sync_prefs.GetPreferredDataTypes(non_passive_types))); |
+ sync_prefs.GetPreferredDataTypes(user_types))); |
} |
} |
@@ -194,8 +187,8 @@ TEST_F(SyncPrefsTest, NullPrefService) { |
EXPECT_FALSE(sync_prefs.IsStartSuppressed()); |
EXPECT_EQ(base::Time(), sync_prefs.GetLastSyncedTime()); |
EXPECT_FALSE(sync_prefs.HasKeepEverythingSynced()); |
- const syncer::ModelTypeSet non_passive_types = GetNonPassiveTypes(); |
- EXPECT_TRUE(sync_prefs.GetPreferredDataTypes(non_passive_types).Empty()); |
+ 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()); |
} |