diff options
author | davidben@chromium.org <davidben@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-03-12 10:31:13 +0000 |
---|---|---|
committer | davidben@chromium.org <davidben@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-03-12 10:31:13 +0000 |
commit | cdf4ea57020a37c6a8a9575b2215dc0cea843edc (patch) | |
tree | d52771acc6958ec19feb83d18c0f3f109ffa8842 /content/browser | |
parent | 827d8ae8fadd4afb373ea7249e86f12e553225fe (diff) | |
download | chromium_src-cdf4ea57020a37c6a8a9575b2215dc0cea843edc.zip chromium_src-cdf4ea57020a37c6a8a9575b2215dc0cea843edc.tar.gz chromium_src-cdf4ea57020a37c6a8a9575b2215dc0cea843edc.tar.bz2 |
Remove RVH::OnNavigate and RVH::OnDidStartProvisionalLoad.
They have no callers left.
BUG=none
TEST=compiles
Review URL: https://codereview.chromium.org/195773002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@256512 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'content/browser')
-rw-r--r-- | content/browser/renderer_host/render_view_host_impl.cc | 13 | ||||
-rw-r--r-- | content/browser/renderer_host/render_view_host_impl.h | 10 |
2 files changed, 0 insertions, 23 deletions
diff --git a/content/browser/renderer_host/render_view_host_impl.cc b/content/browser/renderer_host/render_view_host_impl.cc index f51b5df..e1b220c 100644 --- a/content/browser/renderer_host/render_view_host_impl.cc +++ b/content/browser/renderer_host/render_view_host_impl.cc @@ -1233,19 +1233,6 @@ void RenderViewHostImpl::OnRenderProcessGone(int status, int exit_code) { exit_code); } -void RenderViewHostImpl::OnDidStartProvisionalLoadForFrame( - int parent_routing_id, - bool is_main_frame, - const GURL& url) { - NOTREACHED(); -} - -void RenderViewHostImpl::OnNavigate(const IPC::Message& msg) { - // TODO(nasko): Forward calls to the top level RenderFrameHost until all - // callers of this method on RenderViewHost are removed. - delegate_->GetFrameTree()->GetMainFrame()->OnMessageReceived(msg); -} - void RenderViewHostImpl::OnUpdateState(int32 page_id, const PageState& state) { // Without this check, the renderer can trick the browser into using // filenames it can't access in a future session restore. diff --git a/content/browser/renderer_host/render_view_host_impl.h b/content/browser/renderer_host/render_view_host_impl.h index 209b1f6..2e00f54 100644 --- a/content/browser/renderer_host/render_view_host_impl.h +++ b/content/browser/renderer_host/render_view_host_impl.h @@ -484,15 +484,6 @@ class CONTENT_EXPORT RenderViewHostImpl // RenderFrameHost. void AttachToFrameTree(); - // The following IPC handlers are public so RenderFrameHost can call them, - // while we transition the code to not use RenderViewHost. - // - // TODO(nasko): Remove those methods once we are done moving navigation - // into RenderFrameHost. - void OnDidStartProvisionalLoadForFrame(int parent_routing_id, - bool main_frame, - const GURL& url); - // Increases the refcounting on this RVH. This is done by the FrameTree on // creation of a RenderFrameHost. void increment_ref_count() { ++frames_ref_count_; } @@ -531,7 +522,6 @@ class CONTENT_EXPORT RenderViewHostImpl void OnShowFullscreenWidget(int route_id); void OnRenderViewReady(); void OnRenderProcessGone(int status, int error_code); - void OnNavigate(const IPC::Message& msg); void OnUpdateState(int32 page_id, const PageState& state); void OnUpdateTitle(int32 page_id, const base::string16& title, |