Fix bug in ViewCapture computing Rect for PixelCopy

When SDK > 34 the PixelCopy method was not treating the Rect as documented, instead of applying it as a Rect local to the coordinates of the View it was being applied as a Rect relative to the window.

PiperOrigin-RevId: 634008870

Author: copybara-androidxtest

core/CHANGELOG.md

@@ -7,6 +7,7 @@
 **Bug Fixes**
 
 * Remove unused androidx.test.annotation dependency
+* Fix `Rect` handling in `ViewCapture` for SDK >= 34 for non root views.
 
 **New Features**
 

core/java/androidx/test/core/view/ViewCapture.kt

@@ -217,13 +217,7 @@ private suspend fun View.generateBitmapFromPixelCopy(
   return suspendCancellableCoroutine<Bitmap> { cont ->
     var bounds = getBoundsInSurface()
     if (rect != null) {
-      bounds =
-        Rect(
-          bounds.left + rect.left,
-          bounds.top + rect.top,
-          bounds.left + rect.right,
-          bounds.top + rect.bottom,
-        )
+      bounds = Rect(rect).apply { offset(bounds.left, bounds.top) }
     }
     val onCopyFinished =
       PixelCopy.OnPixelCopyFinishedListener { result ->
@@ -302,10 +296,17 @@ private suspend fun View.generateBitmapFromPixelCopy(
   destBitmap: Bitmap,
   rect: Rect? = null,
 ): Bitmap {
+  val boundsInWindow = getBoundsInWindow()
+  val sourceRect =
+    if (rect != null) {
+      Rect(rect).apply { offset(boundsInWindow.left, boundsInWindow.top) }
+    } else {
+      boundsInWindow
+    }
   return suspendCancellableCoroutine<Bitmap> { cont ->
     val request =
       PixelCopy.Request.Builder.ofWindow(this)
-        .setSourceRect(rect ?: getBoundsInWindow())
+        .setSourceRect(sourceRect)
         .setDestinationBitmap(destBitmap)
         .build()
     val onCopyFinished =