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

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

Issue 7015026: Cancel prerenders for pages that call window.print() (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Comment responses. Created 9 years, 7 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 a929b7b676e28afc09ec6cd052ff261cfbae8e7e..ac736da834483124c1c86d4c0ccd29a2fdcff7d8 100644
--- a/chrome/browser/prerender/prerender_manager.cc
+++ b/chrome/browser/prerender/prerender_manager.cc
@@ -359,7 +359,7 @@ std::list<PrerenderManager::PrerenderContentsData>::iterator
return it;
}
-void PrerenderManager::DestroyPreloadForChildRouteIdPair(
+bool PrerenderManager::DestroyPreloadForChildRouteIdPair(
const std::pair<int, int>& child_route_id_pair,
FinalStatus final_status) {
DCHECK(CalledOnValidThread());
@@ -368,7 +368,9 @@ void PrerenderManager::DestroyPreloadForChildRouteIdPair(
if (it != prerender_list_.end()) {
PrerenderContents* prerender_contents = it->contents_;
prerender_contents->Destroy(final_status);
+ return true;
}
+ return false;
}
void PrerenderManager::DeleteOldEntries() {

Powered by Google App Engine
This is Rietveld 408576698