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

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

 // 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.
 
 #include "chrome/browser/media/router/mojo/media_router_mojo_impl.h"
 
 #include <stddef.h>
 #include <utility>
 
 #include "base/bind.h"
@@ skipping 409 matching lines @@
   DCHECK(thread_checker_.CalledOnValidThread());
   issue_manager_.AddIssue(issue);
 }
 
 void MediaRouterMojoImpl::ClearIssue(const Issue::Id& issue_id) {
   DCHECK(thread_checker_.CalledOnValidThread());
   issue_manager_.ClearIssue(issue_id);
 }
 
 void MediaRouterMojoImpl::OnUserGesture() {
+  DoUserGestureDiscovery();
 #if defined(OS_WIN)
   EnsureMdnsDiscoveryEnabled();
 #endif
 }
 
 bool MediaRouterMojoImpl::RegisterMediaSinksObserver(
     MediaSinksObserver* observer) {
   DCHECK(thread_checker_.CalledOnValidThread());
 
   // Create an observer list for the media source and add |observer|
@@ skipping 519 matching lines @@
   DCHECK(current_wake_reason_ != MediaRouteProviderWakeReason::TOTAL_COUNT);
   current_wake_reason_ = MediaRouteProviderWakeReason::TOTAL_COUNT;
 }
 
 #if defined(OS_WIN)
 void MediaRouterMojoImpl::EnsureMdnsDiscoveryEnabled() {
   if (is_mdns_enabled_)
     return;
 
   SetWakeReason(MediaRouteProviderWakeReason::ENABLE_MDNS_DISCOVERY);
   RunOrDefer(base::Bind(&MediaRouterMojoImpl::DoEnsureMdnsDiscoveryEnabled,

[amp, 2016/04/07 20:37:39]

Do I need this RunOrDefer logic for the discovery change as well?

Is there some thread handling going on here from the user gesture to the MR
process or something?

[apacible, 2016/04/08 16:46:44]

Good catch; see mfoltz@'s comment in patch set 2.

                         base::Unretained(this)));
   should_enable_mdns_discovery_ = true;
 }
 
 void MediaRouterMojoImpl::DoEnsureMdnsDiscoveryEnabled() {
   DVLOG_WITH_INSTANCE(1) << "DoEnsureMdnsDiscoveryEnabled";
   if (!is_mdns_enabled_) {
     media_route_provider_->EnableMdnsDiscovery();
     is_mdns_enabled_ = true;
   }
 }
 
 void MediaRouterMojoImpl::OnFirewallCheckComplete(
     bool firewall_can_use_local_ports) {
   if (firewall_can_use_local_ports)
     EnsureMdnsDiscoveryEnabled();
 }
 #endif
 
+void MediaRouterMojoImpl::DoUserGestureDiscovery() {
+    media_route_provider_->StartDiscovery();

[apacible, 2016/04/06 17:50:18]

nit: indent

[amp, 2016/04/07 20:37:39]

Done.

+}
+
 }  // namespace media_router