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

chrome/browser/sync/glue/extension_model_associator.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/sync/glue/extension_model_associator.h"
 
 #include "base/logging.h"
 #include "base/tracked.h"
 #include "chrome/browser/extensions/extension_sync_data.h"
 #include "chrome/browser/sync/engine/syncapi.h"
@@ skipping 14 matching lines @@
       user_share_(user_share) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   DCHECK(extension_service_);
   DCHECK(user_share_);
 }
 
 ExtensionModelAssociator::~ExtensionModelAssociator() {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
 }
 
-bool ExtensionModelAssociator::AssociateModels(SyncError* error) {
+bool ExtensionModelAssociator::AssociateModels() {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   ExtensionDataMap extension_data_map;
   if (!SlurpExtensionData(
           traits_, *extension_service_, user_share_, &extension_data_map)) {
-    error->Reset(FROM_HERE, "Failed to get extension data.", model_type());
     return false;
   }
   if (!FlushExtensionData(
           traits_, extension_data_map, extension_service_, user_share_)) {
-    error->Reset(FROM_HERE, "Failed to flush extension data.", model_type());
     return false;
   }
 
   return true;
 }
 
-bool ExtensionModelAssociator::DisassociateModels(SyncError* error) {
+bool ExtensionModelAssociator::DisassociateModels() {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   // Nothing to do.
   return true;
 }
 
 bool ExtensionModelAssociator::SyncModelHasUserCreatedNodes(bool* has_nodes) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   return RootNodeHasChildren(traits_.root_node_tag, user_share_, has_nodes);
 }
 
 void ExtensionModelAssociator::AbortAssociation() {
   // No implementation needed, this associator runs on the main
   // thread.
 }
 
 bool ExtensionModelAssociator::CryptoReadyIfNecessary() {
   // We only access the cryptographer while holding a transaction.
   sync_api::ReadTransaction trans(FROM_HERE, user_share_);
   const syncable::ModelTypeSet& encrypted_types =
       sync_api::GetEncryptedTypes(&trans);
   return encrypted_types.count(traits_.model_type) == 0 ||
       trans.GetCryptographer()->is_ready();
 }
 
 }  // namespace browser_sync