Facebook/react-native
1
0
Fork 0
mirror of https://github.com/facebook/react-native.git synced 2025-11-01 09:14:26 +00:00
Code Issues Packages Projects Releases Wiki Activity

Remove TurboModuleManager dependency on ReactApplicationContext

Browse Source 
Summary: TurboModuleManager doesn't need access to ReactApplicationContext.

Reviewed By: ejanzer

Differential Revision: D16076581

fbshipit-source-id: 7b825c5a75d352aa6e83ed76c46f1cd70ba92ab2
This commit is contained in:
Ramanpreet Nara
2019-07-01 14:59:16 -07:00
committed by Facebook Github Bot
parent 516b058256
commit 8cfb08609a
1 changed files with 1 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
ReactAndroid/src/main/java/com/facebook/react/turbomodule/core/TurboModuleManager.java
+1 -4
View File
@@ -32,7 +32,6 @@ public class TurboModuleManager implements JSIModule, TurboModuleRegistry {
SoLoader.loadLibrary("turbomodulejsijni");
}
private final ReactApplicationContext mReactApplicationContext;
private final TurboModuleManagerDelegate mTurbomoduleManagerDelegate;
private final Map<String, TurboModule> 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;
}