chrome-devtools-frontend 1.0.1672388 → 1.0.1674545

package/front_end/models/trace/handlers/MetaHandler.ts

@@ -8,7 +8,11 @@ import * as Types from '../types/types.js';
 
 import type {FinalizeOptions} from './types.js';
 
-let config: {showAllEvents: boolean};
+let config: Types.Configuration.Configuration = Types.Configuration.defaults();
+
+export function handleUserConfig(userConfig: Types.Configuration.Configuration): void {
+  config = userConfig;
+}
 
 // We track the renderer processes we see in each frame on the way through the trace.
 let rendererProcessesByFrameId: FrameProcessData = new Map();
@@ -62,6 +66,7 @@ let navigationsByNavigationId = new Map<string, Types.Events.NavigationStart>();
 let softNavigationsById = new Map<number, Types.Events.SoftNavigationStart>();
 let finalDisplayUrlByNavigationId = new Map<string, string>();
 let mainFrameNavigations: Types.Events.NavigationStart[] = [];
+let sameDocumentNavigations: Types.Events.DidCommitSameDocumentNavigation[] = [];
 
 // Represents all the threads in the trace, organized by process. This is mostly for internal
 // bookkeeping so that during the finalize pass we can obtain the main and browser thread IDs.
@@ -97,6 +102,7 @@ export function reset(): void {
   finalDisplayUrlByNavigationId = new Map();
   processNames = new Map();
   mainFrameNavigations = [];
+  sameDocumentNavigations = [];
 
   browserProcessId = Types.Events.ProcessID(-1);
   browserThreadId = Types.Events.ThreadID(-1);
@@ -326,7 +332,7 @@ export function handleEvent(event: Types.Events.Event): void {
     return;
   }
 
-  if (Types.Events.isSoftNavigationStart(event)) {
+  if (config.enableSoftNavigation && Types.Events.isSoftNavigationStart(event)) {
     softNavigationsById.set(event.args.context.performanceTimelineNavigationId, event);
   }
 
@@ -346,21 +352,18 @@ export function handleEvent(event: Types.Events.Event): void {
     return;
   }
 
-  // Update `finalDisplayUrlByNavigationId` to reflect history API navigations.
+  // Track history API navigations to compute final display URLs in finalize.
   if (Types.Events.isDidCommitSameDocumentNavigation(event)) {
     if (event.args.render_frame_host.frame_type !== 'PRIMARY_MAIN_FRAME') {
       return;
     }
-
-    const navigation = mainFrameNavigations.at(-1);
-    const key = navigation?.args.data?.navigationId ?? '';
-    finalDisplayUrlByNavigationId.set(key, event.args.url);
+    sameDocumentNavigations.push(event);
     return;
   }
 }
 
 export async function finalize(options?: FinalizeOptions): Promise<void> {
-  config = {showAllEvents: Boolean(options?.showAllEvents)};
+  config.showAllEvents = Boolean(options?.showAllEvents);
 
   // We try to set the minimum time by finding the event with the smallest
   // timestamp. However, if we also got a timestamp from the
@@ -440,10 +443,44 @@ export async function finalize(options?: FinalizeOptions): Promise<void> {
       mainFrameURL = firstMainFrameNav.args.data.documentLoaderURL;
     }
   }
+
+  // Compute final display URLs for each hard navigation.
+  // A hard navigation's final URL is updated by history API navigations (e.g. pushState).
+  // However, if a soft navigation occurs, subsequent history API navigations belong to the
+  // soft navigation and should not update the hard navigation's URL.
+  const softNavs = Array.from(softNavigationsById.values()).sort((a, b) => a.ts - b.ts);
+
+  // Create an array of intervals for each hard navigation.
+  const hardNavs = [...mainFrameNavigations];
+
+  // Add a faux navigation to cover the time before the first hard navigation, or the whole trace if there are none.
+  const intervals = [
+    {minTs: 0, maxTs: hardNavs.length > 0 ? hardNavs[0].ts : Infinity, key: ''},
+    ...hardNavs.map((nav, i) => ({
+                      minTs: nav.ts,
+                      maxTs: i + 1 < hardNavs.length ? hardNavs[i + 1].ts : Infinity,
+                      key: nav.args.data?.navigationId ?? '',
+                    })),
+  ];
+
+  for (const {minTs, maxTs, key} of intervals) {
+    // Find the first soft navigation within this hard navigation's time window.
+    const firstSoftNav = softNavs.find(sn => sn.ts >= minTs && sn.ts < maxTs);
+    // If a soft navigation occurred, we only consider history events before it.
+    const effectiveMaxTs = firstSoftNav ? firstSoftNav.ts : maxTs;
+
+    // Find the last history API navigation in the effective window.
+    const historyEvents = sameDocumentNavigations.filter(hn => hn.ts >= minTs && hn.ts < effectiveMaxTs);
+
+    const lastHistoryEvent = historyEvents.at(-1);
+    if (lastHistoryEvent) {
+      finalDisplayUrlByNavigationId.set(key, lastHistoryEvent.args.url);
+    }
+  }
 }
 
 export interface MetaHandlerData {
-  config: {showAllEvents: boolean};
+  config: Types.Configuration.Configuration;
   traceIsGeneric: boolean;
   traceBounds: Types.Timing.TraceWindowMicro;
   browserProcessId: Types.Events.ProcessID;

package/front_end/models/trace/handlers/PageLoadMetricsHandler.ts

@@ -53,6 +53,11 @@ export function reset(): void {
   metaCharsetCheckEventsArray = [];
 }
 
+let enableSoftNavigation = true;
+export function handleUserConfig(userConfig: Types.Configuration.Configuration): void {
+  enableSoftNavigation = userConfig.enableSoftNavigation;
+}
+
 let pageLoadEventsArray: Types.Events.PageLoadEvent[] = [];
 
 // Once we've found the LCP events in the trace we want to fetch their DOM Node
@@ -74,6 +79,12 @@ export function handleEvent(event: Types.Events.Event): void {
   if (!Types.Events.eventIsPageLoadEvent(event)) {
     return;
   }
+  if (!enableSoftNavigation &&
+      (Types.Events.isSoftNavigationStart(event) ||
+       event.name === Types.Events.Name.MARK_LCP_CANDIDATE_FOR_SOFT_NAVIGATION ||
+       Types.Events.isSoftFirstContentfulPaint(event))) {
+    return;
+  }
   pageLoadEventsArray.push(event);
 
   // A soft nav entry includes the Soft FCP details but we want to process both

package/front_end/models/trace/insights/CLSCulprits.ts

@@ -599,8 +599,7 @@ export function generateInsight(data: Handlers.Types.HandlerData, context: Insig
   const domLoadingEvents = data.LayoutShifts.domLoadingEvents.filter(isWithinContext);
   const unsizedImageEvents = data.LayoutShifts.layoutImageUnsizedEvents.filter(isWithinContext);
 
-  const clusterKey = context.navigation ? context.navigationId : Types.Events.NO_NAVIGATION;
-  const clusters = data.LayoutShifts.clustersByNavigationId.get(clusterKey) ?? [];
+  const clusters = data.LayoutShifts.clusters.filter(isWithinContext);
   const clustersByScore = clusters.toSorted((a, b) => b.clusterCumulativeScore - a.clusterCumulativeScore);
   const worstCluster = clustersByScore.at(0);
   const layoutShifts = clusters.flatMap(cluster => cluster.events);

package/front_end/models/trace/insights/CharacterSet.ts

@@ -127,7 +127,7 @@ function metaCharsetLabel(disposition: Types.Events.MetaCharsetDisposition|undef
 
 export function generateInsight(
     data: Handlers.Types.HandlerData, context: InsightSetContext): CharacterSetInsightModel {
-  if (!context.navigation) {
+  if (!context.navigation || !('navigationId' in context)) {
     return finalize({});
   }
 

package/front_end/models/trace/insights/Common.ts

@@ -160,7 +160,26 @@ export function getFieldMetricsForInsightSet(
     return null;
   }
 
-  const pageResult = getPageResult(cruxFieldData, insightSet.url.href, insightSet.url.origin, scope);
+  let pageResult = getPageResult(cruxFieldData, insightSet.url.href, insightSet.url.origin, scope);
+
+  if (!pageResult && insightSet.navigation && Types.Events.isSoftNavigationStart(insightSet.navigation)) {
+    pageResult = cruxFieldData.find(result => {
+      const key = scope ? result[`${scope.pageScope}-${scope.deviceScope}`]?.record.key :
+                          (result['url-ALL'] || result['origin-ALL'])?.record.key;
+      if (!key) {
+        return false;
+      }
+      if (key.url) {
+        try {
+          return new URL(key.url).origin === insightSet.url.origin;
+        } catch {
+          return false;
+        }
+      }
+      return key.origin === insightSet.url.origin;
+    });
+  }
+
   if (!pageResult) {
     return null;
   }
@@ -266,7 +285,7 @@ function estimateSavingsWithGraphs(
  */
 export function metricSavingsForWastedBytes(
     wastedBytesByRequestId: Map<string, number>, context: InsightSetContext): MetricSavings|undefined {
-  if (!context.navigation || !context.lantern) {
+  if (!context.navigation || !('lantern' in context) || !context.lantern) {
     return;
   }
 

package/front_end/models/trace/insights/DocumentLatency.ts

@@ -191,7 +191,7 @@ function finalize(partialModel: PartialInsightModel<DocumentLatencyInsightModel>
 
 export function generateInsight(
     data: Handlers.Types.HandlerData, context: InsightSetContext): DocumentLatencyInsightModel {
-  if (!context.navigation) {
+  if (!context.navigation || !('navigationId' in context)) {
     return finalize({});
   }
 

package/front_end/models/trace/insights/LCPBreakdown.ts

@@ -110,10 +110,10 @@ function anyValuesNaN(...values: number[]): boolean {
  * them to be missing on newer traces, but old trace files may lack some of the
  * data we rely on, so we want to handle that case.
  */
-function determineSubparts(
-    nav: Types.Events.NavigationStart, docRequest: Types.Events.SyntheticNetworkRequest,
-    lcpEvent: Types.Events.AnyLargestContentfulPaintCandidate,
-    lcpRequest: Types.Events.SyntheticNetworkRequest|undefined): LCPSubparts|null {
+function determineSubparts(nav: Types.Events.NavigationStart|Types.Events.SoftNavigationStart,
+                           docRequest: Types.Events.SyntheticNetworkRequest,
+                           lcpEvent: Types.Events.AnyLargestContentfulPaintCandidate,
+                           lcpRequest: Types.Events.SyntheticNetworkRequest|undefined): LCPSubparts|null {
   const firstDocByteTs = calculateDocFirstByteTs(docRequest);
   if (firstDocByteTs === null) {
     return null;
@@ -232,9 +232,10 @@ export function generateInsight(
   const lcpMs = Helpers.Timing.microToMilli(metricScore.timing);
   // This helps position things on the timeline's UI accurately for a trace.
   const lcpTs = metricScore.event?.ts ? Helpers.Timing.microToMilli(metricScore.event?.ts) : undefined;
-  const lcpRequest = data.LargestImagePaint.lcpRequestByNavigationId.get(context.navigationId);
+  const lcpRequest =
+      'navigationId' in context ? data.LargestImagePaint.lcpRequestByNavigationId.get(context.navigationId) : undefined;
 
-  const docRequest = networkRequests.byId.get(context.navigationId);
+  const docRequest = 'navigationId' in context ? networkRequests.byId.get(context.navigationId) : undefined;
   if (!docRequest) {
     return finalize({lcpMs, lcpTs, lcpEvent, lcpRequest, warnings: [InsightWarning.NO_DOCUMENT_REQUEST]});
   }

package/front_end/models/trace/insights/LCPDiscovery.ts

@@ -101,7 +101,7 @@ function finalize(partialModel: PartialInsightModel<LCPDiscoveryInsightModel>):
 
 export function generateInsight(
     data: Handlers.Types.HandlerData, context: InsightSetContext): LCPDiscoveryInsightModel {
-  if (!context.navigation) {
+  if (!context.navigation || !('navigationId' in context)) {
     return finalize({});
   }
 

package/front_end/models/trace/insights/ModernHTTP.ts

@@ -7,7 +7,7 @@ import * as Platform from '../../../core/platform/platform.js';
 import * as Handlers from '../handlers/handlers.js';
 import * as Helpers from '../helpers/helpers.js';
 import type * as Lantern from '../lantern/lantern.js';
-import type * as Types from '../types/types.js';
+import * as Types from '../types/types.js';
 
 import {
   InsightCategory,
@@ -199,7 +199,7 @@ function computeWasteWithGraph(
 
 function computeMetricSavings(
     http1Requests: Types.Events.SyntheticNetworkRequest[], context: InsightSetContext): MetricSavings|undefined {
-  if (!context.navigation || !context.lantern) {
+  if (!context.navigation || !('lantern' in context) || !context.lantern) {
     return;
   }
 
@@ -234,7 +234,11 @@ export function generateInsight(data: Handlers.Types.HandlerData, context: Insig
   const contextRequests = data.NetworkRequests.byTime.filter(isWithinContext);
 
   const entityMappings = data.NetworkRequests.entityMappings;
-  const firstPartyUrl = context.navigation?.args.data?.documentLoaderURL ?? data.Meta.mainFrameURL;
+  let firstPartyUrl = data.Meta.mainFrameURL;
+  if (context.navigation && !Types.Events.isSoftNavigationStart(context.navigation)) {
+    firstPartyUrl = context.navigation.args.data?.documentLoaderURL ?? firstPartyUrl;
+  }
+
   const firstPartyEntity = Handlers.Helpers.getEntityForUrl(firstPartyUrl, entityMappings);
   const http1Requests = determineHttp1Requests(contextRequests, entityMappings, firstPartyEntity ?? null);
 

package/front_end/models/trace/insights/NetworkDependencyTree.ts

@@ -674,7 +674,7 @@ export function isNetworkDependencyTreeInsight(model: InsightModel): model is Ne
 
 export function generateInsight(
     data: Handlers.Types.HandlerData, context: InsightSetContext): NetworkDependencyTreeInsightModel {
-  if (!context.navigation) {
+  if (!context.navigation || !('navigationId' in context)) {
     return finalize({
       rootNodes: [],
       maxTime: 0 as Types.Timing.Micro,

package/front_end/models/trace/insights/RenderBlocking.ts

@@ -178,7 +178,7 @@ function finalize(partialModel: PartialInsightModel<RenderBlockingInsightModel>)
 
 export function generateInsight(
     data: Handlers.Types.HandlerData, context: InsightSetContext): RenderBlockingInsightModel {
-  if (!context.navigation) {
+  if (!context.navigation || !('navigationId' in context)) {
     return finalize({
       renderBlockingRequests: [],
     });

package/front_end/models/trace/insights/ThirdParties.ts

@@ -6,7 +6,7 @@ import * as i18n from '../../../core/i18n/i18n.js';
 import * as ThirdPartyWeb from '../../../third_party/third-party-web/third-party-web.js';
 import * as Extras from '../extras/extras.js';
 import * as Handlers from '../handlers/handlers.js';
-import type * as Types from '../types/types.js';
+import * as Types from '../types/types.js';
 
 import {
   InsightCategory,
@@ -82,7 +82,10 @@ export function generateInsight(
     data: Handlers.Types.HandlerData, context: InsightSetContext): ThirdPartiesInsightModel {
   const entitySummaries = Extras.ThirdParties.summarizeByThirdParty(data as Handlers.Types.HandlerData, context.bounds);
 
-  const firstPartyUrl = context.navigation?.args.data?.documentLoaderURL ?? data.Meta.mainFrameURL;
+  let firstPartyUrl = data.Meta.mainFrameURL;
+  if (context.navigation && !Types.Events.isSoftNavigationStart(context.navigation)) {
+    firstPartyUrl = context.navigation.args.data?.documentLoaderURL ?? firstPartyUrl;
+  }
   const firstPartyEntity = ThirdPartyWeb.ThirdPartyWeb.getEntity(firstPartyUrl) ||
       Handlers.Helpers.makeUpEntity(data.Renderer.entityMappings.createdEntityCache, firstPartyUrl);
 

package/front_end/models/trace/insights/types.ts

@@ -11,7 +11,8 @@ import type * as Models from './Models.js';
 /**
  * Context for the portion of the trace an insight should look at.
  */
-export type InsightSetContext = InsightSetContextWithoutNavigation|InsightSetContextWithNavigation;
+export type InsightSetContext =
+    InsightSetContextWithoutNavigation|InsightSetContextWithNavigation|InsightSetContextWithSoftNavigation;
 
 export interface InsightSetContextWithoutNavigation {
   options: Types.Configuration.ParseOptions;
@@ -29,6 +30,13 @@ export interface InsightSetContextWithNavigation {
   lantern?: LanternContext;
 }
 
+export interface InsightSetContextWithSoftNavigation {
+  options: Types.Configuration.ParseOptions;
+  bounds: Types.Timing.TraceWindowMicro;
+  frameId: string;
+  navigation: Types.Events.SoftNavigationStart;
+}
+
 export interface LanternContext {
   requests: Array<Lantern.Types.NetworkRequest<Types.Events.SyntheticNetworkRequest>>;
   graph: Lantern.Graph.Node<Types.Events.SyntheticNetworkRequest>;
@@ -93,10 +101,7 @@ export type InsightModel<UIStrings extends Record<string, string> = Record<strin
        */
       wastedBytes?: number,
       frameId?: string,
-      /**
-       * If this insight is attached to a navigation, this stores its ID.
-       */
-      navigation?: Types.Events.NavigationStart,
+      navigation?: Types.Events.NavigationStart | Types.Events.SoftNavigationStart,
       /** This is lazily-generated because some insights may create many overlays. */
       createOverlays?: () => Types.Overlays.Overlay[],
     };
@@ -120,7 +125,7 @@ export interface InsightSet {
   model: InsightModels;
   /** Contains errors for all insights that had an internal error. */
   modelErrors: InsightModelErrors;
-  navigation?: Types.Events.NavigationStart;
+  navigation?: Types.Events.NavigationStart|Types.Events.SoftNavigationStart;
 }
 
 /**

package/front_end/models/trace/types/Configuration.ts

@@ -42,6 +42,10 @@ export interface Configuration {
    * from the model, but the contents will be empty.
    */
   enableAnimationsFrameHandler: boolean;
+  /**
+   * Whether soft-navs are used in the model/insights.
+   */
+  enableSoftNavigation: boolean;
 }
 
 export const defaults = (): Configuration => ({
@@ -49,6 +53,7 @@ export const defaults = (): Configuration => ({
   debugMode: false,
   maxInvalidationEventsPerEvent: 20,
   enableAnimationsFrameHandler: false,
+  enableSoftNavigation: true,
 });
 
 /**

package/front_end/panels/ai_assistance/AiAssistancePanel.ts

@@ -594,7 +594,7 @@ export class AiAssistancePanel extends UI.Panel.Panel {
   #conversationSummary?: AiAssistanceModel.ConversationSummary.ConversationSummary;
   #viewOutput: PanelViewOutput = {};
   #serverSideLoggingEnabled = isAiAssistanceServerSideLoggingEnabled();
-  #aiAssistanceEnabledSetting: Common.Settings.Setting<boolean>|undefined;
+  #aiAssistanceEnabledSetting: AiAssistanceModel.AiSetting.AiSetting<boolean>;
   #changeManager = new AiAssistanceModel.ChangeManager.ChangeManager();
   #mutex = new Common.Mutex.Mutex();
 
@@ -860,13 +860,12 @@ export class AiAssistancePanel extends UI.Panel.Panel {
     this.requestUpdate();
   }
 
-  #getAiAssistanceEnabledSetting(): Common.Settings.Setting<boolean>|undefined {
-    try {
-      return Common.Settings.Settings.instance().moduleSetting('ai-assistance-enabled') as
-          Common.Settings.Setting<boolean>;
-    } catch {
-      return;
-    }
+  #getAiAssistanceEnabledSetting(): AiAssistanceModel.AiSetting.AiSetting<boolean> {
+    return new AiAssistanceModel.AiSetting.AiSetting(
+        AiAssistanceModel.AiUtils.aiAssistanceEnabledSettingDescriptor,
+        Host.AidaClient.HostConfigTracker.instance(),
+        Common.Settings.Settings.instance(),
+    );
   }
 
   static async instance(opts: {
@@ -1061,7 +1060,8 @@ export class AiAssistancePanel extends UI.Panel.Panel {
 
     AiAssistanceModel.AiHistoryStorage.AiHistoryStorage.instance().addEventListener(
         AiAssistanceModel.AiHistoryStorage.Events.HISTORY_DELETED, this.#onHistoryDeleted, this);
-    this.#aiAssistanceEnabledSetting?.addChangeListener(this.requestUpdate, this);
+    this.#aiAssistanceEnabledSetting.addEventListener(AiAssistanceModel.AiSetting.Events.CHANGED, this.requestUpdate,
+                                                      this);
     Host.AidaClient.HostConfigTracker.instance().addEventListener(
         Host.AidaClient.Events.AIDA_AVAILABILITY_CHANGED, this.#handleAidaAvailabilityChange);
     const initialAvailability = Host.AidaClient.HostConfigTracker.instance().aidaAvailability;
@@ -1107,7 +1107,8 @@ export class AiAssistancePanel extends UI.Panel.Panel {
     super.willHide();
     AiAssistanceModel.AiHistoryStorage.AiHistoryStorage.instance().removeEventListener(
         AiAssistanceModel.AiHistoryStorage.Events.HISTORY_DELETED, this.#onHistoryDeleted, this);
-    this.#aiAssistanceEnabledSetting?.removeChangeListener(this.requestUpdate, this);
+    this.#aiAssistanceEnabledSetting.removeEventListener(AiAssistanceModel.AiSetting.Events.CHANGED, this.requestUpdate,
+                                                         this);
     Host.AidaClient.HostConfigTracker.instance().removeEventListener(
         Host.AidaClient.Events.AIDA_AVAILABILITY_CHANGED, this.#handleAidaAvailabilityChange);
     this.#toggleSearchElementAction?.removeEventListener(

package/front_end/panels/ai_assistance/ai_assistance-meta.ts

@@ -4,9 +4,10 @@
 
 import * as Common from '../../core/common/common.js';
 import * as i18n from '../../core/i18n/i18n.js';
-import type * as Platform from '../../core/platform/platform.js';
 import * as Root from '../../core/root/root.js';
+import * as AiAssistanceModel from '../../models/ai_assistance/ai_assistance.js';
 import * as UI from '../../ui/legacy/legacy.js';
+import * as SettingUIRegistration from '../../ui/settings/settings.js';
 
 import type * as AiAssistance from './ai_assistance.js';
 
@@ -50,21 +51,6 @@ const UIStrings = {
    * @description Text of a context menu item to redirect to the Gemini panel with the current context.
    */
   debugWithGemini: 'Debug with Gemini',
-  /**
-   * @description Message shown to the user if the DevTools locale is not
-   * supported.
-   */
-  wrongLocale: 'To use this feature, set your language preference to English in DevTools settings.',
-  /**
-   * @description Message shown to the user if the user's region is not
-   * supported.
-   */
-  geoRestricted: 'This feature is unavailable in your region.',
-  /**
-   * @description Message shown to the user if the enterprise policy does
-   * not allow this feature.
-   */
-  policyRestricted: 'This setting is managed by your administrator.',
 } as const;
 
 const str_ = i18n.i18n.registerUIStrings('panels/ai_assistance/ai_assistance-meta.ts', UIStrings);
@@ -76,13 +62,6 @@ function i18nAiBrandedString(gemini: string, assistance: string) {
   return () => Root.Runtime.hostConfig.devToolsGeminiRebranding?.enabled ? i18nString(gemini) : i18nString(assistance);
 }
 
-const setting = 'ai-assistance-enabled';
-
-function isLocaleRestricted(): boolean {
-  const devtoolsLocale = i18n.DevToolsLocale.DevToolsLocale.instance();
-  return !devtoolsLocale.locale.startsWith('en-');
-}
-
 function isGeoRestricted(config?: Root.Runtime.HostConfig): boolean {
   return config?.aidaAvailability?.blockedByGeo === true;
 }
@@ -140,30 +119,9 @@ UI.ViewManager.registerViewExtension({
   },
 });
 
-Common.Settings.registerSettingExtension({
+SettingUIRegistration.SettingUIRegistration.register(AiAssistanceModel.AiUtils.aiAssistanceEnabledSettingDescriptor, {
   category: Common.Settings.SettingCategory.AI,
-  settingName: setting,
-  settingType: Common.Settings.SettingType.BOOLEAN,
   title: i18nAiBrandedString(UIStrings.enableGemini, UIStrings.enableAiAssistance),
-  defaultValue: false,
-  reloadRequired: false,
-  condition: isAnyFeatureAvailable,
-  disabledCondition: config => {
-    const reasons: Platform.UIString.LocalizedString[] = [];
-    if (isGeoRestricted(config)) {
-      reasons.push(i18nString(UIStrings.geoRestricted));
-    }
-    if (isPolicyRestricted(config)) {
-      reasons.push(i18nString(UIStrings.policyRestricted));
-    }
-    if (isLocaleRestricted()) {
-      reasons.push(i18nString(UIStrings.wrongLocale));
-    }
-    if (reasons.length > 0) {
-      return {disabled: true, reasons};
-    }
-    return {disabled: false};
-  },
 });
 
 UI.ActionRegistration.registerActionExtension({