[Sync] Introduce ModelError for USS error handling.

components/sync/model/fake_model_type_sync_bridge.cc

Index: components/sync/model/fake_model_type_sync_bridge.cc
diff --git a/components/sync/model/fake_model_type_sync_bridge.cc b/components/sync/model/fake_model_type_sync_bridge.cc
index a3c01918523f9fb89b9ffc2544e4ea397aacc089..7279507b8b3abc8a27e5d225b4f11adf7724e27d 100644
--- a/components/sync/model/fake_model_type_sync_bridge.cc
+++ b/components/sync/model/fake_model_type_sync_bridge.cc
@@ -9,6 +9,7 @@
 #include "base/bind.h"
 #include "base/memory/ptr_util.h"
 #include "components/sync/base/hash_util.h"
+#include "components/sync/model/model_error.h"
 #include "components/sync/model/mutable_data_batch.h"
 #include "components/sync/model_impl/in_memory_metadata_change_list.h"
 #include "testing/gtest/include/gtest/gtest.h"
@@ -198,12 +199,12 @@ FakeModelTypeSyncBridge::CreateMetadataChangeList() {
   return base::MakeUnique<TestMetadataChangeList>();
 }
 
-SyncError FakeModelTypeSyncBridge::MergeSyncData(
+ModelError FakeModelTypeSyncBridge::MergeSyncData(
     std::unique_ptr<MetadataChangeList> metadata_changes,
     EntityDataMap data_map) {
   if (bridge_error_.IsSet()) {
-    SyncError error = bridge_error_;
-    bridge_error_ = SyncError();
+    ModelError error = bridge_error_;
+    bridge_error_ = ModelError();
     return error;
   }
   // Commit any local entities that aren't being overwritten by the server.
@@ -218,15 +219,15 @@ SyncError FakeModelTypeSyncBridge::MergeSyncData(
     db_->PutData(kv.first, kv.second.value());
   }
   ApplyMetadataChangeList(std::move(metadata_changes));
-  return SyncError();
+  return ModelError();
 }
 
-SyncError FakeModelTypeSyncBridge::ApplySyncChanges(
+ModelError FakeModelTypeSyncBridge::ApplySyncChanges(
     std::unique_ptr<MetadataChangeList> metadata_changes,
     EntityChangeList entity_changes) {
   if (bridge_error_.IsSet()) {
-    SyncError error = bridge_error_;
-    bridge_error_ = SyncError();
+    ModelError error = bridge_error_;
+    bridge_error_ = ModelError();
     return error;
   }
   for (const EntityChange& change : entity_changes) {
@@ -246,7 +247,7 @@ SyncError FakeModelTypeSyncBridge::ApplySyncChanges(
     }
   }
   ApplyMetadataChangeList(std::move(metadata_changes));
-  return SyncError();
+  return ModelError();
 }
 
 void FakeModelTypeSyncBridge::ApplyMetadataChangeList(
@@ -282,8 +283,9 @@ void FakeModelTypeSyncBridge::ApplyMetadataChangeList(
 void FakeModelTypeSyncBridge::GetData(StorageKeyList keys,
                                       DataCallback callback) {
   if (bridge_error_.IsSet()) {
-    callback.Run(bridge_error_, nullptr);
-    bridge_error_ = SyncError();
+    ModelError error = bridge_error_;
+    bridge_error_ = ModelError();
+    change_processor()->ReportError(error);
     return;
   }
 
@@ -292,13 +294,14 @@ void FakeModelTypeSyncBridge::GetData(StorageKeyList keys,
     DCHECK(db_->HasData(key)) << "No data for " << key;
     batch->Put(key, CopyEntityData(db_->GetData(key)));
   }
-  callback.Run(SyncError(), std::move(batch));
+  callback.Run(std::move(batch));
 }
 
 void FakeModelTypeSyncBridge::GetAllData(DataCallback callback) {
   if (bridge_error_.IsSet()) {
-    callback.Run(bridge_error_, nullptr);
-    bridge_error_ = SyncError();
+    ModelError error = bridge_error_;
+    bridge_error_ = ModelError();
+    change_processor()->ReportError(error);
     return;
   }
 
@@ -306,7 +309,7 @@ void FakeModelTypeSyncBridge::GetAllData(DataCallback callback) {
   for (const auto& kv : db_->all_data()) {
     batch->Put(kv.first, CopyEntityData(*kv.second));
   }
-  callback.Run(SyncError(), std::move(batch));
+  callback.Run(std::move(batch));
 }
 
 std::string FakeModelTypeSyncBridge::GetClientTag(
@@ -332,9 +335,9 @@ void FakeModelTypeSyncBridge::SetConflictResolution(
       base::MakeUnique<ConflictResolution>(std::move(resolution));
 }
 
-void FakeModelTypeSyncBridge::ErrorOnNextCall(SyncError::ErrorType error_type) {
+void FakeModelTypeSyncBridge::ErrorOnNextCall() {
   DCHECK(!bridge_error_.IsSet());
-  bridge_error_ = SyncError(FROM_HERE, error_type, "TestError", PREFERENCES);
+  bridge_error_ = ModelError(FROM_HERE, "boom");
 }
 
 void FakeModelTypeSyncBridge::CheckPostConditions() {