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

Unified Diff: sync/test/engine/syncer_command_test.h

Issue 19982002: sync: Remove SyncSourceInfo (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Convert SyncShareRecords to typedef Created 7 years, 5 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 | « sync/sync_tests.gypi ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sync/test/engine/syncer_command_test.h
diff --git a/sync/test/engine/syncer_command_test.h b/sync/test/engine/syncer_command_test.h
index 93106b9933060a3572add4f7734a20cead33cd07..66df3990f540fb7164997dc839e6b554cdcb2b13 100644
--- a/sync/test/engine/syncer_command_test.h
+++ b/sync/test/engine/syncer_command_test.h
@@ -103,24 +103,8 @@ class SyncerCommandTestBase : public testing::Test,
// Lazily create a session requesting all datatypes with no state.
sessions::SyncSession* session() {
- ModelTypeInvalidationMap types =
- ModelSafeRoutingInfoToInvalidationMap(routing_info_, std::string());
- return session(sessions::SyncSourceInfo(types));
- }
-
- // Create a session with the provided source.
- sessions::SyncSession* session(const sessions::SyncSourceInfo& source) {
- // These sources require a valid nudge tracker.
- DCHECK_NE(sync_pb::GetUpdatesCallerInfo::LOCAL, source.updates_source);
- DCHECK_NE(sync_pb::GetUpdatesCallerInfo::NOTIFICATION,
- source.updates_source);
- DCHECK_NE(sync_pb::GetUpdatesCallerInfo::DATATYPE_REFRESH,
- source.updates_source);
- if (!session_.get()) {
- std::vector<ModelSafeWorker*> workers = GetWorkers();
- session_.reset(
- sessions::SyncSession::Build(context(), delegate(), source));
- }
+ if (!session_.get())
+ session_.reset(sessions::SyncSession::Build(context(), delegate()));
return session_.get();
}
« no previous file with comments | « sync/sync_tests.gypi ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698