Revert "fix(replay): Fix IllegalArgumentException when Bitmap is initialized with non-positive values (#4536)"

romtsn · romtsn · commit ab1a12c9ec88 · 2025-07-09T11:06:24.000+02:00
This reverts commit 185b5dc.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -18,7 +18,6 @@
 - Correctly capture Dialogs and non full-sized windows ([#4354](https://github.com/getsentry/sentry-java/pull/4354))
 - Fix Session Replay masking for newer versions of Jetpack Compose (1.8+) ([#4485](https://github.com/getsentry/sentry-java/pull/4485))
 - Session Replay: Expand fix for crash on devices to all Unisoc/Spreadtrum chipsets ([#4510](https://github.com/getsentry/sentry-java/pull/4510))
-- Session Replay: Fix `IllegalArgumentException` when `Bitmap` is initialized with non-positive values ([#4536](https://github.com/getsentry/sentry-java/pull/4536))
 
 ## 7.22.5
 
diff --git a/sentry-android-replay/src/main/java/io/sentry/android/replay/util/Views.kt b/sentry-android-replay/src/main/java/io/sentry/android/replay/util/Views.kt
@@ -224,4 +224,6 @@ internal fun View?.removeOnPreDrawListenerSafe(listener: ViewTreeObserver.OnPreD
     }
 }
 
-internal fun View.hasSize(): Boolean = width > 0 && height > 0
+internal fun View.hasSize(): Boolean {
+    return width != 0 && height != 0
+}
diff --git a/sentry-android-replay/src/test/java/io/sentry/android/replay/util/ViewsTest.kt b/sentry-android-replay/src/test/java/io/sentry/android/replay/util/ViewsTest.kt

Original file line number	Diff line number	Diff line change
`@@ -224,4 +224,6 @@ internal fun View?.removeOnPreDrawListenerSafe(listener: ViewTreeObserver.OnPreD`
`224`	`224`	`}`
`225`	`225`	`}`
`226`	`226`
`227`		`-internal fun View.hasSize(): Boolean = width > 0 && height > 0`
	`227`	`+internal fun View.hasSize(): Boolean {`
	`228`	`+ return width != 0 && height != 0`
	`229`	`+}`