Rename FabricJSIModuleProvider -> FabricUIManagerProviderImpl (#41296)

Summary: Pull Request resolved: #41296 As part of adding new implementation for `FabricUIManagerProviderImpl` APIs renaming the old class `FabricJSIModuleProvider` -> `FabricUIManagerProviderImpl` so as to add the new APIs later and preserve history. Changelog: [Internal] internal Reviewed By: philIip Differential Revision: D50949208 fbshipit-source-id: b833c4783b383c175fa682c558d31d8ecfa9f0ac
facebook · Nov 7, 2023 · bc68794 · bc68794
1 parent 4023e45
commit bc68794
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/...t-native/ReactAndroid/src/main/java/com/facebook/react/defaults/DefaultReactNativeHost.kt b/...t-native/ReactAndroid/src/main/java/com/facebook/react/defaults/DefaultReactNativeHost.kt
@@ -21,7 +21,7 @@ import com.facebook.react.bridge.JSIModuleType
 import com.facebook.react.bridge.ReactApplicationContext
 import com.facebook.react.bridge.UIManager
 import com.facebook.react.fabric.ComponentFactory
-import com.facebook.react.fabric.FabricJSIModuleProvider
+import com.facebook.react.fabric.FabricUIManagerProviderImpl
 import com.facebook.react.fabric.ReactNativeConfig
 import com.facebook.react.uimanager.ViewManagerRegistry
 
@@ -63,7 +63,7 @@ protected constructor(
                   val viewManagers =
                       reactInstanceManager.getOrCreateViewManagers(reactApplicationContext)
                   val viewManagerRegistry = ViewManagerRegistry(viewManagers)
-                  return FabricJSIModuleProvider(
+                  return FabricUIManagerProviderImpl(
                       reactApplicationContext,
                       componentFactory,
                       ReactNativeConfig.DEFAULT_CONFIG,

diff --git a/...react/fabric/FabricJSIModuleProvider.java → ...t/fabric/FabricUIManagerProviderImpl.java b/...react/fabric/FabricJSIModuleProvider.java → ...t/fabric/FabricUIManagerProviderImpl.java
@@ -15,14 +15,14 @@
 import com.facebook.react.uimanager.ViewManagerRegistry;
 import com.facebook.systrace.Systrace;
 
-public class FabricJSIModuleProvider implements JSIModuleProvider<UIManager> {
+public class FabricUIManagerProviderImpl implements JSIModuleProvider<UIManager> {
 
   @NonNull private final ReactApplicationContext mReactApplicationContext;
   @NonNull private final ComponentFactory mComponentFactory;
   @NonNull private final ReactNativeConfig mConfig;
   @NonNull private final ViewManagerRegistry mViewManagerRegistry;
 
-  public FabricJSIModuleProvider(
+  public FabricUIManagerProviderImpl(
       @NonNull ReactApplicationContext reactApplicationContext,
       @NonNull ComponentFactory componentFactory,
       @NonNull ReactNativeConfig config,
@@ -35,12 +35,12 @@ public FabricJSIModuleProvider(
 
   @Override
   public UIManager get() {
-    Systrace.beginSection(Systrace.TRACE_TAG_REACT_JAVA_BRIDGE, "FabricJSIModuleProvider.get");
+    Systrace.beginSection(Systrace.TRACE_TAG_REACT_JAVA_BRIDGE, "FabricUIManagerProviderImpl.get");
     final EventBeatManager eventBeatManager = new EventBeatManager();
     final FabricUIManager uiManager = createUIManager(eventBeatManager);
 
     Systrace.beginSection(
-        Systrace.TRACE_TAG_REACT_JAVA_BRIDGE, "FabricJSIModuleProvider.registerBinding");
+        Systrace.TRACE_TAG_REACT_JAVA_BRIDGE, "FabricUIManagerProviderImpl.registerBinding");
     final Binding binding = new BindingImpl();
 
     binding.register(
@@ -59,7 +59,7 @@ public UIManager get() {
 
   private FabricUIManager createUIManager(@NonNull EventBeatManager eventBeatManager) {
     Systrace.beginSection(
-        Systrace.TRACE_TAG_REACT_JAVA_BRIDGE, "FabricJSIModuleProvider.createUIManager");
+        Systrace.TRACE_TAG_REACT_JAVA_BRIDGE, "FabricUIManagerProviderImpl.createUIManager");
 
     FabricUIManager fabricUIManager;
     fabricUIManager =