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

Unified Diff: media/filters/stream_parser_factory.cc

Issue 1624703002: Implement support for vp9 in ISO-BMFF (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: media/filters/stream_parser_factory.cc
diff --git a/media/filters/stream_parser_factory.cc b/media/filters/stream_parser_factory.cc
index 87b84540bc758053eee8b927e84f07321cfd39b3..bdb76d8d614ad9e48cb30a4c0252839c719c8a9b 100644
--- a/media/filters/stream_parser_factory.cc
+++ b/media/filters/stream_parser_factory.cc
@@ -161,6 +161,10 @@ static const CodecInfo kHEVCHEV1CodecInfo = { "hev1.*", CodecInfo::VIDEO, NULL,
static const CodecInfo kHEVCHVC1CodecInfo = { "hvc1.*", CodecInfo::VIDEO, NULL,
CodecInfo::HISTOGRAM_HEVC };
#endif
+static const CodecInfo kMPEG4VP08CodecInfo = {"vp08.*", CodecInfo::VIDEO, NULL,
+ CodecInfo::HISTOGRAM_VP8};
ddorwin 2016/01/27 01:39:15 I wonder whether we want to have different histogr
ddorwin 2016/01/27 03:17:09 Are there other playback UMAs that we should updat
kqyang 2016/01/29 00:34:17 Sounds like a good idea to me. Done.
+static const CodecInfo kMPEG4VP09CodecInfo = {"vp09.*", CodecInfo::VIDEO, NULL,
+ CodecInfo::HISTOGRAM_VP9};
static const CodecInfo kMPEG4AACCodecInfo = { "mp4a.40.*", CodecInfo::AUDIO,
&ValidateMP4ACodecID,
CodecInfo::HISTOGRAM_MPEG4AAC };
@@ -193,6 +197,7 @@ static const CodecInfo* kVideoMP4Codecs[] = {
#if BUILDFLAG(ENABLE_HEVC_DEMUXING)
&kHEVCHEV1CodecInfo, &kHEVCHVC1CodecInfo,
#endif
+ &kMPEG4VP08CodecInfo, &kMPEG4VP09CodecInfo,
&kMPEG4AACCodecInfo, &kMPEG2AACLCCodecInfo, NULL};
static const CodecInfo* kAudioMP4Codecs[] = {&kMPEG4AACCodecInfo,

Powered by Google App Engine
This is Rietveld 408576698