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 448 matching lines @@
 
   VideoFrame::Format codec_format = VideoFrame::YV12;
   int uv_rows = (vpx_image->d_h + 1) / 2;
 
   if (vpx_image->fmt == VPX_IMG_FMT_I444) {
     CHECK(!vpx_codec_alpha_);
     codec_format = VideoFrame::YV24;
     uv_rows = vpx_image->d_h;
   } else if (vpx_codec_alpha_) {
     codec_format = VideoFrame::YV12A;
+  } else if (vpx_image->cs == VPX_CS_BT_709) {
+    // TODO(watk): A limitation of conflating color space with pixel format is
+    // that it's not possible to have BT709 with alpha.
+    // Until color space is separated from format, prefer YV12A over YV12HD.

[xhwang, 2015/03/04 18:07:14]

This seems more like a comment/todo for l.468. Move it up a bit?

[watk, 2015/03/04 18:23:02]

Done.

+    codec_format = VideoFrame::YV12HD;
   }
 
   gfx::Size size(vpx_image->d_w, vpx_image->d_h);
 
   if (!vpx_codec_alpha_ && memory_pool_.get()) {
     *video_frame = VideoFrame::WrapExternalYuvData(
         codec_format,
         size, gfx::Rect(size), config_.natural_size(),
         vpx_image->stride[VPX_PLANE_Y],
         vpx_image->stride[VPX_PLANE_U],
@@ skipping 32 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