[Media Router] Add MediaSink subtypes

chrome/browser/media/router/mojo/media_router.mojom

 // 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.
 
 module media_router.mojom;
 
 import "mojo/common/time.mojom";
 import "url/mojo/origin.mojom";
+import "url/mojo/url.mojom";
 
 // Represents an output sink to which media can be routed.
 struct MediaSink {
   enum IconType {
     CAST,
     CAST_AUDIO,
     CAST_AUDIO_GROUP,
     GENERIC,
     HANGOUT
   };
 
   // The sink identifier, e.g. "rs71w7mFzYLFlabir_qO4NHl6SUc."
   string sink_id;
   // The human-readable name, e.g. "Janet's Chromecast".
   string name;
   // Optional description of the sink.
   string? description;
   // Optional domain of the sink if this sink is associated with an identity.
   string? domain;
   // The type of icon to show in the UI for this media sink.
   IconType icon_type;
+
+  MediaSinkExtraData? extra_data;

[imcheng, 2017/02/24 23:54:21]

Mention that this is currently only set by MediaRouter in OnSinksDiscovered().

[zhaobin, 2017/02/27 21:45:15]

Done.

+};
+
+union MediaSinkExtraData {
+  DialMediaSink dial_media_sink;
+  CastMediaSink cast_media_sink;
+};
+
+struct DialMediaSink {
+  string ip_address;
+
+  // Model name of the sink, if it represents a physical device.
+  string? model_name;

[imcheng, 2017/02/24 23:54:21]

I think this is no longer optional

[zhaobin, 2017/02/27 21:45:15]

Done.

+
+  // Used for DIAL launch
+  url.mojom.Url app_url;
+};
+
+struct CastMediaSink {
+  string ip_address;
+
+  // Model name of the sink, if it represents a physical device.
+  string? model_name;

[imcheng, 2017/02/24 23:54:21]

Same as above; not optional

[zhaobin, 2017/02/27 21:45:15]

Done.

+
+  // A bit vector representing capabilities of the sink. Meaning of capacity
+  // value for each bit:
+  // NONE: 0,
+  // VIDEO_OUT: 1 << 0,
+  // VIDEO_IN: 1 << 1,
+  // AUDIO_OUT: 1 << 2,
+  // AUDIO_IN: 1 << 3,
+  // DEV_MODE: 1 << 4,
+  // MULTIZONE_GROUP: 1 << 5
+  int32 capabilities;
+
+  // ID of cast channel opened by Media Router. The ID can be
+  // used to send call chrome.cast.channel.send.

[imcheng, 2017/02/24 23:54:21]

nit: the second sentence is probably not necessary.

[mark a. foltz, 2017/02/25 01:10:24]

nit: Cast

I think it would be good to mention that the ID is defined by the
chrome.cast.channel API.

[zhaobin, 2017/02/27 21:45:15]

Done.

+  int32 cast_channel_id;
 };
 
 // Should be kept in sync with media_route.h.
 struct MediaRoute {
   // The ID of this media route, e.g. "r_PR1O_blkC9dsKp-tb1ti8qurOo".
   string media_route_id;
   // The ID of the media source being sent through this media route.
   // May be missing if route is not local.
   string? media_source;
   // The ID of sink that is rendering the media content.
@@ skipping 350 matching lines @@
 
   // Called when the a batch of messages arrives from the media sink for the
   // route given by |route_id|.
   // |StartListeningForRouteMessages| must be called first in order to receive
   // messages.
   // |route_id|: ID of route of the messages.
   // |messages|: A non-empty list of messages received.
   OnRouteMessagesReceived(string route_id,
                           array<RouteMessage> messages);
 };