Revert 132455 - Abort sync cycles when download step fails

sync/engine/sync_scheduler.h

Index: sync/engine/sync_scheduler.h
===================================================================
--- sync/engine/sync_scheduler.h	(revision 132475)
+++ sync/engine/sync_scheduler.h	(working copy)
@@ -179,7 +179,6 @@
   };
   friend class SyncSchedulerTest;
   friend class SyncSchedulerWhiteboxTest;
-  friend class SyncerTest;
 
   FRIEND_TEST_ALL_PREFIXES(SyncSchedulerWhiteboxTest,
       DropNudgeWhileExponentialBackOff);
@@ -248,12 +247,6 @@
 
   static const char* GetDecisionString(JobProcessDecision decision);
 
-  // Assign |start| and |end| to appropriate SyncerStep values for the
-  // specified |purpose|.
-  static void SetSyncerStepsForPurpose(
-      SyncSessionJob::SyncSessionJobPurpose purpose,
-      SyncerStep* start, SyncerStep* end);
-
   // Helpers that log before posting to |sync_loop_|.  These will only post
   // the task in between calls to Start/Stop.
   void PostTask(const tracked_objects::Location& from_here,
@@ -347,6 +340,12 @@
   // Creates a session for a poll and performs the sync.
   void PollTimerCallback();
 
+  // Assign |start| and |end| to appropriate SyncerStep values for the
+  // specified |purpose|.
+  void SetSyncerStepsForPurpose(SyncSessionJob::SyncSessionJobPurpose purpose,
+                                SyncerStep* start,
+                                SyncerStep* end);
+
   // Used to update |connection_code_|, see below.
   void UpdateServerConnectionManagerStatus(
       HttpResponse::ServerConnectionCode code);