[Media Router] Add ProvideSinks() Mojo API

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>
 
@@ skipping 371 matching lines @@
     const std::string& domain,
     const MediaSinkSearchResponseCallback& sink_callback) {
   DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
 
   SetWakeReason(MediaRouteProviderWakeReason::SEARCH_SINKS);
   RunOrDefer(base::Bind(&MediaRouterMojoImpl::DoSearchSinks,
                         base::Unretained(this), sink_id, source_id,
                         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);
 
   // Create an observer list for the media source and add |observer|
   // to it. Fail if |observer| is already registered.
   const std::string& source_id = observer->source().id();
   std::unique_ptr<MediaSinksQuery>& sinks_query = sinks_queries_[source_id];
   bool new_query = false;
   if (!sinks_query) {
@@ skipping 252 matching lines @@
     const std::string& domain,
     const MediaSinkSearchResponseCallback& sink_callback) {
   DVLOG_WITH_INSTANCE(1) << "SearchSinks";
   auto sink_search_criteria = mojom::SinkSearchCriteria::New();
   sink_search_criteria->input = search_input;
   sink_search_criteria->domain = domain;
   media_route_provider_->SearchSinks(
       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) {
   DVLOG_WITH_INSTANCE(1) << "OnRouteMessagesReceived";
 
   if (messages.empty())
     return;
 
   auto it = message_observers_.find(route_id);
   if (it == message_observers_.end()) {
@@ skipping 259 matching lines @@
                         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