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

Unified Diff: chrome/common/media_router/mojo/media_router.mojom

Issue 2837363002: [Media Router] Use DialMediaSinkService in MediaRouterMojoImpl (Closed)
Patch Set: resolve code review comments from Mark Created 3 years, 8 months 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/common/media_router/mojo/media_router.mojom
diff --git a/chrome/common/media_router/mojo/media_router.mojom b/chrome/common/media_router/mojo/media_router.mojom
index f1c657680314510c0ed740c22ce65c529ea58093..3a71d9ba04eb98c71306ff7158205940ea427e32 100644
--- a/chrome/common/media_router/mojo/media_router.mojom
+++ b/chrome/common/media_router/mojo/media_router.mojom
@@ -187,6 +187,15 @@ enum RouteRequestResultCode {
// New values must be added here.
};
+// Pass feature flags from browser to Media Route Provider
+struct MediaRouteProviderConfig {
+ // if browser side DIAL device discovery is enabled.
mark a. foltz 2017/05/01 20:32:57 Would it make more sense to reverse the sense here
zhaobin 2017/05/03 01:54:00 Done.
+ bool enable_dial_discovery;
+
+ // If browser side Cast device discovery is enabled.
+ bool enable_cast_discovery;
+};
+
// Modeled after the MediaRouter interface defined in
// chrome/browser/media/router/media_router.h
interface MediaRouteProvider {
@@ -423,7 +432,7 @@ interface MediaRouter {
// Returns a string that uniquely identifies the Media Router browser
// process.
RegisterMediaRouteProvider(MediaRouteProvider media_router_provider) =>
- (string instance_id);
+ (string instance_id, MediaRouteProviderConfig config);
// Called when the Media Route Manager receives a new list of |sinks|
// compatible with |media_source|. The result is only valid for |origins|. If

Powered by Google App Engine
This is Rietveld 408576698