Add UMA metrics for ffmpeg color ranges.

tools/metrics/histograms/histograms.xml

Index: tools/metrics/histograms/histograms.xml
diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml
index 6a1961d47885f08227f44d481fd0415d107e8a0c..b7dbea3e2f0aae1e7c0cfdb77dc0f468ac8510f9 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -6241,6 +6241,14 @@ other types of suffix sets.
   <summary>Coded width of HTML5 video.</summary>
 </histogram>
 
+<histogram name="Media.VideoColorRange" enum="FFmpegColorRanges">
+  <summary>Pixel format color range of HTML5 video.</summary>

[Mark P, 2013/10/31 10:33:10]

Here and below: Histogram descriptions should clearly state when the histogram
is emitted (video load? video playback? etc.)

[rileya (GONE FROM CHROMIUM), 2013/10/31 17:54:00]

Alright, added a note about this.

+</histogram>
+
+<histogram name="Media.VideoPixelFormat" enum="VideoPixelFormat">
+  <summary>Pixel format of HTML5 video.</summary>
+</histogram>
+
 <histogram name="Media.VideoVisibleAspectRatio">
   <summary>Visible aspect ratio of HTML5 video.</summary>
 </histogram>
@@ -22827,6 +22835,12 @@ other types of suffix sets.
   <int value="1950507339" label="TAK"/>
 </enum>
 
+<enum name="FFmpegColorRanges" type="int">
+  <int value="0" label="UNSPECIFIED"/>
+  <int value="1" label="MPEG"/>
+  <int value="2" label="JPEG"/>
+</enum>
+
 <enum name="FileDialogType" type="int">
   <int value="0" label="Select folder"/>
   <int value="1" label="Save as file"/>
@@ -28113,6 +28127,18 @@ other types of suffix sets.
   <int value="12" label="VP9"/>
 </enum>
 
+<enum name="VideoPixelFormat" type="int">
+  <int value="0" label="INVALID"/>
+  <int value="4" label="RGB32"/>
+  <int value="6" label="YV12"/>
+  <int value="7" label="YV16"/>
+  <int value="9" label="EMPTY"/>
+  <int value="11" label="I420"/>
+  <int value="12" label="NATIVE_TEXTURE"/>
+  <int value="13" label="HOLE"/>
+  <int value="14" label="YV12A"/>

[Mark P, 2013/10/31 10:33:10]

I see the underlying enum has gaps.  Is there a good reason for this?  Why not
change it to be gapless, otherwise we're allocating histogram buckets that
clearly never get used.

[rileya (GONE FROM CHROMIUM), 2013/10/31 17:54:00]

Looking at past revisions, it looks like the gaps came to be due to needing to
mirror an enum in WebKit (WebVideoFrame::Format), from the looks of it this
isn't necessary anymore, since that enum has since been removed in Blink... I'll
ask around and see if there's any reason not to remove the gaps.

[rileya (GONE FROM CHROMIUM), 2013/10/31 19:17:21]

Looks like it's safe to remove the gaps, the newest patch set removes them.

+</enum>
+
 <enum name="ViewFileType" type="int">
   <int value="0" label="other"/>
   <int value="1" label=".3ga"/>