@dynamic-labs/react-native-extension 4.92.4 → 4.93.0

package/android/EmbeddedWebViewController.kt

@@ -113,7 +113,24 @@ object EmbeddedWebViewController {
         )
         return@runOnMain
       }
-      val webView = ensureWebView() ?: return@runOnMain
+      val webView = ensureWebView()
+      if (webView == null) {
+        // ensureWebView() needs appContext.currentActivity to host the
+        // overlay; during early startup or backgrounding there may be none
+        // yet. Surface the failure instead of silently dropping the load —
+        // the JS retry engine treats this domain as transient and retries,
+        // by which point an activity usually exists.
+        emitLoadError(
+          url = url,
+          code = 0,
+          // The event-domain constant trips FireHog's generic high-entropy
+          // string detector; it is a public event name, not a secret.
+          domain = "EmbeddedWebViewNoActivity", // firehog:ignore
+          description = "No current activity available to host the embedded webview",
+          isProvisional = true,
+        )
+        return@runOnMain
+      }
       // Pre-approve the origin so the first `shouldOverrideUrlLoading` call
       // can skip the JS round-trip. On cold boot the JS thread may be too
       // congested to respond within the navigation-decision timeout, silently
@@ -370,7 +387,7 @@ object EmbeddedWebViewController {
       pendingNavigationUrls[id] = url
 
       val timeout = Runnable {
-        // 2 s default-cancel: drop the stashed URL and surface as a load
+        // Default-cancel on timeout: drop the stashed URL and surface as a load
         // error so JS sees the timeout (matches iOS's cancellation semantics
         // but adds an explicit signal — iOS gets WebKitErrorDomain 102, which
         // is filtered out, so neither side emits a network error).

package/android/dynamic/embeddedwebview/EmbeddedWebViewController.kt

@@ -113,7 +113,24 @@ object EmbeddedWebViewController {
         )
         return@runOnMain
       }
-      val webView = ensureWebView() ?: return@runOnMain
+      val webView = ensureWebView()
+      if (webView == null) {
+        // ensureWebView() needs appContext.currentActivity to host the
+        // overlay; during early startup or backgrounding there may be none
+        // yet. Surface the failure instead of silently dropping the load —
+        // the JS retry engine treats this domain as transient and retries,
+        // by which point an activity usually exists.
+        emitLoadError(
+          url = url,
+          code = 0,
+          // The event-domain constant trips FireHog's generic high-entropy
+          // string detector; it is a public event name, not a secret.
+          domain = "EmbeddedWebViewNoActivity", // firehog:ignore
+          description = "No current activity available to host the embedded webview",
+          isProvisional = true,
+        )
+        return@runOnMain
+      }
       // Pre-approve the origin so the first `shouldOverrideUrlLoading` call
       // can skip the JS round-trip. On cold boot the JS thread may be too
       // congested to respond within the navigation-decision timeout, silently
@@ -370,7 +387,7 @@ object EmbeddedWebViewController {
       pendingNavigationUrls[id] = url
 
       val timeout = Runnable {
-        // 2 s default-cancel: drop the stashed URL and surface as a load
+        // Default-cancel on timeout: drop the stashed URL and surface as a load
         // error so JS sees the timeout (matches iOS's cancellation semantics
         // but adds an explicit signal — iOS gets WebKitErrorDomain 102, which
         // is filtered out, so neither side emits a network error).

package/android/embeddedwebview/EmbeddedWebViewController.kt

@@ -113,7 +113,24 @@ object EmbeddedWebViewController {
         )
         return@runOnMain
       }
-      val webView = ensureWebView() ?: return@runOnMain
+      val webView = ensureWebView()
+      if (webView == null) {
+        // ensureWebView() needs appContext.currentActivity to host the
+        // overlay; during early startup or backgrounding there may be none
+        // yet. Surface the failure instead of silently dropping the load —
+        // the JS retry engine treats this domain as transient and retries,
+        // by which point an activity usually exists.
+        emitLoadError(
+          url = url,
+          code = 0,
+          // The event-domain constant trips FireHog's generic high-entropy
+          // string detector; it is a public event name, not a secret.
+          domain = "EmbeddedWebViewNoActivity", // firehog:ignore
+          description = "No current activity available to host the embedded webview",
+          isProvisional = true,
+        )
+        return@runOnMain
+      }
       // Pre-approve the origin so the first `shouldOverrideUrlLoading` call
       // can skip the JS round-trip. On cold boot the JS thread may be too
       // congested to respond within the navigation-decision timeout, silently
@@ -370,7 +387,7 @@ object EmbeddedWebViewController {
       pendingNavigationUrls[id] = url
 
       val timeout = Runnable {
-        // 2 s default-cancel: drop the stashed URL and surface as a load
+        // Default-cancel on timeout: drop the stashed URL and surface as a load
         // error so JS sees the timeout (matches iOS's cancellation semantics
         // but adds an explicit signal — iOS gets WebKitErrorDomain 102, which
         // is filtered out, so neither side emits a network error).

package/android/java/xyz/dynamic/embeddedwebview/EmbeddedWebViewController.kt

@@ -113,7 +113,24 @@ object EmbeddedWebViewController {
         )
         return@runOnMain
       }
-      val webView = ensureWebView() ?: return@runOnMain
+      val webView = ensureWebView()
+      if (webView == null) {
+        // ensureWebView() needs appContext.currentActivity to host the
+        // overlay; during early startup or backgrounding there may be none
+        // yet. Surface the failure instead of silently dropping the load —
+        // the JS retry engine treats this domain as transient and retries,
+        // by which point an activity usually exists.
+        emitLoadError(
+          url = url,
+          code = 0,
+          // The event-domain constant trips FireHog's generic high-entropy
+          // string detector; it is a public event name, not a secret.
+          domain = "EmbeddedWebViewNoActivity", // firehog:ignore
+          description = "No current activity available to host the embedded webview",
+          isProvisional = true,
+        )
+        return@runOnMain
+      }
       // Pre-approve the origin so the first `shouldOverrideUrlLoading` call
       // can skip the JS round-trip. On cold boot the JS thread may be too
       // congested to respond within the navigation-decision timeout, silently
@@ -370,7 +387,7 @@ object EmbeddedWebViewController {
       pendingNavigationUrls[id] = url
 
       val timeout = Runnable {
-        // 2 s default-cancel: drop the stashed URL and surface as a load
+        // Default-cancel on timeout: drop the stashed URL and surface as a load
         // error so JS sees the timeout (matches iOS's cancellation semantics
         // but adds an explicit signal — iOS gets WebKitErrorDomain 102, which
         // is filtered out, so neither side emits a network error).

package/android/main/java/xyz/dynamic/embeddedwebview/EmbeddedWebViewController.kt

@@ -113,7 +113,24 @@ object EmbeddedWebViewController {
         )
         return@runOnMain
       }
-      val webView = ensureWebView() ?: return@runOnMain
+      val webView = ensureWebView()
+      if (webView == null) {
+        // ensureWebView() needs appContext.currentActivity to host the
+        // overlay; during early startup or backgrounding there may be none
+        // yet. Surface the failure instead of silently dropping the load —
+        // the JS retry engine treats this domain as transient and retries,
+        // by which point an activity usually exists.
+        emitLoadError(
+          url = url,
+          code = 0,
+          // The event-domain constant trips FireHog's generic high-entropy
+          // string detector; it is a public event name, not a secret.
+          domain = "EmbeddedWebViewNoActivity", // firehog:ignore
+          description = "No current activity available to host the embedded webview",
+          isProvisional = true,
+        )
+        return@runOnMain
+      }
       // Pre-approve the origin so the first `shouldOverrideUrlLoading` call
       // can skip the JS round-trip. On cold boot the JS thread may be too
       // congested to respond within the navigation-decision timeout, silently
@@ -370,7 +387,7 @@ object EmbeddedWebViewController {
       pendingNavigationUrls[id] = url
 
       val timeout = Runnable {
-        // 2 s default-cancel: drop the stashed URL and surface as a load
+        // Default-cancel on timeout: drop the stashed URL and surface as a load
         // error so JS sees the timeout (matches iOS's cancellation semantics
         // but adds an explicit signal — iOS gets WebKitErrorDomain 102, which
         // is filtered out, so neither side emits a network error).

package/android/src/main/java/xyz/dynamic/embeddedwebview/EmbeddedWebViewController.kt

@@ -113,7 +113,24 @@ object EmbeddedWebViewController {
         )
         return@runOnMain
       }
-      val webView = ensureWebView() ?: return@runOnMain
+      val webView = ensureWebView()
+      if (webView == null) {
+        // ensureWebView() needs appContext.currentActivity to host the
+        // overlay; during early startup or backgrounding there may be none
+        // yet. Surface the failure instead of silently dropping the load —
+        // the JS retry engine treats this domain as transient and retries,
+        // by which point an activity usually exists.
+        emitLoadError(
+          url = url,
+          code = 0,
+          // The event-domain constant trips FireHog's generic high-entropy
+          // string detector; it is a public event name, not a secret.
+          domain = "EmbeddedWebViewNoActivity", // firehog:ignore
+          description = "No current activity available to host the embedded webview",
+          isProvisional = true,
+        )
+        return@runOnMain
+      }
       // Pre-approve the origin so the first `shouldOverrideUrlLoading` call
       // can skip the JS round-trip. On cold boot the JS thread may be too
       // congested to respond within the navigation-decision timeout, silently
@@ -370,7 +387,7 @@ object EmbeddedWebViewController {
       pendingNavigationUrls[id] = url
 
       val timeout = Runnable {
-        // 2 s default-cancel: drop the stashed URL and surface as a load
+        // Default-cancel on timeout: drop the stashed URL and surface as a load
         // error so JS sees the timeout (matches iOS's cancellation semantics
         // but adds an explicit signal — iOS gets WebKitErrorDomain 102, which
         // is filtered out, so neither side emits a network error).

package/android/xyz/dynamic/embeddedwebview/EmbeddedWebViewController.kt

@@ -113,7 +113,24 @@ object EmbeddedWebViewController {
         )
         return@runOnMain
       }
-      val webView = ensureWebView() ?: return@runOnMain
+      val webView = ensureWebView()
+      if (webView == null) {
+        // ensureWebView() needs appContext.currentActivity to host the
+        // overlay; during early startup or backgrounding there may be none
+        // yet. Surface the failure instead of silently dropping the load —
+        // the JS retry engine treats this domain as transient and retries,
+        // by which point an activity usually exists.
+        emitLoadError(
+          url = url,
+          code = 0,
+          // The event-domain constant trips FireHog's generic high-entropy
+          // string detector; it is a public event name, not a secret.
+          domain = "EmbeddedWebViewNoActivity", // firehog:ignore
+          description = "No current activity available to host the embedded webview",
+          isProvisional = true,
+        )
+        return@runOnMain
+      }
       // Pre-approve the origin so the first `shouldOverrideUrlLoading` call
       // can skip the JS round-trip. On cold boot the JS thread may be too
       // congested to respond within the navigation-decision timeout, silently
@@ -370,7 +387,7 @@ object EmbeddedWebViewController {
       pendingNavigationUrls[id] = url
 
       val timeout = Runnable {
-        // 2 s default-cancel: drop the stashed URL and surface as a load
+        // Default-cancel on timeout: drop the stashed URL and surface as a load
         // error so JS sees the timeout (matches iOS's cancellation semantics
         // but adds an explicit signal — iOS gets WebKitErrorDomain 102, which
         // is filtered out, so neither side emits a network error).