Remove RenderViewHost parameter from WebContentsObserver::Did{Start|Stop}Loading.

This parameter is unused in almost every class derived from WebContentsObserver.
This will make refactoring the OnDid{Start|Stop}Loading IPCs from
WebContentsImpl to RenderFrameHostImpl simpler.

BUG=429399,450799

Review URL: https://codereview.chromium.org/1008913002

Cr-Commit-Position: refs/heads/master@{#321341}

1 files changed, 2 insertions, 4 deletions

diff --git a/content/browser/frame_host/navigation_controller_impl_browsertest.cc b/content/browser/frame_host/navigation_controller_impl_browsertest.cc
index 66646f7..6f54e00 100644
--- a/content/browser/frame_host/navigation_controller_impl_browsertest.cc
+++ b/content/browser/frame_host/navigation_controller_impl_browsertest.cc
@@ -289,7 +289,7 @@ class FrameNavigateParamsCapturer : public WebContentsObserver {
       message_loop_runner_->Quit();
   }
 
-  void DidStopLoading(RenderViewHost* render_view_host) override {
+  void DidStopLoading() override {
     if (!navigations_remaining_)
       message_loop_runner_->Quit();
   }
@@ -359,9 +359,7 @@ class LoadCommittedCapturer : public WebContentsObserver {
       message_loop_runner_->Quit();
   }
 
-  void DidStopLoading(RenderViewHost* render_view_host) override {
-    message_loop_runner_->Quit();
-  }
+  void DidStopLoading() override { message_loop_runner_->Quit(); }
 
   // The id of the FrameTreeNode whose navigations to observe.
   int frame_tree_node_id_;