libaddressinput - reduce number of copies in storage class by 2

third_party/libaddressinput/chromium/cpp/src/retriever.cc

 // Copyright (C) 2013 Google Inc.
 //
 // Licensed under the Apache License, Version 2.0 (the "License");
 // you may not use this file except in compliance with the License.
 // You may obtain a copy of the License at
 //
 // http://www.apache.org/licenses/LICENSE-2.0
 //
 // Unless required by applicable law or agreed to in writing, software
 // distributed under the License is distributed on an "AS IS" BASIS,
@@ skipping 193 matching lines @@
 void Retriever::OnDownloaded(bool success,
                              const std::string& url,
                              scoped_ptr<std::string> downloaded_data) {
   const std::string& key = GetKeyForUrl(url);
   std::map<std::string, std::string>::iterator stale_data_it =
       stale_data_.find(key);
 
   if (success) {
     InvokeCallbackForKey(key, success, *downloaded_data);
     AppendTimestamp(downloaded_data.get());
-    // TODO(estade): storage should take a scoped_ptr.
-    storage_->Put(key, *downloaded_data);
+    storage_->Put(key, downloaded_data.Pass());
   } else if (stale_data_it != stale_data_.end()) {
     InvokeCallbackForKey(key, true, stale_data_it->second);
   } else {
     std::string fallback;
     success = FallbackDataStore::Get(key, &fallback);
     InvokeCallbackForKey(key, success, fallback);
   }
 
   if (stale_data_it != stale_data_.end()) {
     stale_data_.erase(stale_data_it);
@@ skipping 28 matching lines @@
   scoped_ptr<Callback> callback(iter->second);
   requests_.erase(iter);
   if (callback == NULL) {
     return;
   }
   (*callback)(success, key, data);
 }
 
 }  // namespace addressinput
 }  // namespace i18n