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

Unified Diff: chrome/browser/sync/engine/sync_scheduler_unittest.cc

Issue 8625005: [Sync] Make ModelSafeWorker a true interface (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Sync to head Created 9 years, 1 month 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/engine/sync_scheduler_unittest.cc
diff --git a/chrome/browser/sync/engine/sync_scheduler_unittest.cc b/chrome/browser/sync/engine/sync_scheduler_unittest.cc
index df06248ecfda25ad10dda8204c0cc53f6c666425..b83322dacb65da921ed7339a88347b682c8010c3 100644
--- a/chrome/browser/sync/engine/sync_scheduler_unittest.cc
+++ b/chrome/browser/sync/engine/sync_scheduler_unittest.cc
@@ -8,10 +8,10 @@
#include "base/memory/weak_ptr.h"
#include "base/message_loop.h"
#include "base/test/test_timeouts.h"
-#include "chrome/browser/sync/engine/mock_model_safe_workers.h"
#include "chrome/browser/sync/engine/sync_scheduler.h"
#include "chrome/browser/sync/engine/syncer.h"
#include "chrome/browser/sync/sessions/test_util.h"
+#include "chrome/browser/sync/test/engine/fake_model_safe_worker_registrar.h"
#include "chrome/browser/sync/test/engine/mock_connection_manager.h"
#include "chrome/browser/sync/test/engine/test_directory_setter_upper.h"
#include "testing/gtest/include/gtest/gtest.h"
@@ -83,16 +83,15 @@ class SyncSchedulerTest : public testing::Test {
};
virtual void SetUp() {
- syncable::ModelTypeBitSet model_types;
- model_types[syncable::BOOKMARKS] = true;
- model_types[syncable::AUTOFILL] = true;
- model_types[syncable::THEMES] = true;
-
syncdb_.SetUp();
syncer_ = new MockSyncer();
delay_ = NULL;
- registrar_.reset(MockModelSafeWorkerRegistrar::PassiveForTypes(
- model_types));
+ ModelSafeRoutingInfo routing_info;
+ routing_info[syncable::BOOKMARKS] = GROUP_UI;
+ routing_info[syncable::AUTOFILL] = GROUP_DB;
+ routing_info[syncable::THEMES] = GROUP_UI;
+ routing_info[syncable::NIGORI] = GROUP_PASSIVE;
+ registrar_.reset(new FakeModelSafeWorkerRegistrar(routing_info));
connection_.reset(new MockConnectionManager(syncdb_.manager(), "Test"));
connection_->SetServerReachable();
context_ = new SyncSessionContext(connection_.get(), syncdb_.manager(),
@@ -205,7 +204,7 @@ class SyncSchedulerTest : public testing::Test {
SyncSessionContext* context_;
MockSyncer* syncer_;
MockDelayProvider* delay_;
- scoped_ptr<MockModelSafeWorkerRegistrar> registrar_;
+ scoped_ptr<FakeModelSafeWorkerRegistrar> registrar_;
MockDirectorySetterUpper syncdb_;
};

Powered by Google App Engine
This is Rietveld 408576698