report first steady ping after initial resources loaded

Author: vzaidman

front_end/core/host/RNPerfMetrics.ts

@@ -29,6 +29,7 @@ class RNPerfMetrics {
   #telemetryInfo: Object = {};
   // map of panel location to panel name
   #currentPanels = new Map<PanelLocation, string>();
+  #initialBundlesLoadedInfo: null|{count: number} = null;
 
   isEnabled(): boolean {
     return globalThis.enableReactNativePerfMetrics === true;
@@ -187,12 +188,7 @@ class RNPerfMetrics {
   }
 
   allInitialDeveloperResourcesLoadingFinished(count: number): void {
-    this.sendEvent({
-      eventName: 'DeveloperResource.AllInitialLoadingFinished',
-      params: {
-        count,
-      },
-    });
+    this.#initialBundlesLoadedInfo = {count};
   }
 
   fuseboxSetClientMetadataStarted(): void {
@@ -215,10 +211,22 @@ class RNPerfMetrics {
     }
   }
 
-  firstSteadyPing(): void {
+  tryReportingSteadyPing(): boolean {
+    if (this.#initialBundlesLoadedInfo === null) {
+      return false;
+    }
+
+    // const [navigationEntry] = performance.getEntriesByType('navigation');
+    // eslint-disable-next-line
+    console.info('Startup time is %s', performance.now());
+
     this.sendEvent({
-      eventName: 'FirstSteadyPing',
+      eventName: 'FirstSteadyPingAfterInitialResourcesLoaded',
+      params: {
+        bundleCount: this.#initialBundlesLoadedInfo.count,
+      }
     });
+    return true;
   }
 
   heapSnapshotStarted(): void {
@@ -511,8 +519,11 @@ export type StackTraceFrameUrlResolutionFailed = Readonly<{
   }>,
 }>;
 
-export type FirstSteadyPing = Readonly<{
-  eventName: 'FirstSteadyPing',
+export type FirstSteadyPingAfterInitialResourcesLoaded = Readonly<{
+  eventName: 'FirstSteadyPingAfterInitialResourcesLoaded',
+  params: Readonly<{
+    bundleCount: number,
+  }>,
 }>;
 
 export type ReactNativeChromeDevToolsEvent =
@@ -521,6 +532,6 @@ export type ReactNativeChromeDevToolsEvent =
     DeveloperResourceLoadingFinishedEvent|AllInitialDeveloperResourcesLoadingFinished|FuseboxSetClientMetadataStartedEvent|
     FuseboxSetClientMetadataFinishedEvent|MemoryPanelActionStartedEvent|MemoryPanelActionFinishedEvent|
     PanelShownEvent|PanelClosedEvent|StackTraceSymbolicationSucceeded|StackTraceSymbolicationFailed|
-    StackTraceFrameUrlResolutionSucceeded|StackTraceFrameUrlResolutionFailed|FirstSteadyPing;
+    StackTraceFrameUrlResolutionSucceeded|StackTraceFrameUrlResolutionFailed|FirstSteadyPingAfterInitialResourcesLoaded;
 
 export type DecoratedReactNativeChromeDevToolsEvent = CommonEventFields&ReactNativeChromeDevToolsEvent;

front_end/entrypoints/inspector_main/InspectorMain.ts

@@ -77,10 +77,15 @@ export class InspectorMainImpl implements Common.Runnable.Runnable {
       this.#consecutiveLowPing++;
     }
 
+    let reportedSteadyPing = false;
     if (this.#consecutiveLowPing > 1) {
-      Host.rnPerfMetrics.firstSteadyPing();
-    } else {
-      setTimeout(() => void this.#measureMainConnectionPing(debuggerModel), COOLDOWN_BETWEEN_PINGS);
+      reportedSteadyPing = Host.rnPerfMetrics.tryReportingSteadyPing();
+    }
+
+    if (!reportedSteadyPing) {
+      setTimeout(() => {
+        void this.#measureMainConnectionPing(debuggerModel);
+      }, COOLDOWN_BETWEEN_PINGS);
     }
   }