Fix the false-positive detection of commit errors

chrome/browser/sync/sessions/test_util.cc

Index: chrome/browser/sync/sessions/test_util.cc
diff --git a/chrome/browser/sync/sessions/test_util.cc b/chrome/browser/sync/sessions/test_util.cc
index 65d307b7387adddf3251381faa0c0a43a029787d..c01f02fd7b1ed5ea34fb0e500d012f6b3d1cc701 100644
--- a/chrome/browser/sync/sessions/test_util.cc
+++ b/chrome/browser/sync/sessions/test_util.cc
@@ -24,13 +24,9 @@ void SimulateDownloadUpdatesFailed(sessions::SyncSession* session,
 
 void SimulateCommitFailed(sessions::SyncSession* session,
                           SyncerStep begin, SyncerStep end) {
-  // Note that a non-zero number of unsynced handles once a session has
-  // completed implies that the Syncer was unable to make forward progress
-  // during a commit, indicating an error occurred.
-  // See implementation of SyncSession::HasMoreToSync.
-  std::vector<int64> handles;
-  handles.push_back(1);
-  session->status_controller()->set_unsynced_handles(handles);
+  // This indicates an error, not necessarily a commit error.  The effect is
+  // the same.
+  session->status_controller()->increment_num_consecutive_errors();
 }
 
 void SimulateSuccess(sessions::SyncSession* session,