Implement Mojo IPC for media::HDRMetadata

media/mojo/interfaces/media_types.mojom

Index: media/mojo/interfaces/media_types.mojom
diff --git a/media/mojo/interfaces/media_types.mojom b/media/mojo/interfaces/media_types.mojom
index ae20147a8916685914fde8a4c8c376756f3d1262..bb58218c570002d94d6ffae252be20e97a7aa5ff 100644
--- a/media/mojo/interfaces/media_types.mojom
+++ b/media/mojo/interfaces/media_types.mojom
@@ -86,6 +86,27 @@ struct VideoColorSpace {
   RangeID range;
 };
 
+// This defines a mojo transport format for media::HDRMetadata.
+// See media/base/hdr_metadata.h for description.
+struct MasteringMetadata {
+  float primary_r_chromaticity_x;
+  float primary_r_chromaticity_y;
+  float primary_g_chromaticity_x;
+  float primary_g_chromaticity_y;
+  float primary_b_chromaticity_x;
+  float primary_b_chromaticity_y;
+  float white_point_chromaticity_x;
+  float white_point_chromaticity_y;
+  float luminance_max;
+  float luminance_min;
+};
+
+struct HDRMetadata {
+  MasteringMetadata mastering_metadata;
+  uint32 max_cll;

[xhwang, 2017/04/07 16:24:47]

+hubbe:

I have no idea what "cll" is and what "fall" means. Could you please add some
reference in media/base/hdr_metadata.h?

[servolk, 2017/04/07 16:30:37]

Yeah, sorry about that, it's HDR jargon. I've added a couple of comments to
explain those values in hdr_metadata.h.

[hubbe, 2017/04/07 17:24:22]

Renaming the variables would be better.

+  uint32 max_fall;
+};
+
 // This defines a mojo transport format for media::AudioDecoderConfig.
 // See media/base/audio_decoder_config.h for descriptions.
 struct AudioDecoderConfig {
@@ -112,6 +133,7 @@ struct VideoDecoderConfig {
   array<uint8> extra_data;
   EncryptionScheme encryption_scheme;
   VideoColorSpace color_space_info;
+  HDRMetadata? hdr_metadata;
 };
 
 // Native struct media::SubsampleEntry;