Chromium Code Reviews| Index: chrome/browser/prerender/prerender_contents.cc |
| diff --git a/chrome/browser/prerender/prerender_contents.cc b/chrome/browser/prerender/prerender_contents.cc |
| index e1e197f9711808ed29beae1f5ddc2081aa524c66..cc720d27d663da6718dea1f8b75745f83b37a571 100644 |
| --- a/chrome/browser/prerender/prerender_contents.cc |
| +++ b/chrome/browser/prerender/prerender_contents.cc |
| @@ -88,6 +88,19 @@ class PrerenderContents::TabContentsDelegateImpl |
| prerender_contents_(prerender_contents) { |
| } |
| + virtual TabContents* OpenURLFromTab(TabContents* source, |
| + const OpenURLParams& params) { |
|
mmenke
2011/12/05 15:28:25
nit: OVERRIDE
cbentzel
2011/12/05 15:30:59
Done.
|
| + // |OpenURLFromTab| is typicalled called when a frame performs a navigation |
|
dominich
2011/12/05 14:40:41
nit: typically called :)
cbentzel
2011/12/05 15:30:21
Done.
|
| + // that requires the browser to perform the transition instead of WebKit. |
| + // Examples include prerendering a site that redirects to an app URL, |
| + // or if --enable-strict-site-isolation is specified and the prerendered |
| + // frame redirects to a different origin. |
| + // TODO(cbentzel): Consider supporting this is if it is a common case |
| + // during prerenders. |
| + prerender_contents_->Destroy(FINAL_STATUS_OPEN_URL); |
| + return NULL; |
| + } |
| + |
| // TabContentsDelegate implementation: |
| virtual bool ShouldAddNavigationToHistory( |
| const history::HistoryAddPageArgs& add_page_args, |