Non-Local Join for Media Router and Presentation API

chrome/browser/media/router/media_router.mojom

Index: chrome/browser/media/router/media_router.mojom
diff --git a/chrome/browser/media/router/media_router.mojom b/chrome/browser/media/router/media_router.mojom
index 2330a1e94c804abb5c5d7bfaee1502032ed59a4a..6407086b4ab82272e7fbb9308ef72f74c12636de 100644
--- a/chrome/browser/media/router/media_router.mojom
+++ b/chrome/browser/media/router/media_router.mojom
@@ -162,10 +162,10 @@ interface MediaRouteProvider {
   // Starts reporting the state of active media routes via
   // OnRoutesUpdated(). Querying will continue until
   // StopObservingMediaRoutes() is called.

[imcheng, 2015/11/19 18:55:08]

Please update comments.

[matt.boetger, 2015/11/24 19:45:23]

Done.

-  StartObservingMediaRoutes();
+  StartObservingMediaRoutes(string media_source);
 
   // Stops querying the state of all media routes.

[imcheng, 2015/11/19 18:55:08]

ditto

[matt.boetger, 2015/11/24 19:45:23]

Done.

-  StopObservingMediaRoutes();
+  StopObservingMediaRoutes(string media_source);
 
   // Called when the MediaRouter is ready to get the next batch of messages
   // associated with |route_id|.
@@ -214,7 +214,8 @@ interface MediaRouter {
   OnIssue(Issue issue);
 
   // Called when list of routes has been updated.
-  OnRoutesUpdated(array<MediaRoute> routes);
+  OnRoutesUpdated(string media_source, array<MediaRoute> routes,
+    array<MediaRoute> joinableRoutes);

[imcheng, 2015/11/19 18:55:08]

+2 indent

[imcheng, 2015/11/19 18:55:08]

Since joinableRoutes is a subset of routes, would it be possible for it to be
just route ids?

[matt.boetger, 2015/11/24 19:45:23]

Done.

[matt.boetger, 2015/11/24 19:45:23]

Done.

 
   // Called when the overall availability of media sinks has been updated.
   OnSinkAvailabilityUpdated(SinkAvailability availability);