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

Unified Diff: media/filters/stream_parser_factory.cc

Issue 16830004: media: Remove VP9 flag, and enable VP9 support by default. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 6 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 2436ef6e37e1eebd327d315ff467ea03288a60f5..da8652212eb1f02ac3b24dfb02dae29b50395780 100644
--- a/media/filters/stream_parser_factory.cc
+++ b/media/filters/stream_parser_factory.cc
@@ -200,7 +200,6 @@ static bool VerifyCodec(
const CodecInfo* codec_info,
std::vector<CodecInfo::HistogramTag>* audio_codecs,
std::vector<CodecInfo::HistogramTag>* video_codecs) {
- const CommandLine* cmd_line = CommandLine::ForCurrentProcess();
jrummell 2013/06/12 22:05:53 This is also needed for EAC3 playback, if configur
Tom Finegan 2013/06/12 23:17:48 Fixed, though slightly reworked the checks because
switch (codec_info->type) {
case CodecInfo::AUDIO:
#if defined(ENABLE_EAC3_PLAYBACK)
@@ -213,12 +212,6 @@ static bool VerifyCodec(
audio_codecs->push_back(codec_info->tag);
return true;
case CodecInfo::VIDEO:
- // TODO(tomfinegan): Remove this check (or negate it, if we just
- // negate the flag) when VP9 is enabled by default.
- if (codec_info->tag == CodecInfo::HISTOGRAM_VP9 &&
- !cmd_line->HasSwitch(switches::kEnableVp9Playback)) {
- return false;
- }
if (video_codecs)
video_codecs->push_back(codec_info->tag);
return true;

Powered by Google App Engine
This is Rietveld 408576698