Increase YUV video clamping

cc/layers/video_layer_impl.cc

 // Copyright 2011 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 "cc/layers/video_layer_impl.h"
 
 #include "base/bind.h"
 #include "base/logging.h"
 #include "cc/layers/video_frame_provider_client_impl.h"
 #include "cc/quads/io_surface_draw_quad.h"
@@ skipping 211 matching lines @@
       DCHECK_GE(frame_resources_.size(), 3u);
       if (frame_resources_.size() < 3u)
         break;
       YUVVideoDrawQuad::ColorSpace color_space = YUVVideoDrawQuad::REC_601;
       if (frame_->format() == media::VideoFrame::YV12J) {
         color_space = YUVVideoDrawQuad::JPEG;
       } else if (frame_->format() == media::VideoFrame::YV12HD) {
         color_space = YUVVideoDrawQuad::REC_709;
       }
 
+      const gfx::Size ya_tex_size = coded_size;
+      const gfx::Size uv_tex_size = media::VideoFrame::PlaneSize(
+          frame_->format(), media::VideoFrame::kUPlane, coded_size);
+      DCHECK(uv_tex_size ==
+             media::VideoFrame::PlaneSize(
+                 frame_->format(), media::VideoFrame::kVPlane, coded_size));
+      if (frame_resources_.size() > 3) {
+        DCHECK(ya_tex_size ==

[enne (OOO), 2015/03/18 19:25:57]

DCHECK_EQ

[magjed_chromium, 2015/03/18 21:38:43]

Unfortunately it's not possible to print gfx::Size.

+               media::VideoFrame::PlaneSize(
+                   frame_->format(), media::VideoFrame::kAPlane, coded_size));
+      }
+
       gfx::RectF tex_coord_rect(
           tex_x_offset, tex_y_offset, tex_width_scale, tex_height_scale);
       YUVVideoDrawQuad* yuv_video_quad =
           render_pass->CreateAndAppendDrawQuad<YUVVideoDrawQuad>();
       yuv_video_quad->SetNew(
           shared_quad_state, quad_rect, opaque_rect, visible_quad_rect,
-          tex_coord_rect, coded_size, frame_resources_[0], frame_resources_[1],
-          frame_resources_[2],
+          tex_coord_rect, ya_tex_size, uv_tex_size, frame_resources_[0],
+          frame_resources_[1], frame_resources_[2],
           frame_resources_.size() > 3 ? frame_resources_[3] : 0, color_space);
       break;
     }
     case VideoFrameExternalResources::RGB_RESOURCE: {
       DCHECK_EQ(frame_resources_.size(), 1u);
       if (frame_resources_.size() < 1u)
         break;
       bool premultiplied_alpha = true;
       gfx::PointF uv_top_left(0.f, 0.f);
       gfx::PointF uv_bottom_right(tex_width_scale, tex_height_scale);
@@ skipping 117 matching lines @@
 void VideoLayerImpl::SetProviderClientImpl(
     scoped_refptr<VideoFrameProviderClientImpl> provider_client_impl) {
   provider_client_impl_ = provider_client_impl;
 }
 
 const char* VideoLayerImpl::LayerTypeAsString() const {
   return "cc::VideoLayerImpl";
 }
 
 }  // namespace cc