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

Unified Diff: media/base/mime_util_internal.h

Issue 2723833002: WebM support for new multipart VP9 string. (Closed)
Patch Set: Use latest ordering (9 parts), Plumb eotf to media client Created 3 years, 10 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/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

Powered by Google App Engine
This is Rietveld 408576698