From c360251b4a460375aa0e06d50647f52ee720eca9 Mon Sep 17 00:00:00 2001 From: Soe Lynn Date: Thu, 26 Jun 2025 14:44:36 -0700 Subject: [PATCH] Back out "Adding `shouldForwardToReactInstance` check in ReactDelegate for Bridgeless" (#52303) Summary: Pull Request resolved: https://github.com/facebook/react-native/pull/52303 Investigating javascript crash regression between Store APK v269 and v270 T228736366 with mid https://www.internalfb.com/logview/system_vros_javascripterrors/4a0131b8b79e3994b639c7ca212db717?ds=%7B%22start%22%3A%221750796919%22%2C%22constraints%22%3A[]%2C%22end%22%3A%22now%22%7D&ds_nux_type=task We suspect that this diff stack may be causing the issue could be caused by this Kotlin migration diff stack D76908041 Changelog: [Internal] Reverting Kotlin migration for ReactDelegate Reviewed By: mullender Differential Revision: D77388413 fbshipit-source-id: b144ca6db6f75614c7988e474ef89f19714a4a09 --- .../src/main/java/com/facebook/react/ReactDelegate.kt | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/ReactDelegate.kt b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/ReactDelegate.kt index 258bb6bcf47..4b2648ef692 100644 --- a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/ReactDelegate.kt +++ b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/ReactDelegate.kt @@ -195,8 +195,7 @@ public open class ReactDelegate { shouldForwardToReactInstance: Boolean ) { if (ReactNativeNewArchitectureFeatureFlags.enableBridgelessArchitecture() && - reactHost != null && - shouldForwardToReactInstance) { + reactHost != null) { reactHost?.onActivityResult(activity, requestCode, resultCode, data) } else { if (reactNativeHost?.hasInstance() == true && shouldForwardToReactInstance) {