[RPP] Expand entry track only when modifying trace window

This fixes the experience for a user hovering on insight tables where
hovering after closing a track re-opens it.

Bug: 398043732
Change-Id: Ief66a7caaec21489343c8d3539a1926a1cc51895
Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/6305885
Auto-Submit: Adriana Ixba <[email protected]>
Reviewed-by: Adam Raine <[email protected]>
Commit-Queue: Adam Raine <[email protected]>
Reviewed-by: Connor Clark <[email protected]>

Author: Adriana Ixba

front_end/panels/timeline/TimelineFlameChartView.ts

@@ -692,11 +692,6 @@ export class TimelineFlameChartView extends Common.ObjectWrapper.eventMixin<Even
       entries.push(...Overlays.Overlays.entriesForOverlay(overlay));
     }
 
-    for (const entry of entries) {
-      // Ensure that the track for the entries are open.
-      this.#expandEntryTrack(entry);
-    }
-
     if (Root.Runtime.experiments.isEnabled(Root.Runtime.ExperimentName.TIMELINE_DIM_UNRELATED_EVENTS)) {
       // The insight's `relatedEvents` property likely already includes the events associated with
       // an overlay, but just in case not, include both arrays. Duplicates are fine.
@@ -710,6 +705,13 @@ export class TimelineFlameChartView extends Common.ObjectWrapper.eventMixin<Even
     }
 
     if (options.updateTraceWindow) {
+      // We should only expand the entry track when we are updating the trace window
+      // (eg. when insight cards are initially opened).
+      // Otherwise the track will open when not intending to.
+      for (const entry of entries) {
+        // Ensure that the track for the entries are open.
+        this.#expandEntryTrack(entry);
+      }
       const overlaysBounds = Overlays.Overlays.traceWindowContainingOverlays(this.#currentInsightOverlays);
       if (overlaysBounds) {
         // Trace window covering all overlays expanded by 100% so that the overlays cover 50% of the visible window.