Implement getJavaScriptContextHolder for BridgelessCatalystInstance (#44054)

arushikesarwani94 · huntie · commit 26854de04be7 · 2024-04-15T11:21:13.000+02:00
Summary: Pull Request resolved: #44054 Implement `getJavaScriptContextHolder()` for BridgelessCatalystInstance Changelog: [Android][Breaking] Implement `getJavaScriptContextHolder()` for Bridgeless Catalyst Instance Reviewed By: christophpurrer Differential Revision: D56046452 fbshipit-source-id: b7fed1da3064608d8ef5fa84f4e53a4f7a84cba7
diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/runtime/BridgelessCatalystInstance.kt b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/runtime/BridgelessCatalystInstance.kt
@@ -145,8 +145,8 @@ public class BridgelessCatalystInstance(private val reactHost: ReactHostImpl) :
   }
 
   @Deprecated(message = "This API is unsupported in the New Architecture.")
-  override fun getJavaScriptContextHolder(): JavaScriptContextHolder {
-    throw UnsupportedOperationException("Unimplemented method 'getJavaScriptContextHolder'")
+  override fun getJavaScriptContextHolder(): JavaScriptContextHolder? {
+    return reactHost.getJavaScriptContextHolder()
   }
 
   override fun getRuntimeExecutor(): RuntimeExecutor? {

Original file line number	Diff line number	Diff line change
`@@ -145,8 +145,8 @@ public class BridgelessCatalystInstance(private val reactHost: ReactHostImpl) :`
`145`	`145`	`}`
`146`	`146`
`147`	`147`	`@Deprecated(message = "This API is unsupported in the New Architecture.")`
`148`		`- override fun getJavaScriptContextHolder(): JavaScriptContextHolder {`
`149`		`- throw UnsupportedOperationException("Unimplemented method 'getJavaScriptContextHolder'")`
	`148`	`+ override fun getJavaScriptContextHolder(): JavaScriptContextHolder? {`
	`149`	`+ return reactHost.getJavaScriptContextHolder()`
`150`	`150`	`}`
`151`	`151`
`152`	`152`	`override fun getRuntimeExecutor(): RuntimeExecutor? {`