[RPP] Do not update trace window when mouse leaves insight table

Bug: 371143933
Change-Id: I3225151c4d0f1431c0b41a8bfb5fdd636e4b851c
Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/5908528
Commit-Queue: Jack Franklin <jacktfranklin@chromium.org>
Auto-Submit: Connor Clark <cjamcl@chromium.org>
Reviewed-by: Jack Franklin <jacktfranklin@chromium.org>

front_end/panels/timeline/components/insights/Helpers.ts

@@ -134,14 +134,11 @@ export abstract class BaseInsight extends HTMLElement {
    * This enables the hover/click table interactions.
    */
   toggleTemporaryOverlays(
-      overlays: Overlays.Overlays.TimelineOverlay[]|null, options?: Overlays.Overlays.TimelineOverlaySetOptions): void {
+      overlays: Overlays.Overlays.TimelineOverlay[]|null, options: Overlays.Overlays.TimelineOverlaySetOptions): void {
     if (!this.isActive()) {
       return;
     }
 
-    if (!options) {
-      options = {updateTraceWindow: true};
-    }
     this.dispatchEvent(new SidebarInsight.InsightProvideOverlays(overlays ?? this.getInitialOverlays(), options));
   }
 

front_end/panels/timeline/components/insights/Table.ts

@@ -137,7 +137,7 @@ export class Table extends HTMLElement {
         this.#insight.toggleTemporaryOverlays(overlays, {updateTraceWindow: !opts.isHover});
       }
     } else {
-      this.#insight.toggleTemporaryOverlays(null);
+      this.#insight.toggleTemporaryOverlays(null, {updateTraceWindow: false});
     }
 
     this.#state.selectedRowEl?.classList.remove('selected');