fix(android): Fix ConcurrentModificationException (#5588)

* fix(android): Fix ConcurrentModificationException when disabling native crash handling

When enableNativeCrashHandling is set to false, the code was iterating over
the integrations list with a for-each loop while calling remove() directly,
which causes a ConcurrentModificationException at runtime.

Fixed by using Java 8's removeIf() method which safely handles iteration
and removal in a single operation. This is more concise and follows modern
Java best practices.

Added unit tests to verify the fix and ensure integrations are properly
removed without throwing exceptions.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

* Lint fix

---------

Co-authored-by: Claude Sonnet 4.5 <noreply@anthropic.com>

Author: antonis

packages/core/RNSentryAndroidTester/app/src/test/java/io/sentry/react/RNSentryStartTest.kt

@@ -236,4 +236,41 @@ class RNSentryStartTest {
         assertEquals("android", result?.getTag("event.origin"))
         assertEquals("java", result?.getTag("event.environment"))
     }
+
+    @Test
+    fun `when enableNativeCrashHandling is false, native crash integrations are removed without ConcurrentModificationException`() {
+        val rnOptions = JavaOnlyMap.of("enableNativeCrashHandling", false)
+        val options = SentryAndroidOptions()
+
+        // This should not throw ConcurrentModificationException
+        RNSentryStart.getSentryAndroidOptions(options, rnOptions, logger)
+
+        // Verify integrations were removed
+        val integrations = options.getIntegrations()
+        assertFalse(
+            "UncaughtExceptionHandlerIntegration should be removed",
+            integrations.any { it is io.sentry.UncaughtExceptionHandlerIntegration },
+        )
+        assertFalse(
+            "AnrIntegration should be removed",
+            integrations.any { it is io.sentry.android.core.AnrIntegration },
+        )
+        assertFalse(
+            "NdkIntegration should be removed",
+            integrations.any { it is io.sentry.android.core.NdkIntegration },
+        )
+    }
+
+    @Test
+    fun `when enableNativeCrashHandling is true, native crash integrations are kept`() {
+        val rnOptions = JavaOnlyMap.of("enableNativeCrashHandling", true)
+        val options = SentryAndroidOptions()
+
+        RNSentryStart.getSentryAndroidOptions(options, rnOptions, logger)
+
+        // When enabled, the default integrations should still be present
+        // Note: This test verifies that we don't remove integrations when the flag is true
+        val integrations = options.getIntegrations()
+        assertNotNull("Integrations list should not be null", integrations)
+    }
 }

packages/core/android/src/main/java/io/sentry/react/RNSentryStart.java

@@ -6,7 +6,6 @@
 import com.facebook.react.bridge.ReadableType;
 import com.facebook.react.common.JavascriptException;
 import io.sentry.ILogger;
-import io.sentry.Integration;
 import io.sentry.ProfileLifecycle;
 import io.sentry.Sentry;
 import io.sentry.SentryEvent;
@@ -25,7 +24,6 @@
 import io.sentry.react.replay.RNSentryReplayUnmask;
 import java.net.URI;
 import java.net.URISyntaxException;
-import java.util.List;
 import org.jetbrains.annotations.NotNull;
 import org.jetbrains.annotations.Nullable;
 
@@ -183,14 +181,13 @@ static void getSentryAndroidOptions(
 
     if (rnOptions.hasKey("enableNativeCrashHandling")
         && !rnOptions.getBoolean("enableNativeCrashHandling")) {
-      final List<Integration> integrations = options.getIntegrations();
-      for (final Integration integration : integrations) {
-        if (integration instanceof UncaughtExceptionHandlerIntegration
-            || integration instanceof AnrIntegration
-            || integration instanceof NdkIntegration) {
-          integrations.remove(integration);
-        }
-      }
+      options
+          .getIntegrations()
+          .removeIf(
+              integration ->
+                  integration instanceof UncaughtExceptionHandlerIntegration
+                      || integration instanceof AnrIntegration
+                      || integration instanceof NdkIntegration);
     }
     logger.log(
         SentryLevel.INFO, String.format("Native Integrations '%s'", options.getIntegrations()));