Change the video color space default.

media/base/video_decoder_config.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef MEDIA_BASE_VIDEO_DECODER_CONFIG_H_
 #define MEDIA_BASE_VIDEO_DECODER_CONFIG_H_
 
 #include <string>
 #include <vector>
 
@@ skipping 51 matching lines @@
   VP8PROFILE_ANY = VP8PROFILE_MIN,
   VP8PROFILE_MAX = VP8PROFILE_ANY,
   VP9PROFILE_MIN = 12,
   VP9PROFILE_ANY = VP9PROFILE_MIN,
   VP9PROFILE_MAX = VP9PROFILE_ANY,
   VIDEO_CODEC_PROFILE_MAX = VP9PROFILE_MAX,
 };
 
 class MEDIA_EXPORT VideoDecoderConfig {
  public:
+  // Get the color space defaults. The default for src playbacks depends on
+  // whether the video is SD or HD.
+  static VideoFrame::ColorSpace DefaultMseColorSpace();

[DaleCurtis, 2015/07/06 22:36:14]

How about just DefaultColorSpace() and MSE can hard code somewhere in MSE code?
We don't want to tie this class to a MSE/src name.

Actually SrcColorSpace is only used in one place, why not hard code it there
instead?

[watk, 2015/07/06 23:15:58]

Done.

+  static VideoFrame::ColorSpace DefaultSrcColorSpace(
+      const gfx::Size& natural_size);
+
   // Constructs an uninitialized object. Clients should call Initialize() with
   // appropriate values before using.
   VideoDecoderConfig();
 
   // Constructs an initialized object. It is acceptable to pass in NULL for
   // |extra_data|, otherwise the memory is copied.
   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,
                      const uint8* extra_data, size_t extra_data_size,
                      bool is_encrypted);
 
   ~VideoDecoderConfig();
 
   // Resets the internal state of this object.
   void Initialize(VideoCodec codec,
@@ skipping 20 matching lines @@
   std::string AsHumanReadableString() const;
 
   std::string GetHumanReadableCodecName() const;
 
   VideoCodec codec() const;
   VideoCodecProfile profile() const;
 
   // Video format used to determine YUV buffer sizes.
   VideoFrame::Format format() const;
 
+  // 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;
+
   // Width and height of video frame immediately post-decode. Not all pixels
   // in this region are valid.
   gfx::Size coded_size() const;
 
   // Region of |coded_size_| that is visible.
   gfx::Rect visible_rect() const;
 
   // Final visible width and height of a video frame with aspect ratio taken
   // into account.
   gfx::Size natural_size() const;
 
   // 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;
 
   // 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;
 
  private:
   VideoCodec codec_;
   VideoCodecProfile profile_;
 
   VideoFrame::Format format_;
+  VideoFrame::ColorSpace color_space_;
 
   gfx::Size coded_size_;
   gfx::Rect visible_rect_;
   gfx::Size natural_size_;
 
   std::vector<uint8> extra_data_;
 
   bool is_encrypted_;
 
   // Not using DISALLOW_COPY_AND_ASSIGN here intentionally to allow the compiler
   // generated copy constructor and assignment operator. Since the extra data is
   // typically small, the performance impact is minimal.
 };
 
 }  // namespace media
 
 #endif  // MEDIA_BASE_VIDEO_DECODER_CONFIG_H_