Remove usage of RenderViewHost which aren't needed anymore.

content/browser/frame_host/navigator_impl.cc

Index: content/browser/frame_host/navigator_impl.cc
diff --git a/content/browser/frame_host/navigator_impl.cc b/content/browser/frame_host/navigator_impl.cc
index ebd10e9739d85a3dbb38636447002fd1893e0df6..3af3c3944d3556e33451861c534c126db0194537 100644
--- a/content/browser/frame_host/navigator_impl.cc
+++ b/content/browser/frame_host/navigator_impl.cc
@@ -265,8 +265,7 @@ void NavigatorImpl::DidStartProvisionalLoad(
                                              std::string(),
                                              controller_->GetBrowserContext()));
       entry->set_site_instance(
-          static_cast<SiteInstanceImpl*>(
-              render_frame_host->render_view_host()->GetSiteInstance()));
+          static_cast<SiteInstanceImpl*>(render_frame_host->GetSiteInstance()));

[Charlie Reis, 2015/02/04 20:37:14]

We can drop the static_cast here.  (Co-variant return types FTW.)

[nasko, 2015/02/04 20:42:44]

Ah, good point!

       // TODO(creis): If there's a pending entry already, find a safe way to
       // update it instead of replacing it and copying over things like this.
       if (pending_entry) {