fix(replay): Reconfig on single-dim resizes and recycle SurfaceView bitmap on throw

Two review-bot findings:

1. determineWindowSize used && to compare new vs last-known dimensions, so
   single-dimension resizes (split-screen drag, partial multi-window
   adjustments, foldable transitions where only one dim shifts) were
   silently dropped — onWindowSizeChanged only fired when both width AND
   height differed. The new layout listener already detects single-dim
   changes with ||, but then delegated to a function that AND'd them away.
   Switch the existing checks to || so any size delta reconfigs the
   recorder, matching the listener's intent.

2. In captureSurfaceViews, if PixelCopy.request or getLocationOnScreen
   threw after svBitmap was allocated, the catch path logged the error
   but never recycled the bitmap, leaking it until GC. Track the bitmap
   in a nullable local that the catch block recycles, and clear it after
   PixelCopy.request returns successfully so ownership transfers to the
   async callback without double-recycling.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: romtsn

sentry-android-replay/src/main/java/io/sentry/android/replay/WindowRecorder.kt

@@ -181,7 +181,7 @@ internal class WindowRecorder(
 
   fun determineWindowSize(root: View) {
     if (root.hasSize()) {
-      if (root.width != lastKnownWindowSize.x && root.height != lastKnownWindowSize.y) {
+      if (root.width != lastKnownWindowSize.x || root.height != lastKnownWindowSize.y) {
         lastKnownWindowSize.set(root.width, root.height)
         windowCallback.onWindowSizeChanged(root.width, root.height)
       }
@@ -197,7 +197,7 @@ internal class WindowRecorder(
             }
             if (root.hasSize()) {
               root.removeOnPreDrawListenerSafe(this)
-              if (root.width != lastKnownWindowSize.x && root.height != lastKnownWindowSize.y) {
+              if (root.width != lastKnownWindowSize.x || root.height != lastKnownWindowSize.y) {
                 lastKnownWindowSize.set(root.width, root.height)
                 windowCallback.onWindowSizeChanged(root.width, root.height)
               }

sentry-android-replay/src/main/java/io/sentry/android/replay/screenshot/PixelCopyStrategy.kt

@@ -181,37 +181,43 @@ internal class PixelCopyStrategy(
         continue
       }
 
+      var svBitmap: Bitmap? = null
       try {
-        val svBitmap =
+        svBitmap =
           Bitmap.createBitmap(surfaceView.width, surfaceView.height, Bitmap.Config.ARGB_8888)
+        val bitmapToCapture = svBitmap
 
         surfaceView.getLocationOnScreen(svLocation)
         val capturedX = svLocation[0]
         val capturedY = svLocation[1]
 
         PixelCopy.request(
           surfaceView,
-          svBitmap,
+          bitmapToCapture,
           { copyResult: Int ->
             if (isClosed.get()) {
-              svBitmap.recycle()
+              bitmapToCapture.recycle()
               // still drive the completion latch so any prior captures get recycled by the
               // composite step's early-return path.
               onCaptureComplete()
               return@request
             }
             if (copyResult == PixelCopy.SUCCESS) {
-              captures[index] = SurfaceViewCapture(svBitmap, capturedX, capturedY)
+              captures[index] = SurfaceViewCapture(bitmapToCapture, capturedX, capturedY)
             } else {
-              svBitmap.recycle()
+              bitmapToCapture.recycle()
               options.logger.log(INFO, "Failed to capture SurfaceView: %d", copyResult)
             }
             onCaptureComplete()
           },
           mainLooperHandler.handler,
         )
+        // Ownership transferred to the PixelCopy callback — clear local so catch doesn't
+        // double-recycle if the recycle paths above already ran.
+        svBitmap = null
       } catch (e: Throwable) {
         options.logger.log(WARNING, "Failed to capture SurfaceView", e)
+        svBitmap?.recycle()
         onCaptureComplete()
       }
     }