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

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

Issue 10316020: Remove WebContentsImpl::OnDidRedirectProvisionalLoad. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Don't break unittest Created 8 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_tab_helper.h
diff --git a/chrome/browser/prerender/prerender_tab_helper.h b/chrome/browser/prerender/prerender_tab_helper.h
index 7baccbcc06c2104386223849e5803e38b54800c3..55ac8507eae5783912e774474a03905d3678c180 100644
--- a/chrome/browser/prerender/prerender_tab_helper.h
+++ b/chrome/browser/prerender/prerender_tab_helper.h
@@ -7,6 +7,8 @@
#include "base/time.h"
#include "base/memory/scoped_ptr.h"
+#include "content/public/browser/notification_observer.h"
+#include "content/public/browser/notification_registrar.h"
#include "content/public/browser/web_contents_observer.h"
#include "googleurl/src/gurl.h"
@@ -18,16 +20,13 @@ class PrerenderManager;
// PrerenderTabHelper is responsible for recording perceived pageload times
// to compare PLT's with prerendering enabled and disabled.
-class PrerenderTabHelper : public content::WebContentsObserver {
+class PrerenderTabHelper : public content::NotificationObserver,
+ public content::WebContentsObserver {
public:
explicit PrerenderTabHelper(TabContentsWrapper* tab);
virtual ~PrerenderTabHelper();
// content::WebContentsObserver implementation.
- virtual void ProvisionalChangeToMainFrameUrl(
- const GURL& url,
- const GURL& opener_url,
- content::RenderViewHost* render_view_host) OVERRIDE;
virtual void DidStopLoading() OVERRIDE;
virtual void DidStartProvisionalLoadForFrame(
int64 frame_id,
@@ -62,6 +61,13 @@ class PrerenderTabHelper : public content::WebContentsObserver {
bool IsTopSite(const GURL& url);
+ void HandleResourceReceivedRedirect(const GURL& new_url);
+
+ // content::NotificationObserver
+ virtual void Observe(int type,
+ const content::NotificationSource& source,
+ const content::NotificationDetails& details) OVERRIDE;
+
// TabContentsWrapper we're created for.
TabContentsWrapper* tab_;
@@ -77,6 +83,8 @@ class PrerenderTabHelper : public content::WebContentsObserver {
// Current URL being loaded.
GURL url_;
+ content::NotificationRegistrar notification_registrar_;
+
DISALLOW_COPY_AND_ASSIGN(PrerenderTabHelper);
};

Powered by Google App Engine
This is Rietveld 408576698