Audit/fix use of media::VideoFrame::coded_size()

content/common/gpu/media/android_video_encode_accelerator.cc

Index: content/common/gpu/media/android_video_encode_accelerator.cc
diff --git a/content/common/gpu/media/android_video_encode_accelerator.cc b/content/common/gpu/media/android_video_encode_accelerator.cc
index 05f18b6b12513c318a1753885b416710447f41ca..e474137871631b064b67a10700fbff688d54c1c2 100644
--- a/content/common/gpu/media/android_video_encode_accelerator.cc
+++ b/content/common/gpu/media/android_video_encode_accelerator.cc
@@ -202,15 +202,16 @@ void AndroidVideoEncodeAccelerator::Encode(
 
   // MediaCodec doesn't have a way to specify stride for non-Packed formats, so
   // we insist on being called with packed frames and no cropping :(
-  RETURN_ON_FAILURE(frame->row_bytes(VideoFrame::kYPlane) ==
-                            frame->stride(VideoFrame::kYPlane) &&
-                        frame->row_bytes(VideoFrame::kUPlane) ==
-                            frame->stride(VideoFrame::kUPlane) &&
-                        frame->row_bytes(VideoFrame::kVPlane) ==
-                            frame->stride(VideoFrame::kVPlane) &&
-                        gfx::Rect(frame->coded_size()) == frame->visible_rect(),
-                    "Non-packed frame, or visible rect != coded size",
-                    kInvalidArgumentError);
+  RETURN_ON_FAILURE(
+      frame->row_bytes(VideoFrame::kYPlane) ==
+              frame->stride(VideoFrame::kYPlane) &&
+          frame->row_bytes(VideoFrame::kUPlane) ==
+              frame->stride(VideoFrame::kUPlane) &&
+          frame->row_bytes(VideoFrame::kVPlane) ==
+              frame->stride(VideoFrame::kVPlane) &&
+          frame->coded_size().width() == frame->visible_rect().width(),

[Ami GONE FROM CHROMIUM, 2014/02/28 00:48:13]

I still think this is wrong because there's no way to communicate visible_rect
to MediaCodec.  If there are extra coded rows (for example), they'll end up
visible in the encoded result.
Ditto for left margins that are excluded from a visible_rect that is
right-aligned with coded_size; they'll end up visible too.

+      "Non-packed frame, or visible_rect/coded_size mismatch",
+      kInvalidArgumentError);
 
   pending_frames_.push(MakeTuple(frame, force_keyframe, base::Time::Now()));
   DoIOTask();