Avoid useless SYNC_CYCLE_CONTINUATION sync cycle

chrome/browser/sync/engine/resolve_conflicts_command.cc

Index: chrome/browser/sync/engine/resolve_conflicts_command.cc
diff --git a/chrome/browser/sync/engine/resolve_conflicts_command.cc b/chrome/browser/sync/engine/resolve_conflicts_command.cc
index e76b1c7f19259312697b15e331074dbf5521076f..253312e566608ff73b449d863d310b3069a6d429 100644
--- a/chrome/browser/sync/engine/resolve_conflicts_command.cc
+++ b/chrome/browser/sync/engine/resolve_conflicts_command.cc
@@ -19,26 +19,26 @@ std::set<ModelSafeGroup> ResolveConflictsCommand::GetGroupsToChange(
   return session.GetEnabledGroupsWithConflicts();
 }
 
-void ResolveConflictsCommand::ModelChangingExecuteImpl(
+SyncerError ResolveConflictsCommand::ModelChangingExecuteImpl(
     sessions::SyncSession* session) {
   ConflictResolver* resolver = session->context()->resolver();
-  DCHECK(resolver);
-  if (!resolver)
-    return;
+  CHECK(resolver);
 
   syncable::ScopedDirLookup dir(session->context()->directory_manager(),
                                 session->context()->account_name());
   if (!dir.good())
-    return;
+    return DIRECTORY_LOOKUP_FAILED;
   sessions::StatusController* status = session->mutable_status_controller();
   const sessions::ConflictProgress* progress = status->conflict_progress();
   if (!progress)
-    return;  // Nothing to do.
+    return NO_ERROR;  // Nothing to do.
   syncable::WriteTransaction trans(FROM_HERE, syncable::SYNCER, dir);
   const Cryptographer* cryptographer =
       session->context()->directory_manager()->GetCryptographer(&trans);
   status->update_conflicts_resolved(
       resolver->ResolveConflicts(&trans, cryptographer, *progress, status));
+
+  return NO_ERROR;
 }
 
 }  // namespace browser_sync