Avoid useless SYNC_CYCLE_CONTINUATION sync cycle

chrome/browser/sync/engine/syncer_command.h

-// Copyright (c) 2009 The Chromium Authors. All rights reserved.
+// Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_SYNC_ENGINE_SYNCER_COMMAND_H_
 #define CHROME_BROWSER_SYNC_ENGINE_SYNCER_COMMAND_H_
 #pragma once
 
 #include "base/basictypes.h"
 
+#include "chrome/browser/sync/engine/syncer_error.h"
+
 namespace browser_sync {
 
 namespace sessions {
 class SyncSession;
 }
 
 // Implementation of a simple command pattern intended to be driven by the
 // Syncer. SyncerCommand is abstract and all subclasses must implement
 // ExecuteImpl(). This is done so that chunks of syncer operation can be unit
 // tested.
 //
 // Example Usage:
 //
 //   SyncSession session = ...;
 //   SyncerCommand *cmd = SomeCommandFactory.createCommand(...);
 //   cmd->Execute(session);
 //   delete cmd;
 
 class SyncerCommand {
  public:
   SyncerCommand();
   virtual ~SyncerCommand();
 
   // Execute dispatches to a derived class's ExecuteImpl.
-  void Execute(sessions::SyncSession* session);
+  SyncerError Execute(sessions::SyncSession* session);
 
   // ExecuteImpl is where derived classes actually do work.
-  virtual void ExecuteImpl(sessions::SyncSession* session) = 0;
+  virtual SyncerError ExecuteImpl(sessions::SyncSession* session) = 0;
  private:
   void SendNotifications(sessions::SyncSession* session);
   DISALLOW_COPY_AND_ASSIGN(SyncerCommand);
 };
 
 }  // namespace browser_sync
 
 #endif  // CHROME_BROWSER_SYNC_ENGINE_SYNCER_COMMAND_H_