[Media Router] Use DialMediaSinkService in MediaRouterMojoImpl

chrome/browser/media/router/mojo/media_router_mojo_test.h

 // Copyright 2015 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.
 
 #ifndef CHROME_BROWSER_MEDIA_ROUTER_MOJO_MEDIA_ROUTER_MOJO_TEST_H_
 #define CHROME_BROWSER_MEDIA_ROUTER_MOJO_MEDIA_ROUTER_MOJO_TEST_H_
 
 #include <string>
 
 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
+#include "base/test/mock_callback.h"
 #include "chrome/browser/media/router/mock_media_router.h"
 #include "chrome/browser/media/router/mojo/media_router_mojo_impl.h"
 #include "chrome/browser/media/router/test_helper.h"
 #include "chrome/common/media_router/mojo/media_router.mojom.h"
 #include "chrome/test/base/testing_profile.h"
 #include "content/public/test/test_browser_thread_bundle.h"
 #include "extensions/browser/event_page_tracker.h"
 #include "extensions/common/extension.h"
 #include "mojo/public/cpp/bindings/binding.h"
 #include "testing/gmock/include/gmock/gmock.h"
@@ skipping 128 matching lines @@
 class MockMediaRouteControllerObserver : public MediaRouteController::Observer {
  public:
   MockMediaRouteControllerObserver(
       scoped_refptr<MediaRouteController> controller);
   ~MockMediaRouteControllerObserver() override;
 
   MOCK_METHOD1(OnMediaStatusUpdated, void(const MediaStatus& status));
   MOCK_METHOD0(OnControllerInvalidated, void());
 };
 
+// Mockable class for awaiting RegisterMediaRouteProvider callbacks.
+class RegisterMediaRouteProviderHandler {
+ public:
+  RegisterMediaRouteProviderHandler();
+  ~RegisterMediaRouteProviderHandler();
+
+  void Invoke(const std::string& instance_id,
+              mojom::MediaRouteProviderConfigPtr config) {

[Kevin M, 2017/05/03 21:07:28]

This is to deal with move-only |config|, right? Maybe add a little comment to
that effect here.

[zhaobin, 2017/05/04 23:52:36]

Done.

+    InvokeRaw(instance_id, config.get());
+  }
+
+  MOCK_METHOD2(InvokeRaw,
+               void(const std::string& instance_id,
+                    mojom::MediaRouteProviderConfig* config));
+};
+
 // Tests the API call flow between the MediaRouterMojoImpl and the Media Router
 // Mojo service in both directions.
 class MediaRouterMojoTest : public ::testing::Test {
  public:
   MediaRouterMojoTest();
   ~MediaRouterMojoTest() override;
 
  protected:
   void SetUp() override;
   void TearDown() override;
 
   void ProcessEventLoop();
 
   void ConnectProviderManagerService();
 
   const std::string& extension_id() const { return extension_->id(); }
 
   MediaRouterMojoImpl* router() const { return mock_media_router_.get(); }
 
   // Mock objects.
   MockMediaRouteProvider mock_media_route_provider_;
   testing::NiceMock<MockEventPageTracker> mock_event_page_tracker_;
 
   // Mojo proxy object for |mock_media_router_|
   media_router::mojom::MediaRouterPtr media_router_proxy_;
 
+  RegisterMediaRouteProviderHandler provide_handler_;
+
  private:
   content::TestBrowserThreadBundle test_thread_bundle_;
   scoped_refptr<extensions::Extension> extension_;
+  std::unique_ptr<TestingProfile> profile_;

[Kevin M, 2017/05/03 21:07:28]

Why is this a unique_ptr?

[zhaobin, 2017/05/04 23:52:36]

Done.

   std::unique_ptr<MediaRouterMojoImpl> mock_media_router_;

[Kevin M, 2017/05/03 21:07:28]

Ditto here

[zhaobin, 2017/05/04 23:52:36]

It was uniqe_ptr before. Leave it as it it...

   std::unique_ptr<mojo::Binding<mojom::MediaRouteProvider>> binding_;
 
   DISALLOW_COPY_AND_ASSIGN(MediaRouterMojoTest);
 };
 
 }  // namespace media_router
 
 #endif  // CHROME_BROWSER_MEDIA_ROUTER_MOJO_MEDIA_ROUTER_MOJO_TEST_H_