From 8cfb08609ae650f1cd7d04b3d393125a41270236 Mon Sep 17 00:00:00 2001 From: Ramanpreet Nara Date: Mon, 1 Jul 2019 14:55:27 -0700 Subject: [PATCH] Remove TurboModuleManager dependency on ReactApplicationContext Summary: TurboModuleManager doesn't need access to ReactApplicationContext. Reviewed By: ejanzer Differential Revision: D16076581 fbshipit-source-id: 7b825c5a75d352aa6e83ed76c46f1cd70ba92ab2 --- .../facebook/react/turbomodule/core/TurboModuleManager.java | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/ReactAndroid/src/main/java/com/facebook/react/turbomodule/core/TurboModuleManager.java b/ReactAndroid/src/main/java/com/facebook/react/turbomodule/core/TurboModuleManager.java index c887c777bc0..b3bcbe47e9f 100644 --- a/ReactAndroid/src/main/java/com/facebook/react/turbomodule/core/TurboModuleManager.java +++ b/ReactAndroid/src/main/java/com/facebook/react/turbomodule/core/TurboModuleManager.java @@ -32,7 +32,6 @@ public class TurboModuleManager implements JSIModule, TurboModuleRegistry { SoLoader.loadLibrary("turbomodulejsijni"); } - private final ReactApplicationContext mReactApplicationContext; private final TurboModuleManagerDelegate mTurbomoduleManagerDelegate; private final Map mTurboModules = new HashMap<>(); @@ -41,9 +40,7 @@ public class TurboModuleManager implements JSIModule, TurboModuleRegistry { @SuppressWarnings("unused") private final HybridData mHybridData; - public TurboModuleManager( - ReactApplicationContext reactApplicationContext, JavaScriptContextHolder jsContext, TurboModuleManagerDelegate tmmDelegate, JSCallInvokerHolder instanceHolder) { - mReactApplicationContext = reactApplicationContext; + public TurboModuleManager(JavaScriptContextHolder jsContext, TurboModuleManagerDelegate tmmDelegate, JSCallInvokerHolder instanceHolder) { mHybridData = initHybrid(jsContext.get(), (JSCallInvokerHolderImpl) instanceHolder, tmmDelegate); mTurbomoduleManagerDelegate = tmmDelegate; }