From ea1aff455af8d180a078f0d8fbf1af20bbbfe4d1 Mon Sep 17 00:00:00 2001 From: Nicola Corti Date: Mon, 4 Aug 2025 07:51:30 -0700 Subject: [PATCH] Make `ReactCxxErrorHandler` internal (#53024) Summary: Pull Request resolved: https://github.com/facebook/react-native/pull/53024 This class is Legacy Arch and is not used in OSS. Let's make it internal. Changelog: [Internal] [Changed] - Reviewed By: RSNara Differential Revision: D79556615 fbshipit-source-id: d157fe8f04784038d64657c6d240b0c51e41d82d --- packages/react-native/ReactAndroid/api/ReactAndroid.api | 5 ----- .../java/com/facebook/react/bridge/ReactCxxErrorHandler.kt | 4 ++-- 2 files changed, 2 insertions(+), 7 deletions(-) diff --git a/packages/react-native/ReactAndroid/api/ReactAndroid.api b/packages/react-native/ReactAndroid/api/ReactAndroid.api index e0fcc225a9d..503fa203c12 100644 --- a/packages/react-native/ReactAndroid/api/ReactAndroid.api +++ b/packages/react-native/ReactAndroid/api/ReactAndroid.api @@ -1140,11 +1140,6 @@ public abstract class com/facebook/react/bridge/ReactContextBaseJavaModule : com protected final fun getCurrentActivity ()Landroid/app/Activity; } -public final class com/facebook/react/bridge/ReactCxxErrorHandler { - public static final field INSTANCE Lcom/facebook/react/bridge/ReactCxxErrorHandler; - public static final fun setHandleErrorFunc (Ljava/lang/Object;Ljava/lang/reflect/Method;)V -} - public final class com/facebook/react/bridge/ReactMarker { public static final field INSTANCE Lcom/facebook/react/bridge/ReactMarker; public static final fun addFabricListener (Lcom/facebook/react/bridge/ReactMarker$FabricMarkerListener;)V diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/bridge/ReactCxxErrorHandler.kt b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/bridge/ReactCxxErrorHandler.kt index 7ae58bd3b9a..f2a3a4a3e68 100644 --- a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/bridge/ReactCxxErrorHandler.kt +++ b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/bridge/ReactCxxErrorHandler.kt @@ -16,7 +16,7 @@ import java.lang.reflect.Method @DoNotStrip @LegacyArchitecture -public object ReactCxxErrorHandler { +internal object ReactCxxErrorHandler { init { LegacyArchitectureLogger.assertLegacyArchitecture( "ReactCxxErrorHandler", LegacyArchitectureLogLevel.WARNING) @@ -27,7 +27,7 @@ public object ReactCxxErrorHandler { @DoNotStrip @JvmStatic - public fun setHandleErrorFunc(handlerObject: Any?, handleErrorFunc: Method?) { + fun setHandleErrorFunc(handlerObject: Any?, handleErrorFunc: Method?) { this.handlerObject = handlerObject this.handleErrorFunc = handleErrorFunc }