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

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

Issue 10985008: sync: Add DeviceInfo protobuf and supporting code (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Another missing include Created 8 years, 2 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/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());
}
« no previous file with comments | « chrome/browser/sync/profile_sync_service_session_unittest.cc ('k') | chrome/browser/sync/test_profile_sync_service.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698