Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(299)

Unified Diff: tools/metrics/histograms/histograms.xml

Issue 770423004: Add dedicated PipelineStatus UMAs for encrypted playback (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address CL comments Created 6 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
Download patch
« no previous file with comments | « media/filters/decrypting_video_decoder.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/metrics/histograms/histograms.xml
diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml
index f46a8b87c2a38de7735ce4a4eeb88030552d6451..108c985d882aa4537283a734833fcdf4dce445fa 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -58413,15 +58413,46 @@ To add a new entry, add it with any value and run test to compute valid value.
<suffix name="AudioVideo.Other"
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."/>
+ label="PipelineStatus for AV streams with VP8 software decoder."/>
+ <suffix name="AudioVideo.VP8.HW"
+ label="PipelineStatus for AV streams with VP8 hardware decoder."/>
+ <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.H264.HW"
- label="PipelineStatus for hardware decoded AV streams with H264
- decoder."/>
+ 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.SW"
label="PipelineStatus for software decoded AV streams with H264
decoder."/>
+ <suffix name="AudioVideo.H264.HW"
+ label="PipelineStatus for hardware decoded AV streams with H264
+ decoder."/>
+ <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."/>
« no previous file with comments | « media/filters/decrypting_video_decoder.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698