[rAc - i18n] don't call callbacks on Storage destruction

third_party/libaddressinput/chromium/chrome_storage_impl.cc

 // Copyright 2014 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.
 
 #include "third_party/libaddressinput/chromium/chrome_storage_impl.h"
 
 #include "base/prefs/writeable_pref_store.h"
 #include "base/values.h"
 
 ChromeStorageImpl::ChromeStorageImpl(WriteablePrefStore* store)
     : backing_store_(store),
       scoped_observer_(this) {
   scoped_observer_.Add(backing_store_);
 }
 
-ChromeStorageImpl::~ChromeStorageImpl() {
-  // TODO(estade): this shouldn't be necessary.
-  for (std::vector<Request*>::iterator iter =
-           outstanding_requests_.begin();
-       iter != outstanding_requests_.end(); ++iter) {
-    (*(*iter)->callback)(false, (*iter)->key, std::string());
-  }
-}
+ChromeStorageImpl::~ChromeStorageImpl() {}
 
 void ChromeStorageImpl::Put(const std::string& key, const std::string& data) {
   backing_store_->SetValue(key, new base::StringValue(data));
 }
 
 void ChromeStorageImpl::Get(
     const std::string& key,
     scoped_ptr<Storage::Callback> data_ready) const {
   // |Get()| should not be const, so this is just a thunk that fixes that.
   const_cast<ChromeStorageImpl*>(this)->DoGet(key, data_ready.Pass());
@@ skipping 27 matching lines @@
     (*data_ready)(true, key, result);
   } else {
     (*data_ready)(false, key, std::string());
   }
 }
 
 ChromeStorageImpl::Request::Request(const std::string& key,
                                     scoped_ptr<Storage::Callback> callback)
     : key(key),
       callback(callback.Pass()) {}