[Media Router] Add ProvideSinks() Mojo API

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 5236f70eaa38ae938e6414bcb3785aab921674c5..180311bab29686b340538a674729541fdd5f56e0 100644
--- a/chrome/browser/media/router/mojo/media_router_mojo_impl.cc
+++ b/chrome/browser/media/router/mojo/media_router_mojo_impl.cc
@@ -389,6 +389,16 @@ void MediaRouterMojoImpl::SearchSinks(
                         search_input, domain, sink_callback));
 }
 
+void MediaRouterMojoImpl::ProvideSinks(const std::string& provider_name,
+                                       const MediaSinkList& sinks) {
+  DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
+
+  SetWakeReason(MediaRouteProviderWakeReason::PROVIDE_SINKS);
+  RunOrDefer(base::Bind(&MediaRouterMojoImpl::DoProvideSinks,
+                        base::Unretained(this), provider_name,
+                        base::ConstRef(sinks)));

[mark a. foltz, 2017/03/01 06:31:43]

base::Passed?

[imcheng, 2017/03/01 23:28:36]

or just |sinks|? I assume that the reference is to a vector in a
MediaSinkService, so we should be making a copy in the callback (in case it gets
queued).

[zhaobin, 2017/03/02 20:24:04]

Done.

+}
+
 bool MediaRouterMojoImpl::RegisterMediaSinksObserver(
     MediaSinksObserver* observer) {
   DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
@@ -661,6 +671,12 @@ void MediaRouterMojoImpl::DoSearchSinks(
       sink_id, source_id, std::move(sink_search_criteria), sink_callback);
 }
 
+void MediaRouterMojoImpl::DoProvideSinks(const std::string& provider_name,
+                                         const MediaSinkList& sinks) {
+  DVLOG_WITH_INSTANCE(1) << "DoProvideSinks";
+  media_route_provider_->ProvideSinks(provider_name, sinks);
+}
+
 void MediaRouterMojoImpl::OnRouteMessagesReceived(
     const std::string& route_id,
     const std::vector<RouteMessage>& messages) {