Revert 94128 - [Sync] Refactor sync datatype error handling.

chrome/browser/sync/glue/model_associator.h

Index: chrome/browser/sync/glue/model_associator.h
===================================================================
--- chrome/browser/sync/glue/model_associator.h	(revision 94142)
+++ chrome/browser/sync/glue/model_associator.h	(working copy)
@@ -9,8 +9,6 @@
 #include "base/basictypes.h"
 #include "chrome/browser/sync/syncable/model_type.h"
 
-class SyncError;
-
 namespace sync_api {
 class BaseNode;
 }
@@ -29,12 +27,10 @@
   // should be identical and corresponding. Returns true on
   // success. On failure of this step, we should abort the sync
   // operation and report an error to the user.
-  // TODO(zea): return a SyncError instead of passing one in.
-  virtual bool AssociateModels(SyncError* error) = 0;
+  virtual bool AssociateModels() = 0;
 
   // Clears all the associations between the chrome and sync models.
-  // TODO(zea): return a SyncError instead of passing one in.
-  virtual bool DisassociateModels(SyncError* error) = 0;
+  virtual bool DisassociateModels() = 0;
 
   // The has_nodes out parameter is set to true if the sync model has
   // nodes other than the permanent tagged nodes.  The method may