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

Unified Diff: components/sync/driver/sync_prefs_unittest.cc

Issue 2388673002: Revert of [Sync] Move //components/sync to the syncer namespace. (patchset #5 id:40001 of https://co (Closed)
Patch Set: Created 4 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
« no previous file with comments | « components/sync/driver/sync_prefs.cc ('k') | components/sync/driver/sync_service.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/sync/driver/sync_prefs_unittest.cc
diff --git a/components/sync/driver/sync_prefs_unittest.cc b/components/sync/driver/sync_prefs_unittest.cc
index 4ccde341645b70bba1cd655adec095714b298e4a..3458e39213c6088a17857ddb111c16db5d47e030 100644
--- a/components/sync/driver/sync_prefs_unittest.cc
+++ b/components/sync/driver/sync_prefs_unittest.cc
@@ -20,7 +20,7 @@
#include "testing/gmock/include/gmock/gmock.h"
#include "testing/gtest/include/gtest/gtest.h"
-namespace syncer {
+namespace sync_driver {
namespace {
@@ -73,18 +73,19 @@ TEST_F(SyncPrefsTest, DefaultTypes) {
sync_prefs.SetKeepEverythingSynced(false);
// Only bookmarks and device info are enabled by default.
- ModelTypeSet expected(BOOKMARKS, DEVICE_INFO);
- ModelTypeSet preferred_types = sync_prefs.GetPreferredDataTypes(UserTypes());
+ syncer::ModelTypeSet expected(syncer::BOOKMARKS, syncer::DEVICE_INFO);
+ syncer::ModelTypeSet preferred_types =
+ sync_prefs.GetPreferredDataTypes(syncer::UserTypes());
EXPECT_EQ(expected, preferred_types);
// Simulate an upgrade to delete directives + proxy tabs support. None of the
// new types or their pref group types should be registering, ensuring they
// don't have pref values.
- ModelTypeSet registered_types = UserTypes();
- registered_types.Remove(PROXY_TABS);
- registered_types.Remove(TYPED_URLS);
- registered_types.Remove(SESSIONS);
- registered_types.Remove(HISTORY_DELETE_DIRECTIVES);
+ syncer::ModelTypeSet registered_types = syncer::UserTypes();
+ registered_types.Remove(syncer::PROXY_TABS);
+ registered_types.Remove(syncer::TYPED_URLS);
+ registered_types.Remove(syncer::SESSIONS);
+ registered_types.Remove(syncer::HISTORY_DELETE_DIRECTIVES);
// Enable all other types.
sync_prefs.SetPreferredDataTypes(registered_types, registered_types);
@@ -94,18 +95,18 @@ TEST_F(SyncPrefsTest, DefaultTypes) {
// Proxy tabs should not be enabled (since sessions wasn't), but history
// delete directives should (since typed urls was).
- preferred_types = sync_prefs.GetPreferredDataTypes(UserTypes());
- EXPECT_FALSE(preferred_types.Has(PROXY_TABS));
- EXPECT_TRUE(preferred_types.Has(HISTORY_DELETE_DIRECTIVES));
+ preferred_types = sync_prefs.GetPreferredDataTypes(syncer::UserTypes());
+ EXPECT_FALSE(preferred_types.Has(syncer::PROXY_TABS));
+ EXPECT_TRUE(preferred_types.Has(syncer::HISTORY_DELETE_DIRECTIVES));
// Now manually enable sessions, which should result in proxy tabs also being
// enabled. Also, manually disable typed urls, which should mean that history
// delete directives are not enabled.
pref_service_.SetBoolean(prefs::kSyncTypedUrls, false);
pref_service_.SetBoolean(prefs::kSyncSessions, true);
- preferred_types = sync_prefs.GetPreferredDataTypes(UserTypes());
- EXPECT_TRUE(preferred_types.Has(PROXY_TABS));
- EXPECT_FALSE(preferred_types.Has(HISTORY_DELETE_DIRECTIVES));
+ preferred_types = sync_prefs.GetPreferredDataTypes(syncer::UserTypes());
+ EXPECT_TRUE(preferred_types.Has(syncer::PROXY_TABS));
+ EXPECT_FALSE(preferred_types.Has(syncer::HISTORY_DELETE_DIRECTIVES));
}
TEST_F(SyncPrefsTest, PreferredTypesKeepEverythingSynced) {
@@ -113,12 +114,12 @@ TEST_F(SyncPrefsTest, PreferredTypesKeepEverythingSynced) {
EXPECT_TRUE(sync_prefs.HasKeepEverythingSynced());
- const ModelTypeSet user_types = UserTypes();
+ const syncer::ModelTypeSet user_types = syncer::UserTypes();
EXPECT_EQ(user_types, sync_prefs.GetPreferredDataTypes(user_types));
- const ModelTypeSet user_visible_types = UserSelectableTypes();
- for (ModelTypeSet::Iterator it = user_visible_types.First(); it.Good();
- it.Inc()) {
- ModelTypeSet preferred_types;
+ const syncer::ModelTypeSet user_visible_types = syncer::UserSelectableTypes();
+ 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(user_types, preferred_types);
EXPECT_EQ(user_types, sync_prefs.GetPreferredDataTypes(user_types));
@@ -130,48 +131,48 @@ TEST_F(SyncPrefsTest, PreferredTypesNotKeepEverythingSynced) {
sync_prefs.SetKeepEverythingSynced(false);
- const ModelTypeSet user_types = UserTypes();
+ const syncer::ModelTypeSet user_types = syncer::UserTypes();
EXPECT_NE(user_types, sync_prefs.GetPreferredDataTypes(user_types));
- const ModelTypeSet user_visible_types = UserSelectableTypes();
- for (ModelTypeSet::Iterator it = user_visible_types.First(); it.Good();
- it.Inc()) {
- ModelTypeSet preferred_types;
+ const syncer::ModelTypeSet user_visible_types = syncer::UserSelectableTypes();
+ for (syncer::ModelTypeSet::Iterator it = user_visible_types.First();
+ it.Good(); it.Inc()) {
+ syncer::ModelTypeSet preferred_types;
preferred_types.Put(it.Get());
- ModelTypeSet expected_preferred_types(preferred_types);
- if (it.Get() == AUTOFILL) {
- expected_preferred_types.Put(AUTOFILL_PROFILE);
- expected_preferred_types.Put(AUTOFILL_WALLET_DATA);
- expected_preferred_types.Put(AUTOFILL_WALLET_METADATA);
+ syncer::ModelTypeSet expected_preferred_types(preferred_types);
+ if (it.Get() == syncer::AUTOFILL) {
+ expected_preferred_types.Put(syncer::AUTOFILL_PROFILE);
+ expected_preferred_types.Put(syncer::AUTOFILL_WALLET_DATA);
+ expected_preferred_types.Put(syncer::AUTOFILL_WALLET_METADATA);
}
- if (it.Get() == PREFERENCES) {
- expected_preferred_types.Put(DICTIONARY);
- expected_preferred_types.Put(PRIORITY_PREFERENCES);
- expected_preferred_types.Put(SEARCH_ENGINES);
+ if (it.Get() == syncer::PREFERENCES) {
+ expected_preferred_types.Put(syncer::DICTIONARY);
+ expected_preferred_types.Put(syncer::PRIORITY_PREFERENCES);
+ expected_preferred_types.Put(syncer::SEARCH_ENGINES);
}
- if (it.Get() == APPS) {
- expected_preferred_types.Put(APP_LIST);
- expected_preferred_types.Put(APP_NOTIFICATIONS);
- expected_preferred_types.Put(APP_SETTINGS);
- expected_preferred_types.Put(ARC_PACKAGE);
- expected_preferred_types.Put(READING_LIST);
+ if (it.Get() == syncer::APPS) {
+ expected_preferred_types.Put(syncer::APP_LIST);
+ expected_preferred_types.Put(syncer::APP_NOTIFICATIONS);
+ expected_preferred_types.Put(syncer::APP_SETTINGS);
+ expected_preferred_types.Put(syncer::ARC_PACKAGE);
+ expected_preferred_types.Put(syncer::READING_LIST);
}
- if (it.Get() == EXTENSIONS) {
- expected_preferred_types.Put(EXTENSION_SETTINGS);
+ if (it.Get() == syncer::EXTENSIONS) {
+ expected_preferred_types.Put(syncer::EXTENSION_SETTINGS);
}
- if (it.Get() == TYPED_URLS) {
- expected_preferred_types.Put(HISTORY_DELETE_DIRECTIVES);
- expected_preferred_types.Put(SESSIONS);
- expected_preferred_types.Put(FAVICON_IMAGES);
- expected_preferred_types.Put(FAVICON_TRACKING);
+ if (it.Get() == syncer::TYPED_URLS) {
+ expected_preferred_types.Put(syncer::HISTORY_DELETE_DIRECTIVES);
+ expected_preferred_types.Put(syncer::SESSIONS);
+ expected_preferred_types.Put(syncer::FAVICON_IMAGES);
+ expected_preferred_types.Put(syncer::FAVICON_TRACKING);
}
- if (it.Get() == PROXY_TABS) {
- expected_preferred_types.Put(SESSIONS);
- expected_preferred_types.Put(FAVICON_IMAGES);
- expected_preferred_types.Put(FAVICON_TRACKING);
+ if (it.Get() == syncer::PROXY_TABS) {
+ expected_preferred_types.Put(syncer::SESSIONS);
+ expected_preferred_types.Put(syncer::FAVICON_IMAGES);
+ expected_preferred_types.Put(syncer::FAVICON_TRACKING);
}
// Device info is always preferred.
- expected_preferred_types.Put(DEVICE_INFO);
+ expected_preferred_types.Put(syncer::DEVICE_INFO);
sync_prefs.SetPreferredDataTypes(user_types, preferred_types);
EXPECT_EQ(expected_preferred_types,
@@ -229,24 +230,27 @@ TEST_F(SyncPrefsTest, ClearPreferences) {
// Device info should always be enabled.
TEST_F(SyncPrefsTest, DeviceInfo) {
SyncPrefs sync_prefs(&pref_service_);
- EXPECT_TRUE(sync_prefs.GetPreferredDataTypes(UserTypes()).Has(DEVICE_INFO));
+ EXPECT_TRUE(sync_prefs.GetPreferredDataTypes(syncer::UserTypes())
+ .Has(syncer::DEVICE_INFO));
sync_prefs.SetKeepEverythingSynced(true);
- EXPECT_TRUE(sync_prefs.GetPreferredDataTypes(UserTypes()).Has(DEVICE_INFO));
+ EXPECT_TRUE(sync_prefs.GetPreferredDataTypes(syncer::UserTypes())
+ .Has(syncer::DEVICE_INFO));
sync_prefs.SetKeepEverythingSynced(false);
- EXPECT_TRUE(sync_prefs.GetPreferredDataTypes(UserTypes()).Has(DEVICE_INFO));
+ EXPECT_TRUE(sync_prefs.GetPreferredDataTypes(syncer::UserTypes())
+ .Has(syncer::DEVICE_INFO));
}
// Verify that invalidation versions are persisted and loaded correctly.
TEST_F(SyncPrefsTest, InvalidationVersions) {
- std::map<ModelType, int64_t> versions;
- versions[BOOKMARKS] = 10;
- versions[SESSIONS] = 20;
- versions[PREFERENCES] = 30;
+ std::map<syncer::ModelType, int64_t> versions;
+ versions[syncer::BOOKMARKS] = 10;
+ versions[syncer::SESSIONS] = 20;
+ versions[syncer::PREFERENCES] = 30;
SyncPrefs sync_prefs(&pref_service_);
sync_prefs.UpdateInvalidationVersions(versions);
- std::map<ModelType, int64_t> versions2;
+ std::map<syncer::ModelType, int64_t> versions2;
sync_prefs.GetInvalidationVersions(&versions2);
EXPECT_EQ(versions.size(), versions2.size());
@@ -257,4 +261,4 @@ TEST_F(SyncPrefsTest, InvalidationVersions) {
} // namespace
-} // namespace syncer
+} // namespace sync_driver
« no previous file with comments | « components/sync/driver/sync_prefs.cc ('k') | components/sync/driver/sync_service.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698