Chromium Code Reviews| Index: chrome/browser/prerender/prerender_manager.h |
| diff --git a/chrome/browser/prerender/prerender_manager.h b/chrome/browser/prerender/prerender_manager.h |
| index f1f46ff28ee356de6ca0f215d9606b287c27abe8..88446acaa0ccd9efbb357776bf7717b54a92c19e 100644 |
| --- a/chrome/browser/prerender/prerender_manager.h |
| +++ b/chrome/browser/prerender/prerender_manager.h |
| @@ -72,6 +72,12 @@ class PrerenderManager : public base::SupportsWeakPtr<PrerenderManager>, |
| PRERENDER_MODE_MAX |
| }; |
| + struct PrerenderedPageProperties { |
|
cbentzel
2011/05/04 17:10:15
This seems like overkill right now.
I'd just have
Shishir
2011/05/05 23:09:54
Then I think we might as well have a bool for now
|
| + PrerenderedPageProperties() : has_opener_set_(false) { } |
| + // Is the page supposed to have the window.opener property set. |
| + bool has_opener_set_; |
| + }; |
| + |
| // Owned by a Profile object for the lifetime of the profile. |
| explicit PrerenderManager(Profile* profile); |
| @@ -100,8 +106,14 @@ class PrerenderManager : public base::SupportsWeakPtr<PrerenderManager>, |
| // determines whether a preloaded version of the URL can be used, |
| // and substitutes the prerendered RVH into the TabContents. Returns |
| // whether or not a prerendered RVH could be used or not. |
| - bool MaybeUsePreloadedPage(TabContents* tab_contents, const GURL& url); |
| - bool MaybeUsePreloadedPageOld(TabContents* tab_contents, const GURL& url); |
| + bool MaybeUsePreloadedPage( |
| + TabContents* tab_contents, |
| + const GURL& url, |
| + const PrerenderManager::PrerenderedPageProperties& properties); |
| + bool MaybeUsePreloadedPageOld( |
| + TabContents* tab_contents, |
| + const GURL& url, |
| + const PrerenderManager::PrerenderedPageProperties& properties); |
| // Allows PrerenderContents to remove itself when prerendering should |
| // be cancelled. |