[Media Router] Wire through a new MRPM call to update media sinks.

chrome/browser/media/router/mojo/media_router_mojo_impl.cc

Index: chrome/browser/media/router/mojo/media_router_mojo_impl.cc
diff --git a/chrome/browser/media/router/mojo/media_router_mojo_impl.cc b/chrome/browser/media/router/mojo/media_router_mojo_impl.cc
index d9c822cc294ccf15129f028caa447a5b06183d12..666e927f8e7676162ae98f7b4963882babb52db2 100644
--- a/chrome/browser/media/router/mojo/media_router_mojo_impl.cc
+++ b/chrome/browser/media/router/mojo/media_router_mojo_impl.cc
@@ -20,6 +20,7 @@
 #include "chrome/browser/media/router/media_router_factory.h"
 #include "chrome/browser/media/router/media_routes_observer.h"
 #include "chrome/browser/media/router/media_sinks_observer.h"
+#include "chrome/browser/media/router/media_source_helper.h"
 #include "chrome/browser/media/router/mojo/media_route_provider_util_win.h"
 #include "chrome/browser/media/router/mojo/media_router_mojo_metrics.h"
 #include "chrome/browser/media/router/mojo/media_router_type_converters.h"
@@ -429,6 +430,8 @@ void MediaRouterMojoImpl::ClearIssue(const Issue::Id& issue_id) {
 }
 
 void MediaRouterMojoImpl::OnUserGesture() {
+  UpdateMediaSinks(MediaSourceForDesktop().id());

[imcheng, 2016/04/15 18:37:02]

could you add a comment here why this has to be done?

[amp, 2016/04/15 20:29:33]

Done.  This was added per mfoltz comment on
chrome/browser/media/router/mojo/media_router.mojom in patch set 2.

I haven't seen a reply as to whether Desktop is the best source to use here.

The comment I will add will be '// Allow MRPM to intelligently update sinks and
observers by passing in a media source.'

+
 #if defined(OS_WIN)
   EnsureMdnsDiscoveryEnabled();
 #endif
@@ -988,4 +991,17 @@ void MediaRouterMojoImpl::OnFirewallCheckComplete(
 }
 #endif
 
+void MediaRouterMojoImpl::UpdateMediaSinks(
+    const MediaSource::Id& source_id) {
+  SetWakeReason(MediaRouteProviderWakeReason::UPDATE_MEDIA_SINKS);
+  RunOrDefer(base::Bind(&MediaRouterMojoImpl::DoUpdateMediaSinks,
+                        base::Unretained(this), source_id));
+}
+
+void MediaRouterMojoImpl::DoUpdateMediaSinks(
+    const MediaSource::Id& source_id) {
+  DVLOG_WITH_INSTANCE(1) << "DoUpdateMediaSinks" << source_id;
+  media_route_provider_->UpdateMediaSinks(source_id);
+}
+
 }  // namespace media_router