Abort sync cycles when download step fails

sync/sessions/status_controller.cc

Index: sync/sessions/status_controller.cc
diff --git a/sync/sessions/status_controller.cc b/sync/sessions/status_controller.cc
index 19fdbaff76f18d4b714e8728ca4e4d716086a92c..816f635f4cba74b39a80c0cdbe5d575d4121a061 100644
--- a/sync/sessions/status_controller.cc
+++ b/sync/sessions/status_controller.cc
@@ -291,19 +291,6 @@ int StatusController::TotalNumConflictingItems() const {
   return sum;
 }
 
-bool StatusController::ServerSaysNothingMoreToDownload() const {
-  if (!download_updates_succeeded())
-    return false;
-
-  if (!updates_response().get_updates().has_changes_remaining()) {
-    NOTREACHED();  // Server should always send changes remaining.
-    return false;  // Avoid looping forever.
-  }
-  // Changes remaining is an estimate, but if it's estimated to be
-  // zero, that's firm and we don't have to ask again.
-  return updates_response().get_updates().changes_remaining() == 0;
-}
-
 void StatusController::set_debug_info_sent() {
   shared_.control_params.debug_info_sent = true;
 }