Remove usage of [Client=..] from web_ui_mojo test.

content/browser/webui/web_ui_mojo_browsertest.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <limits>
 
 #include "base/command_line.h"
 #include "base/files/file_path.h"
 #include "base/files/file_util.h"
 #include "base/run_loop.h"
@@ skipping 49 matching lines @@
   return true;
 }
 
 class BrowserTargetImpl : public mojo::InterfaceImpl<BrowserTarget> {
  public:
   explicit BrowserTargetImpl(base::RunLoop* run_loop) : run_loop_(run_loop) {}
 
   ~BrowserTargetImpl() override {}
 
   // mojo::InterfaceImpl<BrowserTarget> overrides:
-  void PingResponse() override { NOTREACHED(); }
+  void Start(const mojo::Closure& closure) override {
+    closure.Run();
+  }
+  void Stop() override {
+    got_message = true;
+    run_loop_->Quit();
+  }
 
  protected:
   base::RunLoop* run_loop_;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(BrowserTargetImpl);
 };
 
-class PingBrowserTargetImpl : public BrowserTargetImpl {
- public:
-  explicit PingBrowserTargetImpl(base::RunLoop* run_loop)
-      : BrowserTargetImpl(run_loop) {}
-
-  ~PingBrowserTargetImpl() override {}
-
-  // Quit the RunLoop when called.
-  void PingResponse() override {
-    got_message = true;
-    run_loop_->Quit();
-  }
-
- private:
-  DISALLOW_COPY_AND_ASSIGN(PingBrowserTargetImpl);
-};
-
 // WebUIController that sets up mojo bindings.
 class TestWebUIController : public WebUIController {
  public:
    TestWebUIController(WebUI* web_ui, base::RunLoop* run_loop)
       : WebUIController(web_ui),
         run_loop_(run_loop) {
     content::WebUIDataSource* data_source =
         WebUIDataSource::AddMojoDataSource(
             web_ui->GetWebContents()->GetBrowserContext());
     data_source->SetRequestFilter(base::Bind(&GetResource));
@@ skipping 10 matching lines @@
 // TestWebUIController that additionally creates the ping test BrowserTarget
 // implementation at the right time.
 class PingTestWebUIController : public TestWebUIController {
  public:
    PingTestWebUIController(WebUI* web_ui, base::RunLoop* run_loop)
        : TestWebUIController(web_ui, run_loop) {
    }
    ~PingTestWebUIController() override {}
 
   // WebUIController overrides:
-   void RenderViewCreated(RenderViewHost* render_view_host) override {
+  void RenderViewCreated(RenderViewHost* render_view_host) override {
     render_view_host->GetMainFrame()->GetServiceRegistry()->
         AddService<BrowserTarget>(base::Bind(
             &PingTestWebUIController::CreateHandler, base::Unretained(this)));
   }
 
   void CreateHandler(mojo::InterfaceRequest<BrowserTarget> request) {
     browser_target_.reset(mojo::WeakBindToRequest(
-        new PingBrowserTargetImpl(run_loop_), &request));
-    browser_target_->client()->Ping();
+        new BrowserTargetImpl(run_loop_), &request));
   }
 
  private:
   DISALLOW_COPY_AND_ASSIGN(PingTestWebUIController);
 };
 
 // WebUIControllerFactory that creates TestWebUIController.
 class TestWebUIControllerFactory : public WebUIControllerFactory {
  public:
   TestWebUIControllerFactory() : run_loop_(NULL) {}
@@ skipping 77 matching lines @@
   NavigateToURL(other_shell, test_url);
   // RunLoop is quit when message received from page.
   other_run_loop.Run();
   EXPECT_TRUE(got_message);
   EXPECT_EQ(shell()->web_contents()->GetRenderProcessHost(),
             other_shell->web_contents()->GetRenderProcessHost());
 }
 
 }  // namespace
 }  // namespace content