From 0bf6a8e914a6385f2d5d470426add81234c678b0 Mon Sep 17 00:00:00 2001 From: David Vacca Date: Mon, 28 Jan 2019 19:41:29 -0800 Subject: [PATCH] Remove ReactInstanceManager out of FabricJSIModuleProvider Summary: easy cleanup in FabricJSIModuleProvider Reviewed By: shergin Differential Revision: D13838336 fbshipit-source-id: 12738572394e9c091e559817f9886fa3aaf23570 --- .../com/facebook/react/fabric/FabricJSIModuleProvider.java | 4 ---- 1 file changed, 4 deletions(-) diff --git a/ReactAndroid/src/main/java/com/facebook/react/fabric/FabricJSIModuleProvider.java b/ReactAndroid/src/main/java/com/facebook/react/fabric/FabricJSIModuleProvider.java index 11ee1aae05b..4f0cbb775e6 100644 --- a/ReactAndroid/src/main/java/com/facebook/react/fabric/FabricJSIModuleProvider.java +++ b/ReactAndroid/src/main/java/com/facebook/react/fabric/FabricJSIModuleProvider.java @@ -1,7 +1,6 @@ // Copyright 2004-present Facebook. All Rights Reserved. package com.facebook.react.fabric; -import com.facebook.react.ReactInstanceManager; import com.facebook.react.bridge.JSIModuleProvider; import com.facebook.react.bridge.JavaScriptContextHolder; import com.facebook.react.bridge.ReactApplicationContext; @@ -34,17 +33,14 @@ import com.facebook.systrace.Systrace; public class FabricJSIModuleProvider implements JSIModuleProvider { - private final ReactInstanceManager mReactInstanceManager; private final JavaScriptContextHolder mJSContext; private final ReactApplicationContext mReactApplicationContext; private final ComponentFactoryDelegate mComponentFactoryDelegate; public FabricJSIModuleProvider( - ReactInstanceManager reactInstanceManager, ReactApplicationContext reactApplicationContext, JavaScriptContextHolder jsContext, ComponentFactoryDelegate componentFactoryDelegate) { - mReactInstanceManager = reactInstanceManager; mReactApplicationContext = reactApplicationContext; mJSContext = jsContext; mComponentFactoryDelegate = componentFactoryDelegate;