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

Unified Diff: media/base/video_decoder_config.h

Issue 1221903003: Change the video color space default. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 5 years, 5 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/test_helpers.cc ('k') | media/base/video_decoder_config.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/base/video_decoder_config.h
diff --git a/media/base/video_decoder_config.h b/media/base/video_decoder_config.h
index 00de84e2fb56bc02e500b34d7b467806a0b1883e..d586a7824f4aac7c1c546098decfbc710cee72bc 100644
--- a/media/base/video_decoder_config.h
+++ b/media/base/video_decoder_config.h
@@ -78,6 +78,7 @@ class MEDIA_EXPORT VideoDecoderConfig {
VideoDecoderConfig(VideoCodec codec,
VideoCodecProfile profile,
VideoFrame::Format format,
+ VideoFrame::ColorSpace color_space,
const gfx::Size& coded_size,
const gfx::Rect& visible_rect,
const gfx::Size& natural_size,
@@ -112,38 +113,44 @@ class MEDIA_EXPORT VideoDecoderConfig {
std::string GetHumanReadableCodecName() const;
- VideoCodec codec() const;
- VideoCodecProfile profile() const;
+ VideoCodec codec() const { return codec_; }
+ VideoCodecProfile profile() const { return profile_; }
// Video format used to determine YUV buffer sizes.
- VideoFrame::Format format() const;
+ VideoFrame::Format format() const { return format_; }
+
+ // The default color space of the decoded frames. Decoders should output
+ // frames tagged with this color space unless they find a different value in
+ // the bitstream.
+ VideoFrame::ColorSpace color_space() const { return color_space_; }
// Width and height of video frame immediately post-decode. Not all pixels
// in this region are valid.
- gfx::Size coded_size() const;
+ gfx::Size coded_size() const { return coded_size_; }
// Region of |coded_size_| that is visible.
- gfx::Rect visible_rect() const;
+ gfx::Rect visible_rect() const { return visible_rect_; }
// Final visible width and height of a video frame with aspect ratio taken
// into account.
- gfx::Size natural_size() const;
+ gfx::Size natural_size() const { return natural_size_; }
// Optional byte data required to initialize video decoders, such as H.264
// AAVC data.
const uint8* extra_data() const;
- size_t extra_data_size() const;
+ size_t extra_data_size() const { return extra_data_.size(); }
// Whether the video stream is potentially encrypted.
// Note that in a potentially encrypted video stream, individual buffers
// can be encrypted or not encrypted.
- bool is_encrypted() const;
+ bool is_encrypted() const { return is_encrypted_; }
private:
VideoCodec codec_;
VideoCodecProfile profile_;
VideoFrame::Format format_;
+ VideoFrame::ColorSpace color_space_;
gfx::Size coded_size_;
gfx::Rect visible_rect_;
« no previous file with comments | « media/base/test_helpers.cc ('k') | media/base/video_decoder_config.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698