Support VP9 video in the BT709 color space.

media/filters/vpx_video_decoder.cc

 // 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.
 
 #include "media/filters/vpx_video_decoder.h"
 
 #include <algorithm>
 #include <string>
 #include <vector>
 
@@ skipping 442 matching lines @@
                                      const struct vpx_image* vpx_image_alpha,
                                      scoped_refptr<VideoFrame>* video_frame) {
   CHECK(vpx_image);
   CHECK(vpx_image->fmt == VPX_IMG_FMT_I420 ||
         vpx_image->fmt == VPX_IMG_FMT_YV12 ||
         vpx_image->fmt == VPX_IMG_FMT_I444);
 
   VideoFrame::Format codec_format = VideoFrame::YV12;
   int uv_rows = (vpx_image->d_h + 1) / 2;
 
-  if (vpx_image->fmt == VPX_IMG_FMT_I444) {
+  if ((vpx_image->fmt == VPX_IMG_FMT_YV12 ||
+       vpx_image->fmt == VPX_IMG_FMT_I420) &&
+      vpx_image->cs == VPX_CS_BT_709) {

[xhwang, 2015/03/03 19:53:30]

Can vpx_codec_alpha_ be true here?

[watk, 2015/03/03 22:01:28]

Yeah, it can. Unfortunately we have to choose between alpha and BT709, but I
made the wrong choice! Alpha should be preferred.

+    codec_format = VideoFrame::YV12HD;
+  } else if (vpx_image->fmt == VPX_IMG_FMT_I444) {

[xhwang, 2015/03/03 19:53:30]

Can vpx_image->cs == VPX_CS_BT_709 here?

[watk, 2015/03/03 22:01:28]

It can, but we can't support it at the moment :( I added a comment

     CHECK(!vpx_codec_alpha_);
     codec_format = VideoFrame::YV24;
     uv_rows = vpx_image->d_h;
   } else if (vpx_codec_alpha_) {
     codec_format = VideoFrame::YV12A;
   }
 
   gfx::Size size(vpx_image->d_w, vpx_image->d_h);
 
   if (!vpx_codec_alpha_ && memory_pool_.get()) {
@@ skipping 37 matching lines @@
         vpx_image->stride[VPX_PLANE_Y], vpx_image->d_h, video_frame->get());
     return;
   }
   CopyAPlane(vpx_image_alpha->planes[VPX_PLANE_Y],
              vpx_image_alpha->stride[VPX_PLANE_Y],
              vpx_image_alpha->d_h,
              video_frame->get());
 }
 
 }  // namespace media