sync: Remove ModelNeutralExecuteImpl()

chrome/browser/sync/engine/model_changing_syncer_command.h

Index: chrome/browser/sync/engine/model_changing_syncer_command.h
diff --git a/chrome/browser/sync/engine/model_changing_syncer_command.h b/chrome/browser/sync/engine/model_changing_syncer_command.h
index 36172e7d041d2f2afa576818739cbe7199e70bc0..b44e174208b1020e196b4d93b74c5ff59e4ffbad 100644
--- a/chrome/browser/sync/engine/model_changing_syncer_command.h
+++ b/chrome/browser/sync/engine/model_changing_syncer_command.h
@@ -40,14 +40,6 @@ class ModelChangingSyncerCommand : public SyncerCommand {
     return UnrecoverableErrorInfo();
   }
 
-  // Sometimes, a command has work to do that needs to touch global state
-  // belonging to multiple ModelSafeGroups, but in a way that is known to be
-  // safe.  This will be called once, prior to ModelChangingExecuteImpl,
-  // *without* a ModelSafeGroup restriction in place on the SyncSession.
-  // Returns true on success, false on failure.
-  // TODO(tim): Remove this (bug 36594).
-  virtual bool ModelNeutralExecuteImpl(sessions::SyncSession* session);
-
   // Abstract method to be implemented by subclasses to handle logic that
   // operates on the model.  This is invoked with a SyncSession ModelSafeGroup
   // restriction in place so that bits of state belonging to data types