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 TurboModule debug logs

Browse Source 
Summary:
These logs are no longer necessary, because data indicates that the TurboModule eager init crash was fixed.

Changelog: [Internal]

Reviewed By: fkgozali

Differential Revision: D21852743

fbshipit-source-id: ddeefd6396283ee5e15980a33fb006cb83a81532
This commit is contained in:
Ramanpreet Nara
2020-06-02 20:11:53 -07:00
committed by Facebook GitHub Bot
parent d3b937f990
commit e5bef7338f
4 changed files with 0 additions and 47 deletions
Download Patch File Download Diff File Expand all files Collapse all files
ReactAndroid/src/main/java/com/facebook/react/ReactInstanceManager.java
-24
View File
@@ -1279,39 +1279,15 @@ public class ReactInstanceManager {
reactContext.initializeWithInstance(catalystInstance);
if (ReactFeatureFlags.enableTurboModuleDebugLogs) {
// TODO(T46487253): Remove after task is closed
FLog.e(
ReactConstants.TAG,
"ReactInstanceManager.createReactContext: mJSIModulePackage "
+ (mJSIModulePackage != null ? "not null" : "null"));
}
if (mJSIModulePackage != null) {
catalystInstance.addJSIModules(
mJSIModulePackage.getJSIModules(
reactContext, catalystInstance.getJavaScriptContextHolder()));
if (ReactFeatureFlags.enableTurboModuleDebugLogs) {
// TODO(T46487253): Remove after task is closed
FLog.e(
ReactConstants.TAG,
"ReactInstanceManager.createReactContext: ReactFeatureFlags.useTurboModules == "
+ (ReactFeatureFlags.useTurboModules == false ? "false" : "true"));
}
if (ReactFeatureFlags.useTurboModules) {
JSIModule turboModuleManager =
catalystInstance.getJSIModule(JSIModuleType.TurboModuleManager);
if (ReactFeatureFlags.enableTurboModuleDebugLogs) {
// TODO(T46487253): Remove after task is closed
FLog.e(
ReactConstants.TAG,
"ReactInstanceManager.createReactContext: TurboModuleManager "
+ (turboModuleManager == null ? "not created" : "created"));
}
catalystInstance.setTurboModuleManager(turboModuleManager);
TurboModuleRegistry registry = (TurboModuleRegistry) turboModuleManager;
ReactAndroid/src/main/java/com/facebook/react/bridge/CatalystInstanceImpl.java
-10
View File
@@ -582,16 +582,6 @@ public class CatalystInstanceImpl implements CatalystInstance {
public NativeModule getNativeModule(String moduleName) {
if (getTurboModuleRegistry() != null) {
TurboModule turboModule = getTurboModuleRegistry().getModule(moduleName);
if (ReactFeatureFlags.enableTurboModuleDebugLogs) {
// TODO(T46487253): Remove after task is closed
FLog.e(
ReactConstants.TAG,
"CatalystInstanceImpl.getNativeModule: TurboModule "
+ moduleName
+ (turboModule == null ? " not" : "")
+ " found");
}
if (turboModule != null) {
return (NativeModule) turboModule;
}
ReactAndroid/src/main/java/com/facebook/react/config/ReactFeatureFlags.java
-3
View File
@@ -23,9 +23,6 @@ public class ReactFeatureFlags {
*/
public static volatile boolean useTurboModules = false;
/** Should we output debug logs to debug the TurboModule infra? */
public static volatile boolean enableTurboModuleDebugLogs = false;
/*
* This feature flag enables logs for Fabric
*/
ReactAndroid/src/main/java/com/facebook/react/turbomodule/core/TurboModuleManager.java
-10
View File
@@ -19,7 +19,6 @@ import com.facebook.react.bridge.JSIModule;
import com.facebook.react.bridge.JavaScriptContextHolder;
import com.facebook.react.bridge.NativeModule;
import com.facebook.react.common.ReactConstants;
import com.facebook.react.config.ReactFeatureFlags;
import com.facebook.react.turbomodule.core.interfaces.CallInvokerHolder;
import com.facebook.react.turbomodule.core.interfaces.TurboModule;
import com.facebook.react.turbomodule.core.interfaces.TurboModuleRegistry;
@@ -142,15 +141,6 @@ public class TurboModuleManager implements JSIModule, TurboModuleRegistry {
/*
* Always return null after cleanup has started, so that getModule(moduleName) returns null.
*/
if (ReactFeatureFlags.enableTurboModuleDebugLogs) {
// TODO(T46487253): Remove after task is closed
FLog.e(
ReactConstants.TAG,
"TurboModuleManager.getOrMaybeCreateTurboModuleHolder: Tried to require TurboModule "
+ moduleName
+ " after cleanup initiated");
}
return null;
}