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

Omit synthetic Debugger.disable message in modern CDP backend (#41912)

Browse Source 
Summary:
Pull Request resolved: https://github.com/facebook/react-native/pull/41912

Progress towards an opt-in setup for our new CDP backend.

- For `DevSupportManagerBase` on Android: Conditionally omit sending `Debugger.disable` CDP message when new CDP backend is enabled.

Reviewed By: motiz88

Differential Revision: D52040149

fbshipit-source-id: 452f46395261d2d9670bd38192d06e6fa8e1f93f
This commit is contained in:
Alex Hunt
2023-12-13 03:37:32 -08:00
committed by Facebook GitHub Bot
parent 7bdba281e7
commit 4fcfde1114
1 changed files with 5 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
packages/react-native/ReactAndroid/src/main/java/com/facebook/react/devsupport/DevSupportManagerBase.java
+5 -2
View File
@@ -33,6 +33,7 @@ import com.facebook.common.logging.FLog;
import com.facebook.infer.annotation.Assertions;
import com.facebook.react.R;
import com.facebook.react.bridge.DefaultJSExceptionHandler;
import com.facebook.react.bridge.InspectorFlags;
import com.facebook.react.bridge.JSBundleLoader;
import com.facebook.react.bridge.ReactContext;
import com.facebook.react.bridge.ReactMarker;
@@ -1043,8 +1044,10 @@ public abstract class DevSupportManagerBase implements DevSupportManager {
@Override
public void onPackagerReloadCommand() {
// Disable debugger to resume the JsVM & avoid thread locks while reloading
mDevServerHelper.disableDebugger();
if (!InspectorFlags.getEnableModernCDPRegistry()) {
// Disable debugger to resume the JsVM & avoid thread locks while reloading
mDevServerHelper.disableDebugger();
}
UiThreadUtil.runOnUiThread(() -> handleReloadJS());
}