sync: Remove ModelNeutralExecuteImpl()

chrome/browser/sync/engine/syncer.cc

Index: chrome/browser/sync/engine/syncer.cc
diff --git a/chrome/browser/sync/engine/syncer.cc b/chrome/browser/sync/engine/syncer.cc
index bffcee6c9e9a105287b663e51da5c68bb15df4f1..a5bb9023a0543205506ac37e8ff61caca4d1f9c8 100644
--- a/chrome/browser/sync/engine/syncer.cc
+++ b/chrome/browser/sync/engine/syncer.cc
@@ -143,8 +143,10 @@ void Syncer::SyncShare(sessions::SyncSession* session,
       }
       case PROCESS_UPDATES: {
         VLOG(1) << "Processing Updates";
-        ProcessUpdatesCommand process_updates;
-        process_updates.Execute(session);
+        if (session->status_controller()->ResponseContainsUpdates()) {

[tim (not reviewing), 2011/11/17 18:46:38]

I like this but the pattern here is typically to figure out next_step before
transitioning, as you'd draw it in a state diagram.  I can see arguments for
doing it this way (happens regardless of path taken to get here, consistent with
the previous implementation).

I'm curious where you're going though... could you provide any info on how this
helps you with the next piece?

+          ProcessUpdatesCommand process_updates;
+          process_updates.Execute(session);
+        }
         next_step = STORE_TIMESTAMPS;
         break;
       }