media: Handling YV12 odd height/width in vpx_video_decoder

media/filters/vpx_video_decoder.cc

Index: media/filters/vpx_video_decoder.cc
diff --git a/media/filters/vpx_video_decoder.cc b/media/filters/vpx_video_decoder.cc
index bd9a722e9a0423e92f59b94ca6ef763f8fcbd376..ee5fe00cfd7eeb21f4154b7dc3ae40918578012f 100644
--- a/media/filters/vpx_video_decoder.cc
+++ b/media/filters/vpx_video_decoder.cc
@@ -337,8 +337,6 @@ void VpxVideoDecoder::CopyVpxImageTo(const vpx_image* vpx_image,
                                      const struct vpx_image* vpx_image_alpha,
                                      scoped_refptr<VideoFrame>* video_frame) {
   CHECK(vpx_image);
-  CHECK_EQ(vpx_image->d_w % 2, 0U);
-  CHECK_EQ(vpx_image->d_h % 2, 0U);
   CHECK(vpx_image->fmt == VPX_IMG_FMT_I420 ||
         vpx_image->fmt == VPX_IMG_FMT_YV12);
 
@@ -357,11 +355,11 @@ void VpxVideoDecoder::CopyVpxImageTo(const vpx_image* vpx_image,
              video_frame->get());
   CopyUPlane(vpx_image->planes[VPX_PLANE_U],
              vpx_image->stride[VPX_PLANE_U],
-             vpx_image->d_h / 2,
+             (1 + vpx_image->d_h) / 2,

[scherkus (not reviewing), 2013/12/02 20:27:33]

nit: can we swap the order of the variable and the "+1"?

similar with conditions, we try to have variables go first in expressions for
the sake of readability

[vignesh, 2013/12/02 21:07:34]

Done.

              video_frame->get());
   CopyVPlane(vpx_image->planes[VPX_PLANE_V],
              vpx_image->stride[VPX_PLANE_V],
-             vpx_image->d_h / 2,
+             (1 + vpx_image->d_h) / 2,
              video_frame->get());
   if (!vpx_codec_alpha_)
     return;