perf(android): Avoid exception-driven control flow in getResourceId

ViewUtils.getResourceId threw Resources.NotFoundException for views with
no id or a generated id, and callers caught and discarded it. During a
view-hierarchy snapshot and on every gesture this ran per view, so in
Compose-heavy apps where most views have generated ids the SDK
constructed an exception (and a native stack trace fill) per view on the
main thread.

Add a non-throwing resolveResourceId that returns null for unresolved
ids and route the hot callers through it. The public getResourceId
remains as a throwing wrapper for backward compatibility. Behavior
(emitted identifiers and fallbacks) is unchanged.

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

Author: runningcode

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

@@ -256,8 +256,10 @@ private static ViewHierarchyNode viewToNode(@NotNull final View view) {
     node.setType(className);
 
     try {
-      final String identifier = ViewUtils.getResourceId(view);
-      node.setIdentifier(identifier);
+      final @Nullable String identifier = ViewUtils.getResourceIdOrNull(view);
+      if (identifier != null) {
+        node.setIdentifier(identifier);
+      }
     } catch (Throwable e) {
       // ignored
     }

sentry-android-core/src/main/java/io/sentry/android/core/internal/gestures/AndroidViewGestureTargetLocator.java

@@ -1,6 +1,5 @@
 package io.sentry.android.core.internal.gestures;
 
-import android.content.res.Resources;
 import android.view.View;
 import android.widget.AbsListView;
 import android.widget.ScrollView;
@@ -42,13 +41,12 @@ && isViewScrollable(view, isAndroidXAvailable.getValue())) {
   }
 
   private UiElement createUiElement(final @NotNull View targetView) {
-    try {
-      final String resourceName = ViewUtils.getResourceId(targetView);
-      @Nullable String className = ClassUtil.getClassName(targetView);
-      return new UiElement(targetView, className, resourceName, null, ORIGIN);
-    } catch (Resources.NotFoundException ignored) {
+    final @Nullable String resourceName = ViewUtils.getResourceIdOrNull(targetView);
+    if (resourceName == null) {
       return null;
     }
+    @Nullable String className = ClassUtil.getClassName(targetView);
+    return new UiElement(targetView, className, resourceName, null, ORIGIN);
   }
 
   private static boolean isViewTappable(final @NotNull View view) {

sentry-android-core/src/main/java/io/sentry/android/core/internal/gestures/ViewUtils.java

@@ -150,13 +150,12 @@ private static final class ViewWithLocation {
    * @return human-readable view id
    */
   static String getResourceIdWithFallback(final @NotNull View view) {
-    final int viewId = view.getId();
-    try {
-      return getResourceId(view);
-    } catch (Resources.NotFoundException e) {
+    final @Nullable String resourceId = getResourceIdOrNull(view);
+    if (resourceId == null) {
       // fall back to hex representation of the id
-      return "0x" + Integer.toString(viewId, 16);
+      return "0x" + Integer.toString(view.getId(), 16);
     }
+    return resourceId;
   }
 
   /**
@@ -167,15 +166,36 @@ static String getResourceIdWithFallback(final @NotNull View view) {
    * @throws Resources.NotFoundException in case the view id was not found
    */
   public static String getResourceId(final @NotNull View view) throws Resources.NotFoundException {
+    final @Nullable String resourceId = getResourceIdOrNull(view);
+    if (resourceId == null) {
+      throw new Resources.NotFoundException();
+    }
+    return resourceId;
+  }
+
+  /**
+   * Retrieves the human-readable view id based on {@code view.getContext().getResources()}, or
+   * {@code null} when the view has no resource-backed id. Unlike {@link #getResourceId(View)} this
+   * does not throw for unresolved ids, avoiding exception-driven control flow on hot, main-thread
+   * paths such as view-hierarchy snapshots and gesture target resolution.
+   *
+   * @param view - the view whose id is being retrieved
+   * @return human-readable view id, or {@code null} if it cannot be resolved
+   */
+  public static @Nullable String getResourceIdOrNull(final @NotNull View view) {
     final int viewId = view.getId();
     if (viewId == View.NO_ID || isViewIdGenerated(viewId)) {
-      throw new Resources.NotFoundException();
+      return null;
     }
     final Resources resources = view.getContext().getResources();
-    if (resources != null) {
+    if (resources == null) {
+      return "";
+    }
+    try {
       return resources.getResourceEntryName(viewId);
+    } catch (Resources.NotFoundException e) {
+      return null;
     }
-    return "";
   }
 
   private static boolean isViewIdGenerated(int id) {

sentry-android-core/src/test/java/io/sentry/android/core/internal/gestures/ViewUtilsTest.kt

@@ -178,6 +178,65 @@ class ViewUtilsTest {
       gestureTargetLocators = listOf(AndroidViewGestureTargetLocator(LazyEvaluator { true }))
     }
 
+  @Test
+  fun `getResourceIdOrNull returns resource name when available`() {
+    val view =
+      mock<View> {
+        whenever(it.id).doReturn(0x7f010001)
+
+        val context = mock<Context>()
+        val resources = mock<Resources>()
+        whenever(resources.getResourceEntryName(it.id)).thenReturn("test_view")
+        whenever(context.resources).thenReturn(resources)
+        whenever(it.context).thenReturn(context)
+      }
+
+    assertEquals("test_view", ViewUtils.getResourceIdOrNull(view))
+  }
+
+  @Test
+  fun `getResourceIdOrNull returns null without throwing for generated id`() {
+    val context = mock<Context>()
+    val view =
+      mock<View> {
+        // View.generateViewId() starts with 1
+        whenever(it.id).doReturn(1)
+        whenever(it.context).thenReturn(context)
+      }
+
+    assertNull(ViewUtils.getResourceIdOrNull(view))
+    verify(context, never()).resources
+  }
+
+  @Test
+  fun `getResourceIdOrNull returns null without throwing when view has no id`() {
+    val context = mock<Context>()
+    val view =
+      mock<View> {
+        whenever(it.id).doReturn(View.NO_ID)
+        whenever(it.context).thenReturn(context)
+      }
+
+    assertNull(ViewUtils.getResourceIdOrNull(view))
+    verify(context, never()).resources
+  }
+
+  @Test
+  fun `getResourceIdOrNull returns null without throwing when resource not found`() {
+    val view =
+      mock<View> {
+        whenever(it.id).doReturn(1234)
+
+        val context = mock<Context>()
+        val resources = mock<Resources>()
+        whenever(resources.getResourceEntryName(it.id)).thenThrow(Resources.NotFoundException())
+        whenever(context.resources).thenReturn(resources)
+        whenever(it.context).thenReturn(context)
+      }
+
+    assertNull(ViewUtils.getResourceIdOrNull(view))
+  }
+
   @Test
   fun `getResourceIdWithFallback falls back to hexadecimal id when resource not found`() {
     val view =