Add UMA metrics for VP9 in iso-bmff |
||||
Issue descriptionI am planning to add some metrics to track "vp9 in iso-bmff" playback. Here is what I have in mind: MSE: Media.MSE.VideoCodec.VP9.WebM Media.MSE.VideoCodec.VP9.MP4 Non-MSE: Media.VideoCodec.VP9.WebM Media.VideoCodec.VP9.MP4 Any other thoughts, suggestions?
,
Apr 25 2017
,
Apr 25 2017
sgtm Would this be purely for VP9 container tracking, or across all audio and video codecs?
,
Apr 25 2017
I am not sure if it is useful for other codecs. What do you think? I can extend it to support all codecs.
,
Apr 25 2017
,
Apr 25 2017
Currently we already have Media.VideoCodec UMA, which uses enums like kCodecH264 kCodecMPEG4 kCodecTheora kCodecVP8 kCodecVP9 It'll be nice if the new ones are compatible with it. How about we have something like: Media.VideoCodec.MSE.MP4 Media.VideoCodec.MSE.WebM Media.VideoCodec.SRC.MP4 Media.VideoCodec.SRC.WebM all of which will use the same codec enum as listed above. Then for your purpose, you basically need to check the kCodecVP9 bucket in Media.VideoCodec.MSE.MP4 and Media.VideoCodec.SRC.MP4.
,
Apr 27 2017
Re #6, I slightly prefer using container type as bucket type, but sure, I am fine with this too. Updated CL: https://codereview.chromium.org/2846693002/.
,
May 4 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/02b14ba085a0bb98b8d87cd713fb13b9c6d2821a commit 02b14ba085a0bb98b8d87cd713fb13b9c6d2821a Author: kqyang <kqyang@chromium.org> Date: Thu May 04 21:05:54 2017 Add UMA metrics for Media.{SRC,MSE}.VideoCodec.{MP4,WebM} BUG= 715161 TEST=Manually tested with SRC/MSE playback (then checked about:://histograms) Review-Url: https://codereview.chromium.org/2846693002 Cr-Commit-Position: refs/heads/master@{#469469} [modify] https://crrev.com/02b14ba085a0bb98b8d87cd713fb13b9c6d2821a/media/filters/ffmpeg_demuxer.cc [modify] https://crrev.com/02b14ba085a0bb98b8d87cd713fb13b9c6d2821a/media/filters/ffmpeg_glue.cc [modify] https://crrev.com/02b14ba085a0bb98b8d87cd713fb13b9c6d2821a/media/filters/ffmpeg_glue.h [modify] https://crrev.com/02b14ba085a0bb98b8d87cd713fb13b9c6d2821a/media/filters/stream_parser_factory.cc [modify] https://crrev.com/02b14ba085a0bb98b8d87cd713fb13b9c6d2821a/tools/metrics/histograms/histograms.xml
,
May 4 2017
|
||||
►
Sign in to add a comment |
||||
Comment 1 by kqyang@chromium.org
, Apr 25 2017