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

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

Issue 8851006: [Sync] Replace all instances of ModelTypeSet with ModelEnumSet (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Cleanup pass #2 Created 9 years 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_setup_flow.cc ('k') | chrome/browser/sync/sync_ui_util.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/sync/sync_setup_wizard_unittest.cc
diff --git a/chrome/browser/sync/sync_setup_wizard_unittest.cc b/chrome/browser/sync/sync_setup_wizard_unittest.cc
index ba48588a225f463f472b318f3bfc763c0c1844d2..879f4653362bcb9a65371a1a21e4a5ee5765022e 100644
--- a/chrome/browser/sync/sync_setup_wizard_unittest.cc
+++ b/chrome/browser/sync/sync_setup_wizard_unittest.cc
@@ -81,7 +81,7 @@ class ProfileSyncServiceForWizardTest : public ProfileSyncService {
}
virtual void OnUserChoseDatatypes(bool sync_everything,
- const syncable::ModelTypeSet& chosen_types) {
+ syncable::ModelEnumSet chosen_types) {
user_chose_data_types_ = true;
chosen_data_types_ = chosen_types;
}
@@ -141,7 +141,7 @@ class ProfileSyncServiceForWizardTest : public ProfileSyncService {
user_cancelled_dialog_ = false;
user_chose_data_types_ = false;
keep_everything_synced_ = false;
- chosen_data_types_.clear();
+ chosen_data_types_.Clear();
}
// Use this to have the service act as if it were running under CrOS.
@@ -169,7 +169,7 @@ class ProfileSyncServiceForWizardTest : public ProfileSyncService {
bool keep_everything_synced_;
bool is_using_secondary_passphrase_;
bool encrypt_everything_;
- syncable::ModelTypeSet chosen_data_types_;
+ syncable::ModelEnumSet chosen_data_types_;
std::string passphrase_;
private:
@@ -342,15 +342,15 @@ TEST_F(SyncSetupWizardTest, ChooseDataTypesSetsPrefs) {
// DONE state and closed the UI.
EXPECT_FALSE(wizard_->IsVisible());
EXPECT_FALSE(service_->keep_everything_synced_);
- EXPECT_EQ(1U, service_->chosen_data_types_.count(syncable::BOOKMARKS));
- EXPECT_EQ(1U, service_->chosen_data_types_.count(syncable::PREFERENCES));
- EXPECT_EQ(0U, service_->chosen_data_types_.count(syncable::THEMES));
- EXPECT_EQ(0U, service_->chosen_data_types_.count(syncable::PASSWORDS));
- EXPECT_EQ(0U, service_->chosen_data_types_.count(syncable::AUTOFILL));
- EXPECT_EQ(0U, service_->chosen_data_types_.count(syncable::EXTENSIONS));
- EXPECT_EQ(1U, service_->chosen_data_types_.count(syncable::TYPED_URLS));
- EXPECT_EQ(1U, service_->chosen_data_types_.count(syncable::APPS));
- EXPECT_EQ(0U, service_->chosen_data_types_.count(
+ EXPECT_TRUE(service_->chosen_data_types_.Has(syncable::BOOKMARKS));
+ EXPECT_TRUE(service_->chosen_data_types_.Has(syncable::PREFERENCES));
+ EXPECT_FALSE(service_->chosen_data_types_.Has(syncable::THEMES));
+ EXPECT_FALSE(service_->chosen_data_types_.Has(syncable::PASSWORDS));
+ EXPECT_FALSE(service_->chosen_data_types_.Has(syncable::AUTOFILL));
+ EXPECT_FALSE(service_->chosen_data_types_.Has(syncable::EXTENSIONS));
+ EXPECT_TRUE(service_->chosen_data_types_.Has(syncable::TYPED_URLS));
+ EXPECT_TRUE(service_->chosen_data_types_.Has(syncable::APPS));
+ EXPECT_FALSE(service_->chosen_data_types_.Has(
syncable::APP_NOTIFICATIONS));
}
« no previous file with comments | « chrome/browser/sync/sync_setup_flow.cc ('k') | chrome/browser/sync/sync_ui_util.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698