Chromium Code Reviews| Index: media/base/mime_util_internal.h |
| diff --git a/media/base/mime_util_internal.h b/media/base/mime_util_internal.h |
| index b0f724e1d0b9e4d535e532ef4ad236da95954bac..704acbb2b078020c0e249b4da3e06fa55c3fba3a 100644 |
| --- a/media/base/mime_util_internal.h |
| +++ b/media/base/mime_util_internal.h |
| @@ -113,12 +113,16 @@ class MEDIA_EXPORT MimeUtil { |
| // whenever |codec_id| is incomplete/invalid, or in some cases when |
| // |ambiguous_codec_string| is set to true. |
| // |is_encrypted| means the codec will be used with encrypted blocks. |
| + // |out_eotf| is the electro-optical transfer function described by the |
| + // |codec_id|. Will be gfx::ColorSpace::TransferID::INVALID for many codec |
| + // strings where EOTF is not described. |
|
servolk
2017/03/02 18:56:36
Yes, VP9 is the only codec that will have eotf in
chcunningham
2017/03/02 23:13:08
Ack. This comment is actually stale. I'm planning
|
| bool ParseCodecString(const std::string& mime_type_lower_case, |
| const std::string& codec_id, |
| Codec* codec, |
| bool* ambiguous_codec_string, |
| VideoCodecProfile* out_profile, |
| - uint8_t* out_level) const; |
| + uint8_t* out_level, |
| + gfx::ColorSpace::TransferID* out_eotf) const; |
| // Returns IsSupported if |codec| when platform supports codec contained in |
| // |mime_type_lower_case|. Returns MayBeSupported when platform support is |
| @@ -132,6 +136,7 @@ class MEDIA_EXPORT MimeUtil { |
| Codec codec, |
| VideoCodecProfile video_profile, |
| uint8_t video_level, |
| + gfx::ColorSpace::TransferID eotf, |
| bool is_encrypted) const; |
| // Wrapper around IsCodecSupported for simple codecs that are entirely |