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

Unified Diff: chrome/browser/extensions/api/web_navigation/web_navigation_api.h

Issue 10823169: Another attempt at fixing dead frames being tracked by webNavigation (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: patch for landing Created 8 years, 4 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/extensions/api/web_navigation/web_navigation_api.h
diff --git a/chrome/browser/extensions/api/web_navigation/web_navigation_api.h b/chrome/browser/extensions/api/web_navigation/web_navigation_api.h
index 5c5f3c4ab4ebb159ccdcf79dfabf3c76643f8ba8..7f5868e3be7de5559c62fa2cd293721602bfc0cb 100644
--- a/chrome/browser/extensions/api/web_navigation/web_navigation_api.h
+++ b/chrome/browser/extensions/api/web_navigation/web_navigation_api.h
@@ -113,7 +113,11 @@ class WebNavigationTabObserver : public content::NotificationObserver,
// Used for tracking registrations to redirect notifications.
content::NotificationRegistrar registrar_;
+ // The current RenderViewHost of the observed WebContents.
content::RenderViewHost* render_view_host_;
+
+ // During a cross site navigation, the WebContents has a second, pending
+ // RenderViewHost.
content::RenderViewHost* pending_render_view_host_;
DISALLOW_COPY_AND_ASSIGN(WebNavigationTabObserver);

Powered by Google App Engine
This is Rietveld 408576698