Remove stl_util's STLDeleteContainerPairFirstPointers.

chrome/browser/predictors/resource_prefetcher_unittest.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 "chrome/browser/predictors/resource_prefetcher.h"
 
 #include <stddef.h>
 
 #include <memory>
 #include <utility>
@@ skipping 88 matching lines @@
 
   void CheckPrefetcherState(size_t inflight, size_t queue, size_t host) {
     EXPECT_EQ(prefetcher_->inflight_requests_.size(), inflight);
     EXPECT_EQ(prefetcher_->request_queue_.size(), queue);
     EXPECT_EQ(prefetcher_->host_inflight_counts_.size(), host);
   }
 
   net::URLRequest* GetInFlightRequest(const std::string& url_str) {
     GURL url(url_str);
 
-    for (std::list<Request*>::const_iterator it =
-         prefetcher_->request_queue_.begin();
+    for (auto it = prefetcher_->request_queue_.begin();

[pasko, 2016/09/01 12:30:45]

nit: while we are here, can you please also convert it to a range-based loop?

          it != prefetcher_->request_queue_.end(); ++it) {
       EXPECT_NE((*it)->resource_url, url);
     }
-    for (std::map<net::URLRequest*, Request*>::const_iterator it =
-         prefetcher_->inflight_requests_.begin();
+    for (auto it = prefetcher_->inflight_requests_.begin();

[pasko, 2016/09/01 12:30:45]

ditto

          it != prefetcher_->inflight_requests_.end(); ++it) {
       if (it->first->original_url() == url)
         return it->first;
     }
-    EXPECT_TRUE(false) << "Infligh request not found: " << url_str;
+    EXPECT_TRUE(false) << "Inflight request not found: " << url_str;

[pasko, 2016/09/01 12:30:45]

nit: s/Inflight/In flight/

     return NULL;
   }
 
 
   void OnReceivedRedirect(const std::string& url) {
     prefetcher_->OnReceivedRedirect(
         GetInFlightRequest(url), net::RedirectInfo(), NULL);
   }
   void OnAuthRequired(const std::string& url) {
     prefetcher_->OnAuthRequired(GetInFlightRequest(url), NULL);
@@ skipping 220 matching lines @@
   // We need to delete requests_ptr here, though it looks to be managed by the
   // scoped_ptr requests. The scoped_ptr requests releases itself and the raw
   // pointer requests_ptr is passed to ResourcePrefetcherFinished(). In the
   // test, ResourcePrefetcherFinished() is a mock function and does not handle
   // the raw pointer properly. In the real code, requests_ptr will eventually be
   // passed to and managed by ResourcePrefetchPredictor::Result::Result.
   delete requests_ptr;
 }
 
 }  // namespace predictors