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

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

Issue 146983002: Move pending prerender logic into PrerenderLinkManager. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase Created 6 years, 11 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
« no previous file with comments | « chrome/browser/prerender/prerender_link_manager.cc ('k') | chrome/browser/prerender/prerender_manager.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/prerender/prerender_manager.h
diff --git a/chrome/browser/prerender/prerender_manager.h b/chrome/browser/prerender/prerender_manager.h
index 72cbddfbaa7ca3eab65d67f9ad97311a62ef2b67..de85b8f1e13a5caa8fdf621e8a41821107e7133d 100644
--- a/chrome/browser/prerender/prerender_manager.h
+++ b/chrome/browser/prerender/prerender_manager.h
@@ -230,6 +230,12 @@ class PrerenderManager : public base::SupportsWeakPtr<PrerenderManager>,
PrerenderContents* GetPrerenderContents(
const content::WebContents* web_contents) const;
+ // Returns the PrerenderContents object for a given child_id, route_id pair,
+ // otherwise returns NULL. Note that the PrerenderContents may have been
+ // Destroy()ed, but not yet deleted.
+ virtual PrerenderContents* GetPrerenderContentsForRoute(
+ int child_id, int route_id) const;
+
// Returns a list of all WebContents being prerendered.
const std::vector<content::WebContents*> GetAllPrerenderingContents() const;
@@ -504,13 +510,6 @@ class PrerenderManager : public base::SupportsWeakPtr<PrerenderManager>,
void SetPrerenderContentsFactory(
PrerenderContents::Factory* prerender_contents_factory);
- // Adds prerenders from the pending Prerenders, called by
- // PrerenderContents::StartPendingPrerenders.
- void StartPendingPrerenders(
- int process_id,
- ScopedVector<PrerenderContents::PendingPrerenderInfo>* pending_prerenders,
- content::SessionStorageNamespace* session_storage_namespace);
-
// Called by a PrerenderData to signal that the launcher has navigated away
// from the context that launched the prerender. A user may have clicked
// a link in a page containing a <link rel=prerender> element, or the user
@@ -617,11 +616,6 @@ class PrerenderManager : public base::SupportsWeakPtr<PrerenderManager>,
const GURL& url,
const content::SessionStorageNamespace* session_storage_namespace);
- // If |child_id| and |route_id| correspond to a RenderView that is an active
- // prerender, returns the PrerenderData object for that prerender. Otherwise,
- // returns NULL.
- PrerenderData* FindPrerenderDataForChildAndRoute(int child_id, int route_id);
-
// Finds the active PrerenderData object currently in a PendingSwap for
// |target_contents|. Otherwise, returns NULL.
PrerenderData* FindPrerenderDataForTargetContents(
« no previous file with comments | « chrome/browser/prerender/prerender_link_manager.cc ('k') | chrome/browser/prerender/prerender_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698