[Sync] ModelTypeStore::ReadMetadataCallback now returns a MetadataBatch

components/sync/device_info/device_info_sync_bridge.cc

Index: components/sync/device_info/device_info_sync_bridge.cc
diff --git a/components/sync/device_info/device_info_sync_bridge.cc b/components/sync/device_info/device_info_sync_bridge.cc
index a0f76d8e34493886a2fd102314a6c783834ba008..714b2d6d722c25e923b110790f959783ebc23267 100644
--- a/components/sync/device_info/device_info_sync_bridge.cc
+++ b/components/sync/device_info/device_info_sync_bridge.cc
@@ -368,33 +368,9 @@ void DeviceInfoSyncBridge::LoadMetadataIfReady() {
 }
 
 void DeviceInfoSyncBridge::OnReadAllMetadata(
-    Result result,
-    std::unique_ptr<RecordList> metadata_records,
-    const std::string& global_metadata) {
-  if (result != Result::SUCCESS) {
-    ReportStartupErrorToSync("Load of metadata completely failed.");
-    return;
-  }
-
-  auto batch = base::MakeUnique<MetadataBatch>();
-  ModelTypeState state;
-  if (state.ParseFromString(global_metadata)) {
-    batch->SetModelTypeState(state);
-  } else {
-    ReportStartupErrorToSync("Failed to deserialize global metadata.");
-    return;
-  }
-
-  for (const Record& r : *metadata_records.get()) {
-    sync_pb::EntityMetadata entity_metadata;
-    if (entity_metadata.ParseFromString(r.value)) {
-      batch->AddMetadata(r.id, entity_metadata);
-    } else {
-      ReportStartupErrorToSync("Failed to deserialize entity metadata.");
-    }
-  }
-
-  change_processor()->OnMetadataLoaded(SyncError(), std::move(batch));
+    SyncError error,
+    std::unique_ptr<MetadataBatch> metadata_batch) {
+  change_processor()->OnMetadataLoaded(error, std::move(metadata_batch));

[skym, 2016/11/05 00:45:42]

So much less code!

However, this is perpetuating the pattern where ModelTypeProcessor methods take
SyncError objects.

[maxbogue, 2016/11/07 17:03:39]

Eh, doesn't make it any harder to get rid of in the future.

   ReconcileLocalAndStored();
 }