Remove a bunch of DeepCopy() calls in the chrome.storage API.

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

 // Copyright (c) 2012 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.
 
 #ifndef CHROME_BROWSER_EXTENSIONS_API_STORAGE_SYNCABLE_SETTINGS_STORAGE_H_
 #define CHROME_BROWSER_EXTENSIONS_API_STORAGE_SYNCABLE_SETTINGS_STORAGE_H_
 
 #include "base/compiler_specific.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 37 matching lines @@
   WriteResult Remove(const std::string& key) override;
   WriteResult Remove(const std::vector<std::string>& keys) override;
   WriteResult Clear() override;
   bool Restore() override;
   bool RestoreKey(const std::string& key) override;
 
   // Sync-related methods, analogous to those on SyncableService (handled by
   // ExtensionSettings), but with looser guarantees about when the methods
   // can be called.
 
-  // Must only be called if sync isn't already active.
+  // Starts syncing this storage area. Must only be called if sync isn't
+  // already active.
+  // |sync_state| is the current state of the extension settings in sync.
+  // |sync_processor| is used to write out any changes.
+  // Returns any error when trying to sync, or an empty error on success.
   syncer::SyncError StartSyncing(
-      const base::DictionaryValue& sync_state,
+      scoped_ptr<base::DictionaryValue> sync_state,
       scoped_ptr<SettingsSyncProcessor> sync_processor);
 
-  // May be called at any time (idempotent).
+  // Stops syncing this storage area. May be called at any time (idempotent).
   void StopSyncing();
 
-  // May be called at any time; changes will be ignored if sync isn't active.
-  syncer::SyncError ProcessSyncChanges(const SettingSyncDataList& sync_changes);
+  // Pushes a list of sync changes into this storage area. May be called at any
+  // time, changes will be ignored if sync isn't active.
+  // Returns any error when trying to sync, or an empty error on success.
+  syncer::SyncError ProcessSyncChanges(
+      scoped_ptr<SettingSyncDataList> sync_changes);
 
  private:
   // Sends the changes from |result| to sync if it's enabled.
   void SyncResultIfEnabled(const ValueStore::WriteResult& result);
 
-  // Sends all local settings to sync (synced settings assumed to be empty).
+  // Sends all local settings to sync. This assumes that there are no settings
+  // in sync yet.
+  // Returns any error when trying to sync, or an empty error on success.
   syncer::SyncError SendLocalSettingsToSync(
-      const base::DictionaryValue& settings);
+      scoped_ptr<base::DictionaryValue> local_state);
 
   // Overwrites local state with sync state.
+  // Returns any error when trying to sync, or an empty error on success.
   syncer::SyncError OverwriteLocalSettingsWithSync(
-      const base::DictionaryValue& sync_state,
-      const base::DictionaryValue& settings);
+      scoped_ptr<base::DictionaryValue> sync_state,
+      scoped_ptr<base::DictionaryValue> local_state);
 
-  // Called when an Add/Update/Remove comes from sync.  Ownership of Value*s
+  // 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);
@@ skipping 16 matching lines @@
 
   const syncer::ModelType sync_type_;
   const syncer::SyncableService::StartSyncFlare flare_;
 
   DISALLOW_COPY_AND_ASSIGN(SyncableSettingsStorage);
 };
 
 }  // namespace extensions
 
 #endif  // CHROME_BROWSER_EXTENSIONS_API_STORAGE_SYNCABLE_SETTINGS_STORAGE_H_