Skip to content

Commit

Permalink
Fix ReactInstanceManager for getting rid of getJSIModule()
Browse files Browse the repository at this point in the history
Summary: Fix ReactInstanceManager for getting rid of `getJSIModule()` in order to make Catalyst and RN-Tester work with the changes for Fabric initialization

Differential Revision: D51338036
  • Loading branch information
arushikesarwani94 authored and facebook-github-bot committed Nov 15, 2023
1 parent 560e0f0 commit f27d2a5
Showing 1 changed file with 0 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,6 @@
import com.facebook.react.bridge.JSBundleLoader;
import com.facebook.react.bridge.JSExceptionHandler;
import com.facebook.react.bridge.JSIModulePackage;
import com.facebook.react.bridge.JSIModuleType;
import com.facebook.react.bridge.JavaJSExecutor;
import com.facebook.react.bridge.JavaScriptExecutor;
import com.facebook.react.bridge.JavaScriptExecutorFactory;
Expand Down Expand Up @@ -1407,7 +1406,6 @@ private ReactApplicationContext createReactContext(
reactContext, catalystInstance.getJavaScriptContextHolder()));
}
if (ReactFeatureFlags.enableFabricRenderer) {
catalystInstance.getJSIModule(JSIModuleType.UIManager);
if (mUIManagerProvider != null) {
catalystInstance.setFabricUIManager(mUIManagerProvider.createUIManager(reactContext));
}
Expand Down

0 comments on commit f27d2a5

Please sign in to comment.