Fix SDK init crash if initialized from background thread (#4449)

* Fix GestureDetectorCompat crashes if initialized from background thread

* Add tests

* Update Changelog

Author: markushi

CHANGELOG.md

@@ -2,6 +2,10 @@
 
 ## Unreleased
 
+### Fixes
+
+- Fix SDK init crash if initialized from background thread while an activiy is resumed ([#4449](https://github.com/getsentry/sentry-java/pull/4449))
+
 ### Dependencies
 
 - Bump Gradle from v8.14 to v8.14.1 ([#4437](https://github.com/getsentry/sentry-java/pull/4437))

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

@@ -1,6 +1,8 @@
 package io.sentry.android.core.internal.gestures;
 
 import android.content.Context;
+import android.os.Handler;
+import android.os.Looper;
 import android.view.MotionEvent;
 import android.view.Window;
 import androidx.core.view.GestureDetectorCompat;
@@ -27,7 +29,7 @@ public SentryWindowCallback(
       final @Nullable SentryOptions options) {
     this(
         delegate,
-        new GestureDetectorCompat(context, gestureListener),
+        new GestureDetectorCompat(context, gestureListener, new Handler(Looper.getMainLooper())),
         gestureListener,
         options,
         new MotionEventObtainer() {});

sentry/src/main/java/io/sentry/Sentry.java

@@ -345,7 +345,16 @@ private static void init(final @NotNull SentryOptions options, final boolean glo
         // and Scopes was still NoOp.
         // Registering integrations here make sure that Scopes is already created.
         for (final Integration integration : options.getIntegrations()) {
-          integration.register(ScopesAdapter.getInstance(), options);
+          try {
+            integration.register(ScopesAdapter.getInstance(), options);
+          } catch (Throwable t) {
+            options
+                .getLogger()
+                .log(
+                    SentryLevel.WARNING,
+                    "Failed to register the integration " + integration.getClass().getName(),
+                    t);
+          }
         }
 
         notifyOptionsObservers(options);

sentry/src/test/java/io/sentry/SentryTest.kt

@@ -125,6 +125,27 @@ class SentryTest {
         verify(integration2).close()
     }
 
+    @Test
+    fun `if a single integration crashes, the SDK and other integrations are still initialized`() {
+        val goodIntegrationInitialized = AtomicBoolean(false)
+        val goodIntegration = Integration { scopes, options ->
+            // no-op
+            goodIntegrationInitialized.set(true)
+        }
+
+        val badIntegration = Integration { scopes, options -> throw IllegalStateException("bad integration") }
+
+        Sentry.init {
+            it.dsn = dsn
+            it.integrations.clear()
+            it.integrations.add(badIntegration)
+            it.integrations.add(goodIntegration)
+        }
+
+        assertTrue(Sentry.isEnabled())
+        assertTrue(goodIntegrationInitialized.get())
+    }
+
     interface CloseableIntegration : Integration, Closeable
 
     @Test