Add color metadata info to VideoDecoderConfig.

ui/gfx/color_space.h

Index: ui/gfx/color_space.h
diff --git a/ui/gfx/color_space.h b/ui/gfx/color_space.h
index ce02b0aa1b6eb20285884f4040aba5d0bb3701c1..83b84dedddfb7a5d028080cccb4b0259ae290f99 100644
--- a/ui/gfx/color_space.h
+++ b/ui/gfx/color_space.h
@@ -116,15 +116,24 @@ class GFX_EXPORT ColorSpace {
     LAST = UNKNOWN,
   };
 
-  // The h264 spec declares this as bool, so only the the first two values
-  // correspond to the h264 spec. Chrome-specific values can start at 2.
+  // The h264 spec declares this as bool, so only the the LIMITED/FULL values
+  // correspond to the h264 spec. Chrome-specific values can start at 3.

[hubbe, 2016/09/22 22:14:17]

I think this should read something like:

This corresponds to the webm range <link to doc or standard>
h264 only uses a bool, which corresponds to the LIMITED/FULL values.
Chrome-specific values start at 1000.

(Because none of these values are chrome-specific, as they were taken from the
webm standard, right?)

[servolk, 2016/09/22 22:34:39]

Done.

   // We use an enum instead of a bool becuase different bit depths may have
   // different definitions of what "limited" means.
   enum class RangeID : int8_t {
-    FULL = 0,
+    // Range is not explicitly specified / unknown.
+    UNSPECIFIED = 0,
+
+    // Limited Rec. 709 color range with RGB values ranging from 16 to 235.
     LIMITED = 1,
 
-    LAST = LIMITED
+    // Full RGB color range with RGB valees from 0 to 255.
+    FULL = 2,
+
+    // Range is defined by TransferID/MatrixID.
+    DERIVED = 3,
+
+    LAST = DERIVED
   };
 
   ColorSpace();