Add dedicated PipelineStatus UMAs for encrypted playback

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 f46a8b87c2a38de7735ce4a4eeb88030552d6451..94ece7ac2a6fbda0bab110a352027053513a4d17 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -58414,14 +58414,45 @@ To add a new entry, add it with any value and run test to compute valid value.
       label="PipelineStatus for the codecs that dont have an explicit metric."/>
   <suffix name="AudioVideo.VP8.SW"
       label="PipelineStatus for AV streams with VP8 Software decoder."/>

[xhwang, 2014/12/04 21:15:22]

here and below s/Soft/soft

[prabhur1, 2014/12/05 01:29:22]

Done.

+  <suffix name="AudioVideo.VP8.HW"

[xhwang, 2014/12/04 21:15:22]

I chatted with ddorwin on this. For video decoders, we already have SW, HW and
DVD. But we only have four real VideoDecoders in total: DVD, GVD, FFmpegVD and
VpxVD. Since we already have the name for each, we might just use decoder names
in the UMA. For example, instead of VP8.HW, we have VP8.GVD; VP8.SW will be
split to VP8.FFmpegVideoDecoder and VP8.VpxVideoDecoder, etc.

Some combinations are invalid and we don't need to list it in this file, e.g.
H264.VpxVideoDecoder.

You don't have to do this now, but we should consider this when we work on the
long term plan to improve/clean-up these UMAs.

[prabhur1, 2014/12/05 01:29:22]

Acknowledged.

+      label="PipelineStatus for AV streams with VP8 Hardware decoder."/>

[xhwang, 2014/12/04 21:15:22]

here and below s/Hard/hard

[prabhur1, 2014/12/05 01:29:22]

Done.

+  <suffix name="AudioVideo.VP8.DDS.SW"
+      label="PipelineStatus for AV streams with VP8 software codec and that
+             go through the DecryptingDemuxerStream (DDS)."/>
+  <suffix name="AudioVideo.VP8.DDS.HW"
+      label="PipelineStatus for AV streams with VP8 hardware codec and that
+             go through the DecryptingDemuxerStream (DDS)."/>
+  <suffix name="AudioVideo.VP8.DVD"
+      label="PipelineStatus for AV streams with VP8 codec and that go through
+             the DecryptingVideoDecoder (DVD)."/>
   <suffix name="AudioVideo.VP9.SW"
       label="PipelineStatus for AV streams with VP9 Software decoder."/>
+  <suffix name="AudioVideo.VP9.HW"
+      label="PipelineStatus for AV streams with VP9 Hardware decoder."/>
+  <suffix name="AudioVideo.VP9.DDS.SW"
+      label="PipelineStatus for AV streams with VP9 codec and that go through
+             the DecryptingDemuxerStream (DDS)."/>
+  <suffix name="AudioVideo.VP9.DDS.HW"
+      label="PipelineStatus for AV streams with VP9 hardware codec and that
+             go through the DecryptingDemuxerStream (DDS)."/>
+  <suffix name="AudioVideo.VP9.DVD"
+      label="PipelineStatus for AV streams with VP9 software codec and that
+             go through the DecryptingVideoDecoder (DVD)."/>
   <suffix name="AudioVideo.H264.HW"
       label="PipelineStatus for hardware decoded AV streams with H264
              decoder."/>
   <suffix name="AudioVideo.H264.SW"
       label="PipelineStatus for software decoded AV streams with H264
              decoder."/>

[xhwang, 2014/12/04 21:15:22]

nit: It seems we have SW before HW in all other places. Reverse the order here
as well?

[prabhur1, 2014/12/05 01:29:22]

Done.

+  <suffix name="AudioVideo.H264.DDS.SW"
+      label="PipelineStatus for AV streams with H264 software decoder and
+             that go through the DecryptingDemuxerStream (DDS)."/>
+  <suffix name="AudioVideo.H264.DDS.HW"
+      label="PipelineStatus for AV streams with H264 hardware decoder and
+             that go through the DecryptingDemuxerStream (DDS)."/>
+  <suffix name="AudioVideo.H264.DVD"
+      label="PipelineStatus for AV streams with H264 decoder and that go
+             through the DecryptingVideoDecoder (DVD)."/>
   <suffix name="AudioOnly" label="PipelineStatus for Audio-only streams."/>
   <suffix name="VideoOnly" label="PipelineStatus for Video-only streams."/>
   <suffix name="Unsupported" label="PipelineStatus for unsupported streams."/>