Fix the false-positive detection of commit errors

chrome/browser/sync/sessions/session_state.h

Index: chrome/browser/sync/sessions/session_state.h
diff --git a/chrome/browser/sync/sessions/session_state.h b/chrome/browser/sync/sessions/session_state.h
index c6ee65ba598ec4838c4b0097189368e3613b079a..2e5c7b350fd8b4378b36893969a6b9a68985de5b 100644
--- a/chrome/browser/sync/sessions/session_state.h
+++ b/chrome/browser/sync/sessions/session_state.h
@@ -28,6 +28,8 @@
 #include "chrome/browser/sync/syncable/model_type_payload_map.h"
 #include "chrome/browser/sync/syncable/syncable.h"
 
+#include "chrome/browser/sync/engine/syncer_proto_util.h"
+
 namespace base {
 class DictionaryValue;
 }
@@ -109,7 +111,7 @@ struct ErrorCounters {
   int consecutive_errors;
 
   // Any protocol errors that we received during this sync session.
-  SyncProtocolError sync_protocol_error;
+  SyncOperationResult sync_protocol_error; // FIXME: umm... really?  Is this used?
 };
 
 // Caller takes ownership of the returned dictionary.
@@ -322,6 +324,12 @@ struct AllModelTypeState {
   // requested_update_types stores the set of types which were requested.
   syncable::ModelTypeBitSet updates_request_types;
   ClientToServerResponse updates_response;
+
+  // FIXME: move these to error counters?
+  SyncOperationResult last_download_updates_result;
+  SyncOperationResult last_post_commit_result;
+  SyncOperationResult last_clear_data_result;
+
   // Used to build the shared commit message.
   DirtyOnWrite<std::vector<int64> > unsynced_handles;
   DirtyOnWrite<SyncerStatus> syncer_status;