Use std::unique_ptr to transfer base::Value ownership in extensions::ValueStoreChange

chrome/browser/extensions/api/storage/syncable_settings_storage.h

Index: chrome/browser/extensions/api/storage/syncable_settings_storage.h
diff --git a/chrome/browser/extensions/api/storage/syncable_settings_storage.h b/chrome/browser/extensions/api/storage/syncable_settings_storage.h
index c49b49e38235594405a36541c8a76ad94714903c..24456e15f6ee481dc0a1833b7fe507db614ae5a2 100644
--- a/chrome/browser/extensions/api/storage/syncable_settings_storage.h
+++ b/chrome/browser/extensions/api/storage/syncable_settings_storage.h
@@ -98,21 +98,17 @@ class SyncableSettingsStorage : public ValueStore {
       std::unique_ptr<base::DictionaryValue> sync_state,
       std::unique_ptr<base::DictionaryValue> local_state);
 
-  // Called when an Add/Update/Remove comes from sync. Ownership of Value*s
-  // are taken.
-  syncer::SyncError OnSyncAdd(
-      const std::string& key,
-      base::Value* new_value,
-      ValueStoreChangeList* changes);
-  syncer::SyncError OnSyncUpdate(
-      const std::string& key,
-      base::Value* old_value,
-      base::Value* new_value,
-      ValueStoreChangeList* changes);
-  syncer::SyncError OnSyncDelete(
-      const std::string& key,
-      base::Value* old_value,
-      ValueStoreChangeList* changes);
+  // Called when an Add/Update/Remove comes from sync.
+  syncer::SyncError OnSyncAdd(const std::string& key,
+                              std::unique_ptr<base::Value> new_value,
+                              ValueStoreChangeList* changes);
+  syncer::SyncError OnSyncUpdate(const std::string& key,
+                                 std::unique_ptr<base::Value> old_value,
+                                 std::unique_ptr<base::Value> new_value,
+                                 ValueStoreChangeList* changes);
+  syncer::SyncError OnSyncDelete(const std::string& key,
+                                 std::unique_ptr<base::Value> old_value,
+                                 ValueStoreChangeList* changes);
 
   // List of observers to settings changes.
   const scoped_refptr<SettingsObserverList> observers_;