media/gpu/v4l2VEA: Invoke error if the requested visible_rect is adjusted by driver

V4L2 encoder driver may need to adjust visible rectangle because of
their restrictions of hardware and firmware. If the adjusted visible
rectangle is different from the requested one, we should invoke the
error rather than processing the encoding.

Bug: 1033799
Test: video.EncodeAccel.* on scarlet
Change-Id: I686f3dc30ee325bb6020ffc87534971291602020
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2078522Reviewed-by: Dan Sanders <sandersd@chromium.org>
Reviewed-by: Alexandre Courbot <acourbot@chromium.org>
Commit-Queue: Hirokazu Honda <hiroh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#748520}

media/gpu/v4l2/v4l2_video_encode_accelerator.cc

@@ -1430,11 +1430,16 @@ bool V4L2VideoEncodeAccelerator::ApplyCrop() {
     visible_rect = crop.c;
   }
 
-  encoder_input_visible_rect_ =
-      gfx::Rect(visible_rect.left, visible_rect.top, visible_rect.width,
-                visible_rect.height);
-  VLOGF(2) << "After adjusted by driver, encoder_input_visible_rect_="
-           << encoder_input_visible_rect_.ToString();
+  const gfx::Rect adjusted_visible_rect(visible_rect.left, visible_rect.top,
+                                        visible_rect.width,
+                                        visible_rect.height);
+  if (encoder_input_visible_rect_ != adjusted_visible_rect) {
+    VLOGF(1) << "Unsupported visible rectangle: "
+             << encoder_input_visible_rect_.ToString()
+             << ", the rectangle adjusted by the driver: "
+             << adjusted_visible_rect.ToString();
+    return false;
+  }
   return true;
 }