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

Unified Diff: media/base/mime_util_internal.cc

Issue 2746013006: use VideoColorSpace in decoder configuration (Closed)
Patch Set: oops, win compile fix reapplied Created 3 years, 9 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
« no previous file with comments | « media/base/ipc/media_param_traits_macros.h ('k') | media/base/video_codecs.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/base/mime_util_internal.cc
diff --git a/media/base/mime_util_internal.cc b/media/base/mime_util_internal.cc
index f0e45d46cb1eb365a4750a75bd4a06dac300b0e0..d2ff7a150e3c55a4030083472a11915307fb34c8 100644
--- a/media/base/mime_util_internal.cc
+++ b/media/base/mime_util_internal.cc
@@ -581,7 +581,7 @@ bool MimeUtil::ParseCodecString(const std::string& mime_type_lower_case,
// Most codec strings do not yet specify color. We choose 709 as default color
// space elsewhere, so defaulting to 709 here as well. See here for context:
// https://crrev.com/1221903003/
- *out_color_space = VideoColorSpace::BT709();
+ *out_color_space = VideoColorSpace::REC709();
std::map<std::string, Codec>::const_iterator itr =
GetStringToCodecMap().find(codec_id);
@@ -650,7 +650,7 @@ SupportsType MimeUtil::IsSimpleCodecSupported(
SupportsType result = IsCodecSupported(
mime_type_lower_case, codec, VIDEO_CODEC_PROFILE_UNKNOWN,
- 0 /* video_level */, VideoColorSpace::BT709(), is_encrypted);
+ 0 /* video_level */, VideoColorSpace::REC709(), is_encrypted);
// Platform support should never be ambiguous for simple codecs (no range of
// profiles to consider).
« no previous file with comments | « media/base/ipc/media_param_traits_macros.h ('k') | media/base/video_codecs.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698