[Inspector V2] Fix bug preventing us from capturing initial load time for the V2 inspector (#8900)

Author: elliette

packages/devtools_app/lib/src/screens/inspector_v2/inspector_controller.dart

@@ -385,12 +385,12 @@ class InspectorController extends DisposableController
     return _waitForPendingUpdateDone();
   }
 
-  Future<void> refreshInspector() async {
-    // If the user is force refreshing the inspector before the first load has
-    // completed, this could indicate a slow load time or that the inspector
+  Future<void> refreshInspector({bool isManualRefresh = false}) async {
+    // If the user is manually refreshing the inspector before the first load
+    // has completed, this could indicate a slow load time or that the inspector
     // failed to load the tree once available.
-    if (!firstInspectorTreeLoadCompleted) {
-      // We do not want to complete this timing operation because the force
+    if (isManualRefresh && !firstInspectorTreeLoadCompleted) {
+      // We do not want to complete this timing operation because the manual
       // refresh will skew the results.
       ga.cancelTimingOperation(InspectorScreen.id, gac.pageReady);
       ga.select(

packages/devtools_app/lib/src/screens/inspector_v2/inspector_screen_body.dart

@@ -107,7 +107,7 @@ class InspectorScreenBodyState extends State<InspectorScreenBody>
     addAutoDisposeListener(preferences.inspector.pubRootDirectories, () {
       if (serviceConnection.serviceManager.connectedState.value.connected &&
           controller.firstInspectorTreeLoadCompleted) {
-        _refreshInspector();
+        controller.refreshInspector();
       }
     });
 
@@ -152,7 +152,7 @@ class InspectorScreenBodyState extends State<InspectorScreenBody>
               InspectorTreeControls(
                 isSearchVisible: searchVisible,
                 constraints: constraints,
-                onRefreshInspectorPressed: _refreshInspector,
+                onRefreshInspectorPressed: _manualInspectorRefresh,
                 onSearchVisibleToggle: _onSearchVisibleToggle,
                 searchFieldBuilder:
                     () => StatelessSearchField<InspectorTreeRow>(
@@ -219,15 +219,15 @@ class InspectorScreenBodyState extends State<InspectorScreenBody>
     _inspectorTreeController.resetSearch();
   }
 
-  void _refreshInspector() {
+  void _manualInspectorRefresh() {
     ga.select(
       gac.inspector,
       gac.refresh,
       screenMetricsProvider: () => InspectorScreenMetrics.v2(),
     );
     unawaited(
       blockWhileInProgress(() async {
-        await controller.refreshInspector();
+        await controller.refreshInspector(isManualRefresh: true);
       }),
     );
   }