chrome-devtools-frontend 1.0.1553956 → 1.0.1555430

package/front_end/panels/timeline/components/SidebarInsightsTab.ts

@@ -1,11 +1,9 @@
 // Copyright 2024 The Chromium Authors
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
-/* eslint-disable @devtools/no-lit-render-outside-of-view */
 
 import * as Trace from '../../../models/trace/trace.js';
 import * as Buttons from '../../../ui/components/buttons/buttons.js';
-import * as ComponentHelpers from '../../../ui/components/helpers/helpers.js';
 import * as UI from '../../../ui/legacy/legacy.js';
 import * as Lit from '../../../ui/lit/lit.js';
 import * as Utils from '../utils/utils.js';
@@ -16,9 +14,136 @@ import sidebarInsightsTabStyles from './sidebarInsightsTab.css.js';
 import {SidebarSingleInsightSet, type SidebarSingleInsightSetData} from './SidebarSingleInsightSet.js';
 
 const {html} = Lit;
+const {widgetConfig} = UI.Widget;
+
+interface ViewInput {
+  parsedTrace: Trace.TraceModel.ParsedTrace;
+  labels: string[];
+  activeInsightSet: Trace.Insights.Types.InsightSet|null;
+  activeInsight: ActiveInsight|null;
+  selectedCategory: Trace.Insights.Types.InsightCategory;
+  onInsightSetToggled: (insightSet: Trace.Insights.Types.InsightSet) => void;
+  onInsightSetHovered: (insightSet: Trace.Insights.Types.InsightSet) => void;
+  onInsightSetUnhovered: () => void;
+  onZoomClick: (insightSet: Trace.Insights.Types.InsightSet) => void;
+}
+
+type View = (input: ViewInput, output: undefined, target: HTMLElement) => void;
+
+export const DEFAULT_VIEW: View = (input, output, target) => {
+  const {
+    parsedTrace,
+    labels,
+    activeInsightSet,
+    activeInsight,
+    selectedCategory,
+    onInsightSetToggled,
+    onInsightSetHovered,
+    onInsightSetUnhovered,
+    onZoomClick,
+  } = input;
+
+  const insights = parsedTrace.insights;
+  if (!insights) {
+    return;
+  }
 
-export class SidebarInsightsTab extends HTMLElement {
-  readonly #shadow = this.attachShadow({mode: 'open'});
+  const hasMultipleInsightSets = insights.size > 1;
+
+  // clang-format off
+  Lit.render(html`
+    <style>${sidebarInsightsTabStyles}</style>
+    <div class="insight-sets-wrapper">
+      ${[...insights.values()].map((insightSet, index) => {
+        const {id, url} = insightSet;
+        const data: SidebarSingleInsightSetData = {
+          insightSetKey: id,
+          activeCategory: selectedCategory,
+          activeInsight,
+          parsedTrace,
+        };
+        const selected = insightSet === activeInsightSet;
+
+        const contents = html`
+          <devtools-widget
+            data-insight-set-key=${id}
+            .widgetConfig=${widgetConfig(SidebarSingleInsightSet, {data})}
+          ></devtools-widget>
+        `;
+
+        if (hasMultipleInsightSets) {
+          return html`<details ?open=${selected}>
+            <summary
+              @click=${() => onInsightSetToggled(insightSet)}
+              @mouseenter=${() => onInsightSetHovered(insightSet)}
+              @mouseleave=${() => onInsightSetUnhovered()}
+              title=${url.href}>
+              ${renderDropdownIcon(selected)}
+              <span>${labels[index]}</span>
+              <span class='zoom-button'
+                @click=${(event: Event) => {
+                  event.stopPropagation();
+                  onZoomClick(insightSet);
+                }}
+              >
+                ${renderZoomButton(selected)}
+              </span>
+            </summary>
+            ${contents}
+          </details>`;
+        }
+
+        return contents;
+      })}
+    </div>
+  `, target);
+  // clang-format on
+};
+
+function renderZoomButton(insightSetToggled: boolean): Lit.TemplateResult {
+  const classes = Lit.Directives.classMap({
+    'zoom-icon': true,
+    active: insightSetToggled,
+  });
+
+  // clang-format off
+  return html`
+  <div class=${classes}>
+      <devtools-button .data=${{
+        variant: Buttons.Button.Variant.ICON,
+        iconName: 'center-focus-weak',
+        size: Buttons.Button.Size.SMALL,
+      } as Buttons.Button.ButtonData}
+    ></devtools-button></div>`;
+  // clang-format on
+}
+
+function renderDropdownIcon(insightSetToggled: boolean): Lit.TemplateResult {
+  const containerClasses = Lit.Directives.classMap({
+    'dropdown-icon': true,
+    active: insightSetToggled,
+  });
+
+  // clang-format off
+  return html`
+    <div class=${containerClasses}>
+      <devtools-button .data=${{
+        variant: Buttons.Button.Variant.ICON,
+        iconName: 'chevron-right',
+        size: Buttons.Button.Size.SMALL,
+      } as Buttons.Button.ButtonData}
+    ></devtools-button></div>
+  `;
+  // clang-format on
+}
+
+export class SidebarInsightsTab extends UI.Widget.Widget {
+  static createWidgetElement(): UI.Widget.WidgetElement<SidebarInsightsTab> {
+    const widgetElement = document.createElement('devtools-widget') as UI.Widget.WidgetElement<SidebarInsightsTab>;
+    return widgetElement;
+  }
+
+  #view: View;
   #parsedTrace: Trace.TraceModel.ParsedTrace|null = null;
   #activeInsight: ActiveInsight|null = null;
   #selectedCategory = Trace.Insights.Types.InsightCategory.ALL;
@@ -26,9 +151,14 @@ export class SidebarInsightsTab extends HTMLElement {
    * When a trace has sets of insights, we show an accordion with each
    * set within. A set can be specific to a single navigation, or include the
    * beginning of the trace up to the first navigation.
-   * You can only have one of these open at any time, and we track it via this ID.
+   * You can only have one of these open at any time.
    */
-  #selectedInsightSetKey: string|null = null;
+  #selectedInsightSet: Trace.Insights.Types.InsightSet|null = null;
+
+  constructor(element?: HTMLElement, view: View = DEFAULT_VIEW) {
+    super(element, {useShadowDom: true});
+    this.#view = view;
+  }
 
   // TODO(paulirish): add back a disconnectedCallback() to avoid memory leaks that doesn't cause b/372943062
 
@@ -38,14 +168,14 @@ export class SidebarInsightsTab extends HTMLElement {
     }
 
     this.#parsedTrace = data;
-    this.#selectedInsightSetKey = null;
+    this.#selectedInsightSet = null;
 
     if (this.#parsedTrace?.insights) {
       /** Select the first set. Filtering out trivial sets was done back in {@link Trace.Processor.#computeInsightsForInitialTracePeriod} */
-      this.#selectedInsightSetKey = [...this.#parsedTrace.insights.keys()].at(0) ?? null;
+      this.#selectedInsightSet = [...this.#parsedTrace.insights.values()].at(0) ?? null;
     }
 
-    void ComponentHelpers.ScheduledRender.scheduleRender(this, this.#render);
+    this.requestUpdate();
   }
 
   get activeInsight(): ActiveInsight|null {
@@ -56,81 +186,39 @@ export class SidebarInsightsTab extends HTMLElement {
     if (active === this.#activeInsight) {
       return;
     }
+
     this.#activeInsight = active;
 
-    // Only update the insightSetKey if there is an active insight. Otherwise, closing an insight
+    // Only update selectedInsightSet if there is an active insight. Otherwise, closing an insight
     // would also collapse the insight set. Usually the proper insight set is already set because
     // the user has it open already in order for this setter to be called, but insights can also
     // be activated by clicking on a insight chip in the Summary panel, which may require opening
     // a different insight set.
     if (this.#activeInsight) {
-      this.#selectedInsightSetKey = this.#activeInsight.insightSetKey;
+      this.#selectedInsightSet = this.#parsedTrace?.insights?.get(this.#activeInsight.insightSetKey) ?? null;
     }
-    void ComponentHelpers.ScheduledRender.scheduleRender(this, this.#render);
+    this.requestUpdate();
   }
 
-  #insightSetToggled(id: string): void {
-    this.#selectedInsightSetKey = this.#selectedInsightSetKey === id ? null : id;
+  #onInsightSetToggled(insightSet: Trace.Insights.Types.InsightSet): void {
+    this.#selectedInsightSet = this.#selectedInsightSet === insightSet ? null : insightSet;
     // Update the active insight set.
-    if (this.#selectedInsightSetKey !== this.#activeInsight?.insightSetKey) {
-      this.dispatchEvent(new Insights.SidebarInsight.InsightDeactivated());
+    if (this.#selectedInsightSet?.id !== this.#activeInsight?.insightSetKey) {
+      this.element.dispatchEvent(new Insights.SidebarInsight.InsightDeactivated());
     }
-    void ComponentHelpers.ScheduledRender.scheduleRender(this, this.#render);
-  }
-
-  #insightSetHovered(id: string): void {
-    const data = this.#parsedTrace?.insights?.get(id);
-    data && this.dispatchEvent(new Insights.SidebarInsight.InsightSetHovered(data.bounds));
+    this.requestUpdate();
   }
 
-  #insightSetUnhovered(): void {
-    this.dispatchEvent(new Insights.SidebarInsight.InsightSetHovered());
+  #onInsightSetHovered(insightSet: Trace.Insights.Types.InsightSet): void {
+    this.element.dispatchEvent(new Insights.SidebarInsight.InsightSetHovered(insightSet.bounds));
   }
 
-  #onZoomClick(event: Event, id: string): void {
-    event.stopPropagation();
-    const data = this.#parsedTrace?.insights?.get(id);
-    if (!data) {
-      return;
-    }
-    this.dispatchEvent(new Insights.SidebarInsight.InsightSetZoom(data.bounds));
+  #onInsightSetUnhovered(): void {
+    this.element.dispatchEvent(new Insights.SidebarInsight.InsightSetHovered());
   }
 
-  #renderZoomButton(insightSetToggled: boolean): Lit.TemplateResult {
-    const classes = Lit.Directives.classMap({
-      'zoom-icon': true,
-      active: insightSetToggled,
-    });
-
-    // clang-format off
-    return html`
-    <div class=${classes}>
-        <devtools-button .data=${{
-          variant: Buttons.Button.Variant.ICON,
-          iconName: 'center-focus-weak',
-          size: Buttons.Button.Size.SMALL,
-        } as Buttons.Button.ButtonData}
-      ></devtools-button></div>`;
-    // clang-format on
-  }
-
-  #renderDropdownIcon(insightSetToggled: boolean): Lit.TemplateResult {
-    const containerClasses = Lit.Directives.classMap({
-      'dropdown-icon': true,
-      active: insightSetToggled,
-    });
-
-    // clang-format off
-    return html`
-      <div class=${containerClasses}>
-        <devtools-button .data=${{
-          variant: Buttons.Button.Variant.ICON,
-          iconName: 'chevron-right',
-          size: Buttons.Button.Size.SMALL,
-        } as Buttons.Button.ButtonData}
-      ></devtools-button></div>
-    `;
-    // clang-format on
+  #onZoomClick(insightSet: Trace.Insights.Types.InsightSet): void {
+    this.element.dispatchEvent(new Insights.SidebarInsight.InsightSetZoom(insightSet.bounds));
   }
 
   highlightActiveInsight(): void {
@@ -139,76 +227,28 @@ export class SidebarInsightsTab extends HTMLElement {
     }
 
     // Find the right set for this insight via the set key.
-    const set = this.#shadow?.querySelector<UI.Widget.WidgetElement<SidebarSingleInsightSet>>(
+    const set = this.element.shadowRoot?.querySelector<UI.Widget.WidgetElement<SidebarSingleInsightSet>>(
         `[data-insight-set-key="${this.#activeInsight.insightSetKey}"]`);
     set?.getWidget()?.highlightActiveInsight();
   }
 
-  #render(): void {
+  override performUpdate(): void {
     if (!this.#parsedTrace?.insights) {
-      Lit.render(Lit.nothing, this.#shadow, {host: this});
       return;
     }
 
-    const insights = this.#parsedTrace.insights;
-    const hasMultipleInsightSets = insights.size > 1;
-    const labels = Utils.Helpers.createUrlLabels([...insights.values()].map(({url}) => url));
-
-    const contents =
-        // clang-format off
-     html`
-      <style>${sidebarInsightsTabStyles}</style>
-      <div class="insight-sets-wrapper">
-        ${[...insights.values()].map(({id, url}, index) => {
-          const data: SidebarSingleInsightSetData = {
-            insightSetKey: id,
-            activeCategory: this.#selectedCategory,
-            activeInsight: this.#activeInsight,
-            parsedTrace: this.#parsedTrace,
-          };
-
-          const contents = html`
-            <devtools-widget
-              data-insight-set-key=${id}
-              .widgetConfig=${UI.Widget.widgetConfig(SidebarSingleInsightSet, {data})}
-            ></devtools-widget>
-          `;
-
-          if (hasMultipleInsightSets) {
-            return html`<details
-              ?open=${id === this.#selectedInsightSetKey}
-            >
-              <summary
-                @click=${() => this.#insightSetToggled(id)}
-                @mouseenter=${() => this.#insightSetHovered(id)}
-                @mouseleave=${() => this.#insightSetUnhovered()}
-                title=${url.href}>
-                ${this.#renderDropdownIcon(id === this.#selectedInsightSetKey)}
-                <span>${labels[index]}</span>
-                <span class='zoom-button' @click=${(event: Event) => this.#onZoomClick(event, id)}>${this.#renderZoomButton(id === this.#selectedInsightSetKey)}</span>
-              </summary>
-              ${contents}
-            </details>`;
-          }
-
-          return contents;
-        })}
-      </div>
-    `;
-    // clang-format on
-
-    // Insight components contain state, so to prevent insights from previous trace loads breaking things we use the parsedTrace
-    // as a render key.
-    // Note: newer Lit has `keyed`, but we don't have that, so we do it manually. https://lit.dev/docs/templates/directives/#keyed
-    const result = Lit.Directives.repeat([contents], () => this.#parsedTrace, template => template);
-    Lit.render(result, this.#shadow, {host: this});
-  }
-}
-
-customElements.define('devtools-performance-sidebar-insights', SidebarInsightsTab);
-
-declare global {
-  interface HTMLElementTagNameMap {
-    'devtools-performance-sidebar-insights': SidebarInsightsTab;
+    const insightSets = [...this.#parsedTrace.insights.values()];
+    const input: ViewInput = {
+      parsedTrace: this.#parsedTrace,
+      labels: Utils.Helpers.createUrlLabels(insightSets.map(({url}) => url)),
+      activeInsightSet: this.#selectedInsightSet,
+      activeInsight: this.#activeInsight,
+      selectedCategory: this.#selectedCategory,
+      onInsightSetToggled: this.#onInsightSetToggled.bind(this),
+      onInsightSetHovered: this.#onInsightSetHovered.bind(this),
+      onInsightSetUnhovered: this.#onInsightSetUnhovered.bind(this),
+      onZoomClick: this.#onZoomClick.bind(this),
+    };
+    this.#view(input, undefined, this.contentElement);
   }
 }

package/front_end/panels/timeline/components/TimelineSummary.ts

@@ -1,10 +1,9 @@
 // Copyright 2024 The Chromium Authors
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
-/* eslint-disable @devtools/no-lit-render-outside-of-view */
 
 import * as i18n from '../../../core/i18n/i18n.js';
-import type * as Trace from '../../../models/trace/trace.js';
+import * as Buttons from '../../../ui/components/buttons/buttons.js';
 import * as UI from '../../../ui/legacy/legacy.js';
 import * as Lit from '../../../ui/lit/lit.js';
 
@@ -33,74 +32,96 @@ export interface CategoryData {
   title: string;
 }
 
-export interface SummaryTableData {
-  total: number;
+interface ViewInput {
   rangeStart: number;
   rangeEnd: number;
+  total: number;
   categories: CategoryData[];
-  selectedEvents: Trace.Types.Events.Event[];
 }
 
-export class CategorySummary extends HTMLElement {
-  readonly #shadow =
-      UI.UIUtils.createShadowRootWithCoreStyles(this, {cssFile: timelineSummaryStyles, delegatesFocus: undefined});
+type View = (input: ViewInput, output: undefined, target: HTMLElement) => void;
+
+export const CATEGORY_SUMMARY_DEFAULT_VIEW: View = (input, _output, target): void => {
+  // clang-format off
+  render(html`
+        <style>${timelineSummaryStyles}</style>
+        <style>@scope to (devtools-widget > *) { ${UI.inspectorCommonStyles} }</style>
+        <style>@scope to (devtools-widget > *) { ${Buttons.textButtonStyles} }</style>
+        <div class="timeline-summary">
+            <div class="summary-range">${i18nString(UIStrings.rangeSS, {PH1: i18n.TimeUtilities.millisToString(input.rangeStart), PH2: i18n.TimeUtilities.millisToString(input.rangeEnd)})}</div>
+            <div class="category-summary">
+                ${input.categories.map(category => {
+                    return html`
+                        <div class="category-row">
+                        <div class="category-swatch" style="background-color: ${category.color};"></div>
+                        <div class="category-name">${category.title}</div>
+                        <div class="category-value">
+                            ${i18n.TimeUtilities.preciseMillisToString(category.value)}
+                            <div class="background-bar-container">
+                                <div class="background-bar" style='width: ${(category.value * 100 / input.total).toFixed(1)}%;'></div>
+                            </div>
+                        </div>
+                        </div>`;
+                })}
+                <div class="category-row">
+                    <div class="category-swatch"></div>
+                    <div class="category-name">${i18nString(UIStrings.total)}</div>
+                    <div class="category-value">
+                        ${i18n.TimeUtilities.preciseMillisToString(input.total)}
+                        <div class="background-bar-container">
+                            <div class="background-bar"></div>
+                        </div>
+                    </div>
+                </div>
+              </div>
+        </div>
+        </div>
+
+      </div>`, target);
+  // clang-format on
+};
+
+export class CategorySummary extends UI.Widget.Widget {
+  #view: View;
 
   #rangeStart = 0;
   #rangeEnd = 0;
   #total = 0;
   #categories: CategoryData[] = [];
 
-  set data(data: SummaryTableData) {
-    this.#total = data.total;
-    this.#categories = data.categories;
-    this.#rangeStart = data.rangeStart;
-    this.#rangeEnd = data.rangeEnd;
-    this.#render();
+  constructor(view?: View) {
+    super();
+    this.#view = view ?? CATEGORY_SUMMARY_DEFAULT_VIEW;
+    this.requestUpdate();
   }
 
-  #render(): void {
-    // clang-format off
-    const output = html`
-          <div class="timeline-summary">
-              <div class="summary-range">${i18nString(UIStrings.rangeSS, {PH1: i18n.TimeUtilities.millisToString(this.#rangeStart), PH2: i18n.TimeUtilities.millisToString(this.#rangeEnd)})}</div>
-              <div class="category-summary">
-                  ${this.#categories.map(category => {
-                      return html`
-                          <div class="category-row">
-                          <div class="category-swatch" style="background-color: ${category.color};"></div>
-                          <div class="category-name">${category.title}</div>
-                          <div class="category-value">
-                              ${i18n.TimeUtilities.preciseMillisToString(category.value)}
-                              <div class="background-bar-container">
-                                  <div class="background-bar" style='width: ${(category.value * 100 / this.#total).toFixed(1)}%;'></div>
-                              </div>
-                          </div>
-                          </div>`;
-                  })}
-                  <div class="category-row">
-                      <div class="category-swatch"></div>
-                      <div class="category-name">${i18nString(UIStrings.total)}</div>
-                      <div class="category-value">
-                          ${i18n.TimeUtilities.preciseMillisToString(this.#total)}
-                          <div class="background-bar-container">
-                              <div class="background-bar"></div>
-                          </div>
-                      </div>
-                  </div>
-                </div>
-          </div>
-          </div>
+  set total(total: number) {
+    this.#total = total;
+    this.requestUpdate();
+  }
 
-        </div>`;
-    // clang-format on
-    render(output, this.#shadow, {host: this});
+  set rangeStart(rangeStart: number) {
+    this.#rangeStart = rangeStart;
+    this.requestUpdate();
   }
-}
 
-customElements.define('devtools-performance-timeline-summary', CategorySummary);
+  set rangeEnd(rangeEnd: number) {
+    this.#rangeEnd = rangeEnd;
+    this.requestUpdate();
+  }
+
+  set categories(categories: CategoryData[]) {
+    this.#categories = categories;
+    this.requestUpdate();
+  }
 
-declare global {
-  interface HTMLElementTagNameMap {
-    'devtools-performance-timeline-summary': CategorySummary;
+  override performUpdate(): void {
+    const viewInput: ViewInput = {
+      rangeStart: this.#rangeStart,
+      rangeEnd: this.#rangeEnd,
+      total: this.#total,
+      categories: this.#categories,
+    };
+    this.#view(viewInput, undefined, this.contentElement);
   }
 }

package/front_end/panels/timeline/components/insights/BaseInsightComponent.ts

@@ -1,7 +1,6 @@
 // Copyright 2024 The Chromium Authors
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
-/* eslint-disable @devtools/no-imperative-dom-api */
 
 import '../../../../ui/components/markdown_view/markdown_view.js';
 
@@ -70,7 +69,7 @@ interface ViewInput {
   model: InsightModel;
   selected: boolean;
   isAIAssistanceContext: boolean;
-  canShowAskAI: boolean;
+  showAskAI: boolean;
   estimatedSavingsString: string|null;
   estimatedSavingsAriaLabel: string|null;
   renderContent: () => Lit.LitTemplate;
@@ -89,7 +88,7 @@ const DEFAULT_VIEW: View = (input, output, target) => {
     estimatedSavingsString,
     estimatedSavingsAriaLabel,
     isAIAssistanceContext,
-    canShowAskAI,
+    showAskAI,
     dispatchInsightToggle,
     renderContent,
     onHeaderKeyDown,
@@ -122,7 +121,7 @@ const DEFAULT_VIEW: View = (input, output, target) => {
       <div class="insight-body">
         <div class="insight-description">${md(model.description)}</div>
         <div class="insight-content">${content}</div>
-        ${canShowAskAI ? html`
+        ${showAskAI ? html`
           <div class="ask-ai-btn-wrap">
             <devtools-button class="ask-ai"
               .variant=${Buttons.Button.Variant.OUTLINED}
@@ -206,9 +205,6 @@ export abstract class BaseInsightComponent<T extends InsightModel> extends UI.Wi
   #view: View;
   abstract internalName: string;
 
-  // This flag tracks if the Insights AI feature is enabled within Chrome for
-  // the active user.
-  #askAiEnabled = false;
   // Tracks if this component is rendered withing the AI assistance panel.
   // Currently only relevant to GreenDev.
   #isAIAssistanceContext = false;
@@ -246,16 +242,6 @@ export abstract class BaseInsightComponent<T extends InsightModel> extends UI.Wi
     return false;
   }
 
-  override wasShown(): void {
-    super.wasShown();
-
-    // Used for unit test purposes when querying the DOM.
-    this.element.dataset.insightName = this.internalName;
-
-    const {devToolsAiAssistancePerformanceAgent} = Root.Runtime.hostConfig;
-    this.#askAiEnabled = Boolean(devToolsAiAssistancePerformanceAgent?.enabled);
-  }
-
   set isAIAssistanceContext(isAIAssistanceContext: boolean) {
     this.#isAIAssistanceContext = isAIAssistanceContext;
     this.requestUpdate();
@@ -300,11 +286,11 @@ export abstract class BaseInsightComponent<T extends InsightModel> extends UI.Wi
     this.requestUpdate();
   }
 
-  set agentFocus(agentFocus: AIAssistance.AIContext.AgentFocus) {
+  set agentFocus(agentFocus: AIAssistance.AIContext.AgentFocus|null) {
     this.#agentFocus = agentFocus;
   }
 
-  set fieldMetrics(fieldMetrics: Trace.Insights.Common.CrUXFieldMetricResults) {
+  set fieldMetrics(fieldMetrics: Trace.Insights.Common.CrUXFieldMetricResults|null) {
     this.#fieldMetrics = fieldMetrics;
   }
 
@@ -422,7 +408,7 @@ export abstract class BaseInsightComponent<T extends InsightModel> extends UI.Wi
       estimatedSavingsString: this.getEstimatedSavingsString(),
       estimatedSavingsAriaLabel: this.#getEstimatedSavingsAriaLabel(),
       isAIAssistanceContext: this.#isAIAssistanceContext,
-      canShowAskAI: this.#canShowAskAI(),
+      showAskAI: this.#canShowAskAI(),
       dispatchInsightToggle: () => this.#dispatchInsightToggle(),
       renderContent: () => this.renderContent(),
       onHeaderKeyDown: () => this.#onHeaderKeyDown,
@@ -527,14 +513,19 @@ export abstract class BaseInsightComponent<T extends InsightModel> extends UI.Wi
   }
 
   #canShowAskAI(): boolean {
-    if (this.#isAIAssistanceContext) {
+    if (this.#isAIAssistanceContext || !this.hasAskAiSupport()) {
       return false;
     }
 
-    const aiAvailable = Root.Runtime.hostConfig.aidaAvailability?.enterprisePolicyValue !==
-            Root.Runtime.GenAiEnterprisePolicyValue.DISABLE &&
-        this.#askAiEnabled && Root.Runtime.hostConfig.aidaAvailability?.enabled === true;
+    // Check if the Insights AI feature enabled within Chrome for the active user.
+    const {devToolsAiAssistancePerformanceAgent} = Root.Runtime.hostConfig;
+    const askAiEnabled = Boolean(devToolsAiAssistancePerformanceAgent?.enabled);
+    if (!askAiEnabled) {
+      return false;
+    }
 
-    return aiAvailable && this.hasAskAiSupport();
+    const {aidaAvailability} = Root.Runtime.hostConfig;
+    return aidaAvailability?.enterprisePolicyValue !== Root.Runtime.GenAiEnterprisePolicyValue.DISABLE &&
+        aidaAvailability?.enabled === true;
   }
 }