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

Unified Diff: media/base/mime_util_internal.h

Issue 2742113002: Gate support for certain EOTFs/primaries/matrices on color management / hdr flags (Closed)
Patch Set: missed updating some tests 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/media_client.h ('k') | media/base/mime_util_internal.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/base/mime_util_internal.h
diff --git a/media/base/mime_util_internal.h b/media/base/mime_util_internal.h
index 1834b304af6490f03c753124ae9c13a6a43eed3a..d39a2d855b08444f78fd4d20bca8b2ff6e8f1bca 100644
--- a/media/base/mime_util_internal.h
+++ b/media/base/mime_util_internal.h
@@ -16,6 +16,9 @@
#include "media/base/video_codecs.h"
namespace media {
+
+class VideoColorSpace;
+
namespace internal {
// Internal utility class for handling mime types. Should only be invoked by
@@ -114,16 +117,15 @@ 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.
+ // |out_color| is the color space described by the
+ // |codec_id|.
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,
- gfx::ColorSpace::TransferID* out_eotf) const;
+ VideoColorSpace* out_colorspace) const;
// Returns IsSupported if |codec| when platform supports codec contained in
// |mime_type_lower_case|. Returns MayBeSupported when platform support is
@@ -137,7 +139,7 @@ class MEDIA_EXPORT MimeUtil {
Codec codec,
VideoCodecProfile video_profile,
uint8_t video_level,
- gfx::ColorSpace::TransferID eotf,
+ const VideoColorSpace& eotf,
bool is_encrypted) const;
// Wrapper around IsCodecSupported for simple codecs that are entirely
« no previous file with comments | « media/base/media_client.h ('k') | media/base/mime_util_internal.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698