Index: apps/app_window_contents.cc |
diff --git a/apps/app_window_contents.cc b/apps/app_window_contents.cc |
index 7b88e67e6f6092e9d2b338d4f6be393fd1fa8ec5..56ae403b2d0faff5ce3df1380f2e5920344ab3cd 100644 |
--- a/apps/app_window_contents.cc |
+++ b/apps/app_window_contents.cc |
@@ -60,16 +60,15 @@ void AppWindowContents::LoadContents(int32 creator_process_id) { |
} |
// TODO(jeremya): there's a bug where navigating a web contents to an |
- // extension URL causes it to create a new RVH and discard the old |
- // (perfectly usable) one. To work around this, we watch for a RVH_CHANGED |
- // message from the web contents (which will be sent during LoadURL) and |
- // suspend resource requests on the new RVH to ensure that we block the new |
- // RVH from loading anything. It should be okay to remove the |
- // NOTIFICATION_RVH_CHANGED registration once http://crbug.com/123007 is |
- // fixed. |
+ // extension URL causes it to create a new RVH and discard the old (perfectly |
+ // usable) one. To work around this, we watch for a |
+ // NOTIFICATION_RENDER_VIEW_HOST_CHANGED message from the web contents (which |
+ // will be sent during LoadURL) and suspend resource requests on the new RVH |
+ // to ensure that we block the new RVH from loading anything. It should be |
+ // okay to remove the NOTIFICATION_RENDER_VIEW_HOST_CHANGED registration once |
+ // http://crbug.com/123007 is fixed. |
registrar_.Add(this, content::NOTIFICATION_RENDER_VIEW_HOST_CHANGED, |
- content::Source<content::NavigationController>( |
- &web_contents()->GetController())); |
+ content::Source<content::WebContents>(web_contents())); |
web_contents_->GetController().LoadURL( |
url_, content::Referrer(), content::PAGE_TRANSITION_LINK, |
std::string()); |