Show Media Router toolbar icon ephemerally for MR dialogs

chrome/browser/media/router/test_helper.cc

Index: chrome/browser/media/router/test_helper.cc
diff --git a/chrome/browser/media/router/test_helper.cc b/chrome/browser/media/router/test_helper.cc
index 8d38dd65ed55d9b8af85535c8e45f51a69cc4e37..e848cbe18ba51366777fa0073c0a567d51e1c541 100644
--- a/chrome/browser/media/router/test_helper.cc
+++ b/chrome/browser/media/router/test_helper.cc
@@ -4,10 +4,16 @@
 
 #include "chrome/browser/media/router/test_helper.h"
 
+#include "chrome/browser/media/router/media_router_factory.h"
 #include "chrome/browser/media/router/media_source.h"
 
 namespace media_router {
 
+void MockPresentationSessionSuccessCallback(

[mark a. foltz, 2016/10/17 18:57:39]

PresentationSessionSessionSuccessCallback is just a base::Callback with template
arguments.

https://cs.chromium.org/chromium/src/chrome/browser/media/router/create_prese...

It would seem better to bind a function in the unit test that verifies if the
callback is called, versus implementing a do-nothing callback.  Check other unit
tests for examples of this pattern.

[takumif, 2016/10/27 23:06:38]

Moving the functions to media_router_dialog_controller_impl_unittest.cc, and
making them mock functions.

+    const content::PresentationSessionInfo&,
+    const MediaRoute::Id&) {}
+void MockPresentationSessionErrorCallback(const content::PresentationError&) {}
+
 MockIssuesObserver::MockIssuesObserver(MediaRouter* router)
     : IssuesObserver(router) {}
 MockIssuesObserver::~MockIssuesObserver() {}
@@ -32,4 +38,24 @@ MockPresentationConnectionStateChangedCallback::
 MockPresentationConnectionStateChangedCallback::
     ~MockPresentationConnectionStateChangedCallback() = default;
 
+MockMediaRouterActionController::MockMediaRouterActionController(
+    Profile* profile)
+    : MediaRouterActionController(
+          profile,
+          media_router::MediaRouterFactory::GetApiForBrowserContext(profile),
+          nullptr,
+          nullptr) {}
+MockMediaRouterActionController::~MockMediaRouterActionController() {}
+
+MockMediaRouterUIService::MockMediaRouterUIService(Profile* profile)
+    : media_router::MediaRouterUIService(profile),
+      action_controller_(
+          base::MakeUnique<MockMediaRouterActionController>(profile)) {}
+MockMediaRouterUIService::~MockMediaRouterUIService() {}
+
+MediaRouterActionController* MockMediaRouterUIService::action_controller()
+    const {
+  return action_controller_.get();
+}
+
 }  // namespace media_router