Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2114)

Unified Diff: chrome/browser/media/router/media_router.mojom

Issue 1415103006: Non-Local Join for Media Router and Presentation API (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Review Fixes Created 5 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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..b75e15b565c5592f67cd0f29b98bf19ee543b948 100644
--- a/chrome/browser/media/router/media_router.mojom
+++ b/chrome/browser/media/router/media_router.mojom
@@ -160,12 +160,15 @@ interface MediaRouteProvider {
StopObservingMediaSinks(string media_source);
// Starts reporting the state of active media routes via
- // OnRoutesUpdated(). Querying will continue until
- // StopObservingMediaRoutes() is called.
- StartObservingMediaRoutes();
+ // OnRoutesUpdated() in the context of the |media_source|.
+ // The |media_source| should be considered when returning joinable
+ // routes in the OnRoutesUpdated() call.
+ // Querying will continue until StopObservingMediaRoutes() is called.
+ StartObservingMediaRoutes(string media_source);
- // Stops querying the state of all media routes.
- StopObservingMediaRoutes();
+ // Stops querying the state of all media routes in the context of
+ // the |media_source|.
+ StopObservingMediaRoutes(string media_source);
// Called when the MediaRouter is ready to get the next batch of messages
// associated with |route_id|.
@@ -213,8 +216,11 @@ interface MediaRouter {
// Called when issues are reported for media routes.
OnIssue(Issue issue);
- // Called when list of routes has been updated.
- OnRoutesUpdated(array<MediaRoute> routes);
+ // Called when list of routes has been updated in the context of the
+ // calling |media_source|. The array |joinable_route_ids| should
+ // contain route IDs of joinable routes found in the |routes| array.
+ OnRoutesUpdated(string media_source, array<MediaRoute> routes,
+ array<string> joinable_route_ids);
// Called when the overall availability of media sinks has been updated.
OnSinkAvailabilityUpdated(SinkAvailability availability);

Powered by Google App Engine
This is Rietveld 408576698