Index: components/sync_driver/device_info_service.cc |
diff --git a/components/sync_driver/device_info_service.cc b/components/sync_driver/device_info_service.cc |
index 6a63dcbeb0a0b68746b37410b56a3c585b3c3574..45d513d89c70e51c6f63546555eaa5a2c0fdab62 100644 |
--- a/components/sync_driver/device_info_service.cc |
+++ b/components/sync_driver/device_info_service.cc |
@@ -42,8 +42,10 @@ using WriteBatch = ModelTypeStore::WriteBatch; |
DeviceInfoService::DeviceInfoService( |
sync_driver::LocalDeviceInfoProvider* local_device_info_provider, |
- const StoreFactoryFunction& callback) |
- : local_device_info_provider_(local_device_info_provider), |
+ const StoreFactoryFunction& callback, |
+ const ChangeProcessorFactory& change_processor_factory) |
+ : ModelTypeService(change_processor_factory), |
+ local_device_info_provider_(local_device_info_provider), |
weak_factory_(this) { |
DCHECK(local_device_info_provider); |
@@ -370,6 +372,9 @@ void DeviceInfoService::OnReadAllMetadata( |
Result result, |
scoped_ptr<RecordList> metadata_records, |
const std::string& global_metadata) { |
+ if (metadata_records->size() > 0 || global_metadata.size() > 0) { |
+ GetOrCreateChangeProcessor(); |
+ } |
if (!change_processor()) { |
// This datatype was disabled while this read was oustanding. |
return; |
@@ -421,10 +426,15 @@ void DeviceInfoService::TryReconcileLocalAndStored() { |
} |
void DeviceInfoService::TryLoadAllMetadata() { |
- if (has_data_loaded_ && change_processor()) { |
+ // TODO(skym): fix the double load metadata issue |
skym
2016/03/24 22:51:37
nit: Comments should look like sentences with capi
Gang Wu
2016/03/25 02:07:00
Done.
|
+ if (has_data_loaded_) { |
store_->ReadAllMetadata(base::Bind(&DeviceInfoService::OnReadAllMetadata, |
weak_factory_.GetWeakPtr())); |
} |
} |
+syncer::ModelType DeviceInfoService::type() const { |
+ return syncer::DEVICE_INFO; |
+} |
+ |
} // namespace sync_driver_v2 |