From 53dc68a31dfeca704d2a5864f2db2dd8f0cd67fa Mon Sep 17 00:00:00 2001 From: Devan Buggay Date: Thu, 2 Oct 2025 07:21:18 -0700 Subject: [PATCH] Don't flash the performance monitor there's no metro connection (#54019) Summary: Pull Request resolved: https://github.com/facebook/react-native/pull/54019 Currently the performance monitor flickers on boot if there's no metro connection. This change checks first if we have a packager before enabling the monitor. No metro connection: {F1982423105} Metro connection: {F1982423112} Changelog: [Internal] Reviewed By: shwanton Differential Revision: D83701372 fbshipit-source-id: d099880d3ab509942a937a6bde11ebdde13450cd --- .../com/facebook/react/devsupport/DevSupportManagerBase.kt | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/devsupport/DevSupportManagerBase.kt b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/devsupport/DevSupportManagerBase.kt index 50e67445414..b3bec4749f7 100644 --- a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/devsupport/DevSupportManagerBase.kt +++ b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/devsupport/DevSupportManagerBase.kt @@ -567,8 +567,10 @@ public abstract class DevSupportManagerBase( perfMonitorOverlayManager?.let { manager -> reactInstanceDevHelper.inspectorTarget?.addPerfMonitorListener(manager) } - perfMonitorOverlayManager?.enable() - perfMonitorOverlayManager?.startBackgroundTrace() + if (isPackagerConnected) { + perfMonitorOverlayManager?.enable() + perfMonitorOverlayManager?.startBackgroundTrace() + } perfMonitorInitialized = true }