fix(screenshot): Avoid crash from uncaught exception in view hierarchy traversal and unnecessary bitmap alloc in MaskRenderer.close()

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: romtsn

sentry-android-core/src/main/java/io/sentry/android/core/ScreenshotEventProcessor.java

@@ -182,20 +182,25 @@ private boolean isMaskingEnabled() {
   }
 
   private @Nullable ViewHierarchyNode buildViewHierarchy(final @NotNull Activity activity) {
-    final @Nullable View rootView =
-        activity.getWindow() != null
-                && activity.getWindow().peekDecorView() != null
-                && activity.getWindow().peekDecorView().getRootView() != null
-            ? activity.getWindow().peekDecorView().getRootView()
-            : null;
-    if (rootView == null) {
+    try {
+      final @Nullable View rootView =
+          activity.getWindow() != null
+                  && activity.getWindow().peekDecorView() != null
+                  && activity.getWindow().peekDecorView().getRootView() != null
+              ? activity.getWindow().peekDecorView().getRootView()
+              : null;
+      if (rootView == null) {
+        return null;
+      }
+
+      final ViewHierarchyNode rootNode =
+          ViewHierarchyNode.Companion.fromView(rootView, null, 0, options.getScreenshot());
+      ViewsKt.traverse(rootView, rootNode, options.getScreenshot(), options.getLogger());
+      return rootNode;
+    } catch (Throwable e) {
+      options.getLogger().log(SentryLevel.ERROR, "Failed to build view hierarchy", e);
       return null;
     }
-
-    final ViewHierarchyNode rootNode =
-        ViewHierarchyNode.Companion.fromView(rootView, null, 0, options.getScreenshot());
-    ViewsKt.traverse(rootView, rootNode, options.getScreenshot(), options.getLogger());
-    return rootNode;
   }
 
   private @Nullable Bitmap applyMasking(

sentry-android-replay/src/main/java/io/sentry/android/replay/util/MaskRenderer.kt

@@ -26,8 +26,9 @@ internal class MaskRenderer : Closeable {
   }
 
   // Single pixel bitmap for dominant color sampling (averaging the region)
-  internal val singlePixelBitmap: Bitmap by
+  private val lazySinglePixelBitmap: Lazy<Bitmap> =
     lazy(NONE) { Bitmap.createBitmap(1, 1, Bitmap.Config.ARGB_8888) }
+  internal val singlePixelBitmap: Bitmap by lazySinglePixelBitmap
   private val singlePixelBitmapCanvas: Canvas by lazy(NONE) { Canvas(singlePixelBitmap) }
   private val maskingPaint by lazy(NONE) { Paint() }
 
@@ -110,7 +111,7 @@ internal class MaskRenderer : Closeable {
 
   /** Releases resources. Call when done with this renderer. */
   override fun close() {
-    if (!singlePixelBitmap.isRecycled) {
+    if (lazySinglePixelBitmap.isInitialized() && !singlePixelBitmap.isRecycled) {
       singlePixelBitmap.recycle()
     }
   }