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

Unified Diff: chrome/browser/prerender/prerender_manager.cc

Issue 1782963002: Remove std::mem_fun uses. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 9 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/prerender/prerender_manager.cc
diff --git a/chrome/browser/prerender/prerender_manager.cc b/chrome/browser/prerender/prerender_manager.cc
index 4bb07b23169a97432e1a9cb06d1239f8751ae0d1..f8093165d41e350bb503470ed64c5ced684e2ca0 100644
--- a/chrome/browser/prerender/prerender_manager.cc
+++ b/chrome/browser/prerender/prerender_manager.cc
@@ -1039,16 +1039,14 @@ void PrerenderManager::PeriodicCleanup() {
// Grab a copy of the current PrerenderContents pointers, so that we
// will not interfere with potential deletions of the list.
- std::vector<PrerenderContents*>
- prerender_contents(active_prerenders_.size());
- std::transform(active_prerenders_.begin(), active_prerenders_.end(),
- prerender_contents.begin(),
- std::mem_fun(&PrerenderData::contents));
+ std::vector<PrerenderContents*> prerender_contents;
+ prerender_contents.reserve(active_prerenders_.size());
+ for (auto* prerender : active_prerenders_)
+ prerender_contents.push_back(prerender->contents());
// And now check for prerenders using too much memory.
- std::for_each(prerender_contents.begin(), prerender_contents.end(),
- std::mem_fun(
- &PrerenderContents::DestroyWhenUsingTooManyResources));
+ for (auto* contents : prerender_contents)
+ contents->DestroyWhenUsingTooManyResources();
// Measure how long the resource checks took. http://crbug.com/305419.
UMA_HISTOGRAM_TIMES("Prerender.PeriodicCleanupResourceCheckTime",
« no previous file with comments | « chrome/browser/browsing_data/cookies_tree_model.cc ('k') | chrome/browser/search_engines/search_provider_install_data.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698