Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1689)

Unified Diff: chrome/browser/predictors/resource_prefetcher_manager.h

Issue 2318303002: Remove stl_util's STLDeleteContainerPairSecondPointers. (Closed)
Patch Set: fix Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/predictors/resource_prefetcher_manager.h
diff --git a/chrome/browser/predictors/resource_prefetcher_manager.h b/chrome/browser/predictors/resource_prefetcher_manager.h
index 6b21feed48a6dd73906efb74bfc8037a55c993a5..cf925d34bb93e2250656c0a9b249ba162288932a 100644
--- a/chrome/browser/predictors/resource_prefetcher_manager.h
+++ b/chrome/browser/predictors/resource_prefetcher_manager.h
@@ -6,6 +6,7 @@
#define CHROME_BROWSER_PREDICTORS_RESOURCE_PREFETCHER_MANAGER_H_
#include <map>
+#include <memory>
#include "base/macros.h"
#include "base/memory/ref_counted.h"
@@ -21,7 +22,7 @@ namespace predictors {
struct NavigationID;
class ResourcePrefetchPredictor;
-// Manages prefetches for multple navigations.
+// Manages prefetches for multiple navigations.
// - Created and owned by the resource prefetch predictor.
// - Needs to be refcounted as it is de-referenced on two different threads.
// - Created on the UI thread, but most functions are called in the IO thread.
@@ -62,15 +63,13 @@ class ResourcePrefetcherManager
friend class base::RefCountedThreadSafe<ResourcePrefetcherManager>;
friend class MockResourcePrefetcherManager;
- typedef std::map<std::string, ResourcePrefetcher*> PrefetcherMap;
-
~ResourcePrefetcherManager() override;
ResourcePrefetchPredictor* predictor_;
const ResourcePrefetchPredictorConfig config_;
net::URLRequestContextGetter* const context_getter_;
- PrefetcherMap prefetcher_map_; // Owns the ResourcePrefetcher pointers.
+ std::map<std::string, std::unique_ptr<ResourcePrefetcher>> prefetcher_map_;
DISALLOW_COPY_AND_ASSIGN(ResourcePrefetcherManager);
};

Powered by Google App Engine
This is Rietveld 408576698