[Media Router] Add ProvideSinks() Mojo API

chrome/browser/media/router/media_router.h

Index: chrome/browser/media/router/media_router.h
diff --git a/chrome/browser/media/router/media_router.h b/chrome/browser/media/router/media_router.h
index 0f8fd4b7386f322a16b2950c04efea0b576f99e2..9c27a836c4d2aefe4a11cd43be671e294db411cd 100644
--- a/chrome/browser/media/router/media_router.h
+++ b/chrome/browser/media/router/media_router.h
@@ -17,6 +17,7 @@
 #include "chrome/browser/media/router/issue.h"
 #include "chrome/browser/media/router/media_route.h"
 #include "chrome/browser/media/router/media_sink.h"
+#include "chrome/browser/media/router/media_sink_internal.h"
 #include "chrome/browser/media/router/media_source.h"
 #include "chrome/browser/media/router/route_message_observer.h"
 #include "components/keyed_service/core/keyed_service.h"
@@ -63,6 +64,7 @@ using PresentationConnectionStateSubscription = base::CallbackList<void(
 class MediaRouter : public KeyedService {
  public:
   using SendRouteMessageCallback = base::Callback<void(bool sent)>;
+  using MediaSinkList = std::vector<MediaSinkInternal>;

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

Nit: For consistency with other APIs, use std::vector explicitly instead of
using.

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

Done.

 
   ~MediaRouter() override = default;
 
@@ -172,6 +174,11 @@ class MediaRouter : public KeyedService {
       const std::string& domain,
       const MediaSinkSearchResponseCallback& sink_callback) = 0;
 
+  // Called when DIAL or CAST MediaSinkService finishes sink discovery.

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

How about:

Notifies the Media Router that the list of MediaSinks discovered by a
MediaSinkService has been updated.
|provider_name|: Name of the MediaSinkService providing the sinks.
...

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

Done.

+  // |sinks|: sinks discovered by MediaSinkService.
+  virtual void ProvideSinks(const std::string& provider_name,
+                            const MediaSinkList& sinks) = 0;
+
   // Adds |callback| to listen for state changes for presentation connected to
   // |route_id|. The returned Subscription object is owned by the caller.
   // |callback| will be invoked whenever there are state changes, until the