Index: chrome/browser/extensions/blacklist_state_fetcher.cc |
diff --git a/chrome/browser/extensions/blacklist_state_fetcher.cc b/chrome/browser/extensions/blacklist_state_fetcher.cc |
index f0ff060b400007a312245dd8a2b1c5bff85b7376..5af640d0e740e00fef6417399df9996431124d8c 100644 |
--- a/chrome/browser/extensions/blacklist_state_fetcher.cc |
+++ b/chrome/browser/extensions/blacklist_state_fetcher.cc |
@@ -4,7 +4,6 @@ |
#include "chrome/browser/extensions/blacklist_state_fetcher.h" |
-#include "base/stl_util.h" |
#include "base/strings/stringprintf.h" |
#include "base/threading/thread_task_runner_handle.h" |
#include "chrome/browser/browser_process.h" |
@@ -28,8 +27,6 @@ BlacklistStateFetcher::BlacklistStateFetcher() |
BlacklistStateFetcher::~BlacklistStateFetcher() { |
DCHECK_CURRENTLY_ON(BrowserThread::UI); |
- base::STLDeleteContainerPairFirstPointers(requests_.begin(), requests_.end()); |
- requests_.clear(); |
} |
void BlacklistStateFetcher::Request(const std::string& id, |
@@ -69,10 +66,10 @@ void BlacklistStateFetcher::SendRequest(const std::string& id) { |
request.SerializeToString(&request_str); |
GURL request_url = RequestUrl(); |
- net::URLFetcher* fetcher = |
- net::URLFetcher::Create(url_fetcher_id_++, request_url, |
- net::URLFetcher::POST, this).release(); |
- requests_[fetcher] = id; |
+ std::unique_ptr<net::URLFetcher> fetcher_ptr = net::URLFetcher::Create( |
+ url_fetcher_id_++, request_url, net::URLFetcher::POST, this); |
+ net::URLFetcher* fetcher = fetcher_ptr.get(); |
+ requests_[fetcher] = {std::move(fetcher_ptr), id}; |
fetcher->SetAutomaticallyRetryOn5xx(false); // Don't retry on error. |
fetcher->SetRequestContext(url_request_context_getter_.get()); |
fetcher->SetUploadData("application/octet-stream", request_str); |
@@ -108,17 +105,14 @@ GURL BlacklistStateFetcher::RequestUrl() const { |
void BlacklistStateFetcher::OnURLFetchComplete(const net::URLFetcher* source) { |
DCHECK_CURRENTLY_ON(BrowserThread::UI); |
- std::map<const net::URLFetcher*, std::string>::iterator it = |
- requests_.find(source); |
+ auto it = requests_.find(source); |
if (it == requests_.end()) { |
NOTREACHED(); |
return; |
} |
- std::unique_ptr<const net::URLFetcher> fetcher; |
- |
- fetcher.reset(it->first); |
- std::string id = it->second; |
+ std::unique_ptr<net::URLFetcher> fetcher = std::move(it->second.first); |
+ std::string id = it->second.second; |
requests_.erase(it); |
BlacklistState state; |