Commit b794e623 authored by pasko's avatar pasko Committed by Commit bot

predictors: Clean up after removing STLDeleteContainerPairFirstPointers

Addressing my own nits expressed in review of:
https://codereview.chromium.org/2293153002/
    (Remove stl_util's STLDeleteContainerPairFirstPointers.)

BUG=555865

Review-Url: https://codereview.chromium.org/2301403002
Cr-Commit-Position: refs/heads/master@{#416239}
parent c7b1ab58
...@@ -7,10 +7,10 @@ ...@@ -7,10 +7,10 @@
#include <stddef.h> #include <stddef.h>
#include <algorithm>
#include <list> #include <list>
#include <map> #include <map>
#include <memory> #include <memory>
#include <utility>
#include <vector> #include <vector>
#include "base/macros.h" #include "base/macros.h"
......
...@@ -106,16 +106,14 @@ class ResourcePrefetcherTest : public testing::Test { ...@@ -106,16 +106,14 @@ class ResourcePrefetcherTest : public testing::Test {
net::URLRequest* GetInFlightRequest(const std::string& url_str) { net::URLRequest* GetInFlightRequest(const std::string& url_str) {
GURL url(url_str); GURL url(url_str);
for (auto it = prefetcher_->request_queue_.begin(); for (auto* request : prefetcher_->request_queue_) {
it != prefetcher_->request_queue_.end(); ++it) { EXPECT_NE(request->resource_url, url);
EXPECT_NE((*it)->resource_url, url);
} }
for (auto it = prefetcher_->inflight_requests_.begin(); for (const auto& key_value : prefetcher_->inflight_requests_) {
it != prefetcher_->inflight_requests_.end(); ++it) { if (key_value.first->original_url() == url)
if (it->first->original_url() == url) return key_value.first;
return it->first;
} }
EXPECT_TRUE(false) << "Inflight request not found: " << url_str; EXPECT_TRUE(false) << "In flight request not found: " << url_str;
return NULL; return NULL;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment