Make ReactContext and ReactApplicationContext classes abstract (faceb…

…ook#43625) Summary: Pull Request resolved: facebook#43625 Changelog: [Android][Breaking] Make ReactApplicationContext and ReactContext abstract. Please instantiate BridgeReactContext instead (bridge mode). Or BridgelessReactContext instead (bridgeless mode). Reviewed By: arushikesarwani94 Differential Revision: D55218590
RSNara · Mar 22, 2024 · 8e46e4f · 8e46e4f
1 parent 9a41bf3
commit 8e46e4f
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/packages/react-native/ReactAndroid/api/ReactAndroid.api b/packages/react-native/ReactAndroid/api/ReactAndroid.api
@@ -1056,7 +1056,7 @@ public class com/facebook/react/bridge/ProxyJavaScriptExecutor$Factory : com/fac
 	public fun stopSamplingProfiler (Ljava/lang/String;)V
 }
 
-public class com/facebook/react/bridge/ReactApplicationContext : com/facebook/react/bridge/ReactContext {
+public abstract class com/facebook/react/bridge/ReactApplicationContext : com/facebook/react/bridge/ReactContext {
 	public fun <init> (Landroid/content/Context;)V
 }
 
@@ -1068,7 +1068,7 @@ public class com/facebook/react/bridge/ReactBridge {
 	public static fun staticInit ()V
 }
 
-public class com/facebook/react/bridge/ReactContext : android/content/ContextWrapper {
+public abstract class com/facebook/react/bridge/ReactContext : android/content/ContextWrapper {
 	protected field mInteropModuleRegistry Lcom/facebook/react/bridge/interop/InteropModuleRegistry;
 	public fun <init> (Landroid/content/Context;)V
 	public fun addActivityEventListener (Lcom/facebook/react/bridge/ActivityEventListener;)V

diff --git a/...-native/ReactAndroid/src/main/java/com/facebook/react/bridge/ReactApplicationContext.java b/...-native/ReactAndroid/src/main/java/com/facebook/react/bridge/ReactApplicationContext.java
@@ -13,7 +13,7 @@
  * A context wrapper that always wraps Android Application {@link Context} and {@link
  * CatalystInstance} by extending {@link ReactContext}
  */
-public class ReactApplicationContext extends ReactContext {
+public abstract class ReactApplicationContext extends ReactContext {
   // We want to wrap ApplicationContext, since there is no easy way to verify that application
   // context is passed as a param, we use {@link Context#getApplicationContext} to ensure that
   // the context we're wrapping is in fact an application context.

diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/bridge/ReactContext.java b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/bridge/ReactContext.java
@@ -37,7 +37,7 @@
  * Abstract ContextWrapper for Android application or activity {@link Context} and {@link
  * CatalystInstance}
  */
-public class ReactContext extends ContextWrapper {
+public abstract class ReactContext extends ContextWrapper {
 
   @DoNotStrip
   public interface RCTDeviceEventEmitter extends JavaScriptModule {