[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;
+  // This is currently only set by MediaRouter in OnSinksDiscovered().
+  MediaSinkExtraData? extra_data;
+};
+
+union MediaSinkExtraData {
+  DialMediaSink dial_media_sink;
+  CastMediaSink cast_media_sink;
+};
+
+struct DialMediaSink {
+  string ip_address;

[dcheng, 2017/03/03 10:37:10]

Can we add a typemap for net::IPAddress? We already have one for
net::IPEndpoint:
https://cs.chromium.org/chromium/src/net/interfaces/host_resolver_service.moj...

[zhaobin, 2017/03/03 23:56:42]

Do it in a seperate patch? Created crbug.com/698423.

[dcheng, 2017/03/04 01:49:16]

The reason I mention is because mojo struct trait getters are currently called
twice on serialization. So IPAddress::ToString() gets called twice, +
deserialization is more complicated.

It's true that this code is probably not perf critical, but where possible, we
should try to use best practices throughout the code.

[zhaobin, 2017/03/06 18:31:26]

Added IPAddress here. host_resolver_service.mojom has this warning:
"// WARNING! Do NOT use this mojom". Adding IPAddress here might not work. Any
suggestions?

[dcheng, 2017/03/06 23:02:05]

How about just creating a new mojom for the struct?

[zhaobin, 2017/03/07 19:28:49]

Is it ok to leave struct IPAddress in media_router.mojom in this patch? Do not
want to touch net/interface code in this patch.

[dcheng, 2017/03/07 19:54:32]

(Sorry just saw this comment)

Hmm... it's not great to separate the mojom struct from where it's actually
defined. I know it's nice not to have to touch //net, but it's better to just
have it in the right place to begin with.

+
+  // Model name of the sink, if it represents a physical device.
+  string model_name;
+
+  // 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;
+
+  // 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 is defined by the
+  // chrome.cast.channel API.
+  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);
 };