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

Unified Diff: chrome/browser/sync/test/integration/sync_test.cc

Issue 8772074: [Sync] Convert syncable/ directory to ModelEnumSet (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix typo 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
Index: chrome/browser/sync/test/integration/sync_test.cc
diff --git a/chrome/browser/sync/test/integration/sync_test.cc b/chrome/browser/sync/test/integration/sync_test.cc
index bb009b8b09ee77ca4ec9788c23f077d4b521b541..9c83bfd8a17a49063ab43f8294d8661774d08157 100644
--- a/chrome/browser/sync/test/integration/sync_test.cc
+++ b/chrome/browser/sync/test/integration/sync_test.cc
@@ -567,7 +567,7 @@ void SyncTest::EnableNotifications() {
}
void SyncTest::TriggerNotification(
- const syncable::ModelTypeSet& changed_types) {
+ syncable::ModelEnumSet changed_types) {
ASSERT_TRUE(ServerSupportsNotificationControl());
const std::string& data =
sync_notifier::P2PNotificationData("from_server",
@@ -589,14 +589,16 @@ bool SyncTest::ServerSupportsErrorTriggering() const {
}
void SyncTest::TriggerMigrationDoneError(
- const syncable::ModelTypeSet& model_types) {
+ syncable::ModelEnumSet model_types) {
ASSERT_TRUE(ServerSupportsErrorTriggering());
std::string path = "chromiumsync/migrate";
char joiner = '?';
- for (syncable::ModelTypeSet::const_iterator it = model_types.begin();
- it != model_types.end(); ++it) {
- path.append(base::StringPrintf("%ctype=%d", joiner,
- syncable::GetExtensionFieldNumberFromModelType(*it)));
+ for (syncable::ModelEnumSet::Iterator it = model_types.First();
+ it.Good(); it.Inc()) {
+ path.append(
+ base::StringPrintf(
+ "%ctype=%d", joiner,
+ syncable::GetExtensionFieldNumberFromModelType(it.Get())));
joiner = '&';
}
ui_test_utils::NavigateToURL(browser(), sync_server_.GetURL(path));
« no previous file with comments | « chrome/browser/sync/test/integration/sync_test.h ('k') | chrome/browser/sync/test/null_directory_change_delegate.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698