[Sync] Refactor sync datatype error handling.

chrome/browser/sync/api/sync_change_processor.h

Index: chrome/browser/sync/api/sync_change_processor.h
diff --git a/chrome/browser/sync/api/sync_change_processor.h b/chrome/browser/sync/api/sync_change_processor.h
index 80be610ab5b4902bf0e1f6021108af4f1f38f2e2..ec3e8d3ac36386adc0f0f73e9322a5d1c034a05c 100644
--- a/chrome/browser/sync/api/sync_change_processor.h
+++ b/chrome/browser/sync/api/sync_change_processor.h
@@ -9,6 +9,7 @@
 #include <vector>
 
 class SyncChange;
+class SyncError;
 
 namespace tracked_objects {
 class Location;
@@ -20,8 +21,14 @@ typedef std::vector<SyncChange> SyncChangeList;
 class SyncChangeProcessor {
  public:
   // Process a list of SyncChanges.
-  virtual void ProcessSyncChanges(const tracked_objects::Location& from_here,
-                                  const SyncChangeList& change_list) = 0;
+  // Returns: False if an error occured, true otherwise.

[akalin, 2011/07/21 23:03:53]

False -> false

[Nicolas Zea, 2011/07/21 23:43:27]

Done.

+  // Inputs:
+  //   |from_here|: allows tracking of where sync changes originate.
+  //   |change_list|: is the list of sync changes in need of processing.
+  //   |error|: if an error occurred, the error tracking information.
+  virtual bool ProcessSyncChanges(const tracked_objects::Location& from_here,
+                                  const SyncChangeList& change_list,
+                                  SyncError* error) = 0;
  protected:
   virtual ~SyncChangeProcessor();
 };