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

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

Issue 8510079: sync: Remove ModelNeutralExecuteImpl() (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix bug that caused integration test failures 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/model_changing_syncer_command.cc
diff --git a/chrome/browser/sync/engine/model_changing_syncer_command.cc b/chrome/browser/sync/engine/model_changing_syncer_command.cc
index 407c299c8d262da807a2f0bb3aed67384d988598..8a0b8f02fcdfc5fcaa4f3913e078904c138515be 100644
--- a/chrome/browser/sync/engine/model_changing_syncer_command.cc
+++ b/chrome/browser/sync/engine/model_changing_syncer_command.cc
@@ -15,9 +15,6 @@ namespace browser_sync {
void ModelChangingSyncerCommand::ExecuteImpl(sessions::SyncSession* session) {
work_session_ = session;
- if (!ModelNeutralExecuteImpl(work_session_)) {
- return;
- }
// Project the list of active types (i.e., types in the routing
// info) to a list of groups.
@@ -59,9 +56,4 @@ void ModelChangingSyncerCommand::ExecuteImpl(sessions::SyncSession* session) {
}
}
-bool ModelChangingSyncerCommand::ModelNeutralExecuteImpl(
- sessions::SyncSession* session) {
- return true;
-}
-
} // namespace browser_sync

Powered by Google App Engine
This is Rietveld 408576698