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>();
+  #initialResourcesLoadedInfo: null|{count: number} = null;
 
   isEnabled(): boolean {
     return globalThis.enableReactNativePerfMetrics === true;
@@ -186,13 +187,8 @@ class RNPerfMetrics {
     });
   }
 
-  allInitialDeveloperResourcesLoadingFinished(count: number): void {
-    this.sendEvent({
-      eventName: 'DeveloperResource.AllInitialLoadingFinished',
-      params: {
-        count,
-      },
-    });
+  initialResourcesLoaded(count: number): void {
+    this.#initialResourcesLoadedInfo = {count};
   }
 
   fuseboxSetClientMetadataStarted(): void {
@@ -215,10 +211,22 @@ class RNPerfMetrics {
     }
   }
 
-  firstSteadyPing(): void {
+  tryReportingSteadyPing(): boolean {
+    if (this.#initialResourcesLoadedInfo === 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.#initialResourcesLoadedInfo.count,
+      }
     });
+    return true;
   }
 
   heapSnapshotStarted(): void {
@@ -432,13 +440,6 @@ export type DeveloperResourceLoadingFinishedEvent = Readonly<{
   }>,
 }>;
 
-export type AllInitialDeveloperResourcesLoadingFinished = Readonly<{
-  eventName: 'DeveloperResource.AllInitialLoadingFinished',
-  params: Readonly<{
-    count: number,
-  }>,
-}>;
-
 export type FuseboxSetClientMetadataStartedEvent = Readonly<{
   eventName: 'FuseboxSetClientMetadataStarted',
 }>;
@@ -511,16 +512,19 @@ export type StackTraceFrameUrlResolutionFailed = Readonly<{
   }>,
 }>;
 
-export type FirstSteadyPing = Readonly<{
-  eventName: 'FirstSteadyPing',
+export type FirstSteadyPingAfterInitialResourcesLoaded = Readonly<{
+  eventName: 'FirstSteadyPingAfterInitialResourcesLoaded',
+  params: Readonly<{
+    bundleCount: number,
+  }>,
 }>;
 
 export type ReactNativeChromeDevToolsEvent =
     EntrypointLoadingStartedEvent|EntrypointLoadingFinishedEvent|DebuggerReadyEvent|BrowserVisibilityChangeEvent|
     BrowserErrorEvent|RemoteDebuggingTerminatedEvent|DeveloperResourceLoadingStartedEvent|
-    DeveloperResourceLoadingFinishedEvent|AllInitialDeveloperResourcesLoadingFinished|FuseboxSetClientMetadataStartedEvent|
+    DeveloperResourceLoadingFinishedEvent|FuseboxSetClientMetadataStartedEvent|
     FuseboxSetClientMetadataFinishedEvent|MemoryPanelActionStartedEvent|MemoryPanelActionFinishedEvent|
     PanelShownEvent|PanelClosedEvent|StackTraceSymbolicationSucceeded|StackTraceSymbolicationFailed|
-    StackTraceFrameUrlResolutionSucceeded|StackTraceFrameUrlResolutionFailed|FirstSteadyPing;
+    StackTraceFrameUrlResolutionSucceeded|StackTraceFrameUrlResolutionFailed|FirstSteadyPingAfterInitialResourcesLoaded;
 
 export type DecoratedReactNativeChromeDevToolsEvent = CommonEventFields&ReactNativeChromeDevToolsEvent;

front_end/core/sdk/PageResourceLoader.ts

@@ -360,7 +360,7 @@ export class PageResourceLoader extends Common.ObjectWrapper.ObjectWrapper<Event
     // Tracking it here so it is next to the rest of "rnPerfMetrics" in this file.
     const allResourcesLoaded = this.#currentlyLoading === 1;
     if (allResourcesLoaded && !this.#reportedAllInitialResourcesLoaded) {
-      Host.rnPerfMetrics.allInitialDeveloperResourcesLoadingFinished(this.getNumberOfResources().resources);
+      Host.rnPerfMetrics.initialResourcesLoaded(this.getNumberOfResources().resources);
       this.#reportedAllInitialResourcesLoaded = true;
     }
 

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);
     }
   }