chrome-devtools-frontend 1.0.1709383 → 1.0.1713292

package/front_end/models/ai_assistance/data_formatters/LighthouseFormatter.ts

@@ -36,17 +36,37 @@ export class LighthouseFormatter {
   }
 
   /**
-   * Formats a Lighthouse report for an AI Agent. If categoryId is 'all', includes
-   * the overall summary followed by each category's audits. Otherwise, returns audits
-   * for the specified category.
+   * Returns the title and score of every failing audit (score < 90), grouped by category.
+   * Descriptions and details tables are left out to keep the prompt small. Each category
+   * heading includes its category ID so the agent can request the full audit details.
+   */
+  failingAuditsSummary(report: LHModel.ReporterTypes.ReportJSON): string {
+    const lines: string[] = [];
+    lines.push('## Failing audits');
+    for (const [categoryId, category] of Object.entries(report.categories)) {
+      lines.push('');
+      lines.push(`### ${category.title} (categoryId: "${categoryId}")`);
+      const failingAudits = this.#findFailingAudits(report, category);
+      if (failingAudits.length === 0) {
+        lines.push('- No failing audits.');
+        continue;
+      }
+      for (const audit of failingAudits) {
+        lines.push(`- ${audit.title}: ${this.#formatScore(audit.score)}`);
+      }
+    }
+    return lines.join('\n');
+  }
+
+  /**
+   * Formats a Lighthouse report for an AI Agent. If categoryId is 'all', returns
+   * each category's audits. Otherwise, returns audits for the specified category.
+   * The output does not include the report summary, because the conversation
+   * context already sends it.
    */
   formatReport(report: LHModel.ReporterTypes.ReportJSON, categoryId: LighthouseCategoryArg): string {
     if (categoryId === 'all') {
-      const sections: string[] = [this.summary(report)];
-      for (const category of Object.values(report.categories)) {
-        sections.push(this.audits(report, category));
-      }
-      return sections.join('\n\n');
+      return Object.values(report.categories).map(category => this.audits(report, category)).join('\n\n');
     }
     return this.audits(report, categoryId);
   }
@@ -69,10 +89,7 @@ export class LighthouseFormatter {
     }
     lines.push('');
 
-    const failingAudits = category.auditRefs.filter(ref => {
-      const audit = report.audits[ref.id];
-      return audit && audit.score !== null && audit.score < 0.9;
-    });
+    const failingAudits = this.#findFailingAudits(report, category);
 
     if (failingAudits.length === 0) {
       lines.push('All audits in this category passed (score >= 90).');
@@ -80,13 +97,8 @@ export class LighthouseFormatter {
     }
 
     lines.push('The following audits in this category have a score below 90 and may need attention:');
-    for (const ref of failingAudits) {
-      const audit = report.audits[ref.id];
-      if (!audit) {
-        continue;
-      }
-      const score = audit.score !== null ? Math.round(audit.score * 100) : 'n/a';
-      let line = `- **${audit.title}**: ${score}`;
+    for (const audit of failingAudits) {
+      let line = `- **${audit.title}**: ${this.#formatScore(audit.score)}`;
       if (audit.displayValue) {
         line += ` (${audit.displayValue})`;
       }
@@ -104,6 +116,22 @@ export class LighthouseFormatter {
     return lines.join('\n');
   }
 
+  /**
+   * Returns the audits in a category that scored below 90, in the category's order.
+   * Audits without a score (such as informative or manual audits) cannot fail, so they
+   * are excluded.
+   */
+  #findFailingAudits(report: LHModel.ReporterTypes.ReportJSON, category: LHModel.ReporterTypes.CategoryJSON):
+      Array<LHModel.ReporterTypes.AuditResultJSON&{score: number}> {
+    return category.auditRefs.map(ref => report.audits[ref.id])
+        .filter((audit): audit is LHModel.ReporterTypes.AuditResultJSON&{score: number} =>
+                    Boolean(audit) && audit.score !== null && audit.score < 0.9);
+  }
+
+  #formatScore(score: number): number {
+    return Math.round(score * 100);
+  }
+
   #formatDetails(details: LHModel.ReporterTypes.DetailsJSON): string {
     switch (details.type) {
       case 'table': {

package/front_end/models/ai_assistance/skills/accessibility.md

@@ -19,6 +19,7 @@ You are an expert accessibility debugging assistant.
    - Use `getStyles` on the backend node ID to inspect layout, color contrast, or font properties.
 
 2. **Lighthouse Accessibility Audits (`getLighthouseAudits` & `runLighthouse`)**:
+   - The Lighthouse report context contains only category scores and failing audit titles. To answer questions about failing accessibility audits, you must first call `getLighthouseAudits` with `categoryId: 'accessibility'` to fetch the full details. Never reply using only the initial summary.
    - If the user asks for a Lighthouse audit or report (such as recording a report or checking accessibility scores), or if audits are needed:
      - If an active Lighthouse report context already exists and no fresh audit is requested, query it via `getLighthouseAudits` with `categoryId: 'accessibility'`.
      - If no active report exists or a fresh audit is requested, use `runLighthouse` with `categoryId: 'accessibility'`:

package/front_end/models/ai_assistance/skills/lighthouse.md

@@ -12,6 +12,7 @@ Your role is to evaluate websites using Lighthouse audits across performance, ac
 # Tools & Workflow
 
 1. **Lighthouse Audits (`runLighthouse` & `getLighthouseAudits`)**:
+   - The Lighthouse report context contains only category scores and failing audit titles. To answer questions about failing audits or how to improve a score, you must first call `getLighthouseAudits` for the relevant category to fetch the full details. Never reply using only the initial summary.
    - If an active Lighthouse report context already exists and no fresh audit is requested:
      - To inspect the entire report or multiple categories, call `getLighthouseAudits` with `categoryId: 'all'`.
      - For a specific category, call `getLighthouseAudits` with the corresponding category ID (e.g. `'performance'`, `'accessibility'`, `'best-practices'`, `'seo'`).

package/front_end/models/ai_assistance/skills/styling.md

@@ -39,3 +39,4 @@ The user selected a DOM element in DevTools and asks a query about the element o
 
 * Meticulously investigate all potential causes for the observed behavior before concluding. Inspect parents, siblings, children, and overlapping elements where relevant.
 * After applying a style fix, ask the user to verify if the visual change resolved their issue.
+* If the target element is in a cross-origin iframe, stop and ask the user to select it in the Elements panel and start a new chat with that context.

package/front_end/models/ai_assistance/tools/ListSources.ts

@@ -43,10 +43,13 @@ export class ListSourcesTool implements
   static lastSourceId = 0;
   static uiSourceCodeId: WeakMap<Workspace.UISourceCode.UISourceCode, number> =
       new WeakMap<Workspace.UISourceCode.UISourceCode, number>();
+  static idToUiSourceCode: Map<number, Workspace.UISourceCode.UISourceCode> =
+      new Map<number, Workspace.UISourceCode.UISourceCode>();
 
   static reset(): void {
     ListSourcesTool.lastSourceId = 0;
     ListSourcesTool.uiSourceCodeId = new WeakMap();
+    ListSourcesTool.idToUiSourceCode = new Map();
   }
 
   static getUISourceCodes(
@@ -78,7 +81,9 @@ export class ListSourcesTool implements
         if (!uiSourceCodes.get(url) || uiSourceCode.contentType().isFromSourceMap()) {
           uiSourceCodes.set(url, uiSourceCode);
           if (!ListSourcesTool.uiSourceCodeId.has(uiSourceCode)) {
-            ListSourcesTool.uiSourceCodeId.set(uiSourceCode, ++ListSourcesTool.lastSourceId);
+            const id = ++ListSourcesTool.lastSourceId;
+            ListSourcesTool.uiSourceCodeId.set(uiSourceCode, id);
+            ListSourcesTool.idToUiSourceCode.set(id, uiSourceCode);
           }
         }
       }
@@ -90,14 +95,15 @@ export class ListSourcesTool implements
   static getSourceById(
       id: number,
       originLock: OriginLockState,
-      // eslint-disable-next-line @devtools/no-instance-of-migrated-singletons
-      workspace: Workspace.Workspace.WorkspaceImpl = Workspace.Workspace.WorkspaceImpl.instance(),
       ): Workspace.UISourceCode.UISourceCode|undefined {
     if (!Number.isInteger(id) || id <= 0) {
       return undefined;
     }
-    return ListSourcesTool.getUISourceCodes(originLock, workspace)
-        .find(file => ListSourcesTool.uiSourceCodeId.get(file) === id);
+    const file = ListSourcesTool.idToUiSourceCode.get(id);
+    if (!file || !isOriginAllowedByLock(originLock, file.securityOrigin())) {
+      return undefined;
+    }
+    return file;
   }
 
   readonly parameters: Host.AidaClient.FunctionObjectParam<never> = {

package/front_end/models/bindings/DebuggerWorkspaceBinding.ts

@@ -144,6 +144,10 @@ export class DebuggerWorkspaceBinding implements SDK.TargetManager.SDKModelObser
     }
     ranges = compilerMapping.getLocationRangesForSameSourceLocation(rawLocation);
     ranges = ranges.filter(range => contained(rawLocation, range));
+    if (mode === SDK.DebuggerModel.StepMode.STEP_OVER) {
+      // Step over functions inlined by the compiler (from encoded source map scopes).
+      ranges = ranges.concat(SourceMapStepping.inlinedCalleeRanges(callFrame));
+    }
     return ranges;
   }
 
@@ -516,7 +520,7 @@ export class DebuggerWorkspaceBinding implements SDK.TargetManager.SDKModelObser
           null :
           await this.computeAutoStep(SDK.DebuggerModel.StepMode.STEP_OVER, callFrames);
     }
-    return null;
+    return await SourceMapStepping.nextAutoStep(debuggerPausedDetails, context, this.computeAutoStep.bind(this));
   }
 
   async #shouldPauseInWasm(debuggerPausedDetails: SDK.DebuggerModel.DebuggerPausedDetails,

package/front_end/models/bindings/SourceMapStepping.ts

@@ -5,11 +5,17 @@
 /**
  * @file Stepping through code with encoded source map scopes: inlined functions, outlined functions and unmapped code.
  *
- * Mirrors the Wasm auto-stepping: user steps get a skip list (see `DebuggerWorkspaceBinding.computeAutoStep`).
+ * Mirrors the Wasm auto-stepping: user steps get a skip list (see `DebuggerWorkspaceBinding.computeAutoStep`), and
+ * pauses that don't correspond to a "logical" step are continued automatically ({@link nextAutoStep}).
+ *
+ * Outlined code is code of an authored function that the compiler moved into a separate generated function marked as
+ * "hidden". A run of outlined frames on top of the stack is one logical frame together with the frame below them (the
+ * "owner"), see {@link logicalDepth}.
  */
 
 import * as Root from '../../core/root/root.js';
-import type * as SDK from '../../core/sdk/sdk.js';
+import * as SDK from '../../core/sdk/sdk.js';
+import * as Protocol from '../../generated/protocol.js';
 
 interface ScopedPosition {
   sourceMap: SDK.SourceMap.SourceMap;
@@ -19,14 +25,17 @@ interface ScopedPosition {
   column: number;
 }
 
-/** @returns the script-relative position of {@link frame}, if it's in a script with encoded source map scopes. */
+/**
+ * @returns the script-relative position of {@link frame}, if it's in a script with encoded source map scopes.
+ *          Scopes without any mappings are ignored: every position would count as unmapped (see {@link isUnmapped}).
+ */
 function scopedPosition(frame: SDK.DebuggerModel.CallFrame): ScopedPosition|null {
   if (!Root.Runtime.hostConfig.devToolsSourceMapScopesInSourcesPanel?.enabled) {
     return null;
   }
   const script = frame.script;
   const sourceMap = script.sourceMap();
-  if (!sourceMap?.hasEncodedScopeInfo()) {
+  if (!sourceMap?.hasEncodedScopeInfo() || sourceMap.mappings().length === 0) {
     return null;
   }
   const {lineNumber, columnNumber} = script.rawLocationToRelativeLocation(frame.location());
@@ -42,9 +51,99 @@ function toLocationRanges({script}: ScopedPosition,
   return ranges.map(({start, end}) => ({start: toLocation(start), end: toLocation(end)}));
 }
 
+export function isScopedFrame(frame: SDK.DebuggerModel.CallFrame): boolean {
+  return scopedPosition(frame) !== null;
+}
+
+/** @returns the number of leading frames that are outlined parts of the (logical) frame below them. */
+function outlinedPrefixLength(callFrames: readonly SDK.DebuggerModel.CallFrame[]): number {
+  let length = 0;
+  for (const frame of callFrames) {
+    const position = scopedPosition(frame);
+    if (!position ||
+        position.sourceMap.translateRawFrame(position.line, position.column)?.kind !==
+            SDK.SourceMapScopesInfo.GeneratedFrameKind.OUTLINED) {
+      break;
+    }
+    ++length;
+  }
+  return length;
+}
+
+/**
+ * @returns the stack depth that doesn't count outlined frames on top of the stack. Entering or leaving an outlined
+ *          part of a function doesn't change it, while calling a function increases it.
+ */
+export function logicalDepth(callFrames: readonly SDK.DebuggerModel.CallFrame[]): number {
+  return callFrames.length - outlinedPrefixLength(callFrames);
+}
+
+/** @returns true iff {@link frame} has encoded scopes, and its position is not mapped to any source. */
+export function isUnmapped(frame: SDK.DebuggerModel.CallFrame): boolean {
+  const position = scopedPosition(frame);
+  return position !== null && position.sourceMap.findEntry(position.line, position.column)?.sourceURL === undefined;
+}
+
+/** @returns true iff both frames have encoded scopes and are mapped to the same original location. */
+export function isSameOriginalLocation(a: SDK.DebuggerModel.CallFrame, b: SDK.DebuggerModel.CallFrame): boolean {
+  const entry = (frame: SDK.DebuggerModel.CallFrame): SDK.SourceMap.SourceMapEntry|null => {
+    const position = scopedPosition(frame);
+    return position?.sourceMap.findEntry(position.line, position.column) ?? null;
+  };
+  const entryA = entry(a);
+  const entryB = entry(b);
+  return entryA?.sourceURL !== undefined && entryA.sourceURL === entryB?.sourceURL &&
+      entryA.sourceLineNumber === entryB.sourceLineNumber && entryA.sourceColumnNumber === entryB.sourceColumnNumber;
+}
+
 /** @returns the body of the innermost inlined function that {@link frame} is paused in (empty if none). */
 export function inlinedFunctionRanges(frame: SDK.DebuggerModel.CallFrame): SDK.DebuggerModel.LocationRange[] {
   const position = scopedPosition(frame);
   const range = position?.sourceMap.inlinedFunctionRange(position.line, position.column);
   return position && range ? toLocationRanges(position, [range]) : [];
 }
+
+/** @returns the bodies of the functions inlined into the logical function that {@link frame} is paused in. */
+export function inlinedCalleeRanges(frame: SDK.DebuggerModel.CallFrame): SDK.DebuggerModel.LocationRange[] {
+  const position = scopedPosition(frame);
+  return position ? toLocationRanges(position, position.sourceMap.inlinedCalleeRanges(position.line, position.column)) :
+                    [];
+}
+
+/**
+ * Decides whether the pause {@link details} completes the user's step {@link context}.
+ *
+ * @returns null to present the pause, or the step to issue instead.
+ */
+export async function nextAutoStep(
+    details: SDK.DebuggerModel.DebuggerPausedDetails, context: SDK.DebuggerModel.StepContext|null,
+    computeAutoStep: SDK.DebuggerModel.ComputeAutoStepCallback): Promise<SDK.DebuggerModel.AutoStep|null> {
+  if (!context || details.reason !== Protocol.Debugger.PausedEventReason.Step) {
+    return null;
+  }
+  const frames = details.callFrames;
+  const start = context.callFrames;
+  if (frames.length === 0 || start.length === 0 || (!isScopedFrame(frames[0]) && !isScopedFrame(start[0]))) {
+    return null;
+  }
+
+  switch (context.mode) {
+    case SDK.DebuggerModel.StepMode.STEP_INTO:
+      return isUnmapped(frames[0]) ? await computeAutoStep(SDK.DebuggerModel.StepMode.STEP_INTO, frames) : null;
+
+    case SDK.DebuggerModel.StepMode.STEP_OVER: {
+      const startDepth = logicalDepth(start);
+      const depth = logicalDepth(frames);
+      if (isUnmapped(frames[0]) || (depth === startDepth && isSameOriginalLocation(frames[0], start[0]))) {
+        // Unmapped code, or still on the same original location (the skip list only covers the generated ranges that
+        // contain the start position): keep stepping over.
+        return await computeAutoStep(SDK.DebuggerModel.StepMode.STEP_OVER, frames);
+      }
+      return null;
+    }
+
+    case SDK.DebuggerModel.StepMode.STEP_OUT:
+      // Stepping into calls of the unmapped code would present a pause inside the callee, so continue as step over.
+      return isUnmapped(frames[0]) ? await computeAutoStep(SDK.DebuggerModel.StepMode.STEP_OVER, frames) : null;
+  }
+}

package/front_end/models/bindings/StylesSourceMapping.ts

@@ -191,6 +191,9 @@ export class StyleFile implements TextUtils.ContentProvider.ContentProvider {
 
   addHeader(header: SDK.CSSStyleSheetHeader.CSSStyleSheetHeader): void {
     this.headers.add(header);
+    if (header.hasSourceURL) {
+      NetworkProject.setSourceURLSynthesized(this.uiSourceCode);
+    }
     NetworkProject.addFrameAttribution(this.uiSourceCode, header.frameId);
   }
 

package/front_end/models/emulation/DeviceModeModel.ts

@@ -319,6 +319,8 @@ export class DeviceModeModel extends Common.ObjectWrapper.ObjectWrapper<EventTyp
     if (this.#type === Type.Device && this.#device && this.#mode) {
       const orientation = this.#device.orientationByName(this.#mode.orientation);
       this.#scaleSetting.set(this.calculateFitScale(orientation.width, orientation.height));
+    } else if (this.#type === Type.Responsive) {
+      this.#scaleSetting.set(this.calculateFitScale(this.#widthSetting.get(), this.#heightSetting.get()));
     }
   }
 
@@ -377,7 +379,7 @@ export class DeviceModeModel extends Common.ObjectWrapper.ObjectWrapper<EventTyp
           if (savedDevice) {
             this.#autoAdjustScaleSetting.set(savedDevice.autoAdjust);
             // An explicitly requested scale takes precedence over the saved one.
-            if (scale === undefined && !savedDevice.autoAdjust) {
+            if (scale === undefined && !savedDevice.autoAdjust && savedDevice.scale > 0) {
               scale = savedDevice.scale;
             }
           } else {
@@ -404,10 +406,13 @@ export class DeviceModeModel extends Common.ObjectWrapper.ObjectWrapper<EventTyp
           const savedDevice = map['Responsive'];
           if (savedDevice) {
             this.#autoAdjustScaleSetting.set(savedDevice.autoAdjust);
-            if (!savedDevice.autoAdjust) {
+            if (!savedDevice.autoAdjust && savedDevice.scale > 0) {
               this.#scaleSetting.set(savedDevice.scale);
+            } else if (this.#initialized) {
+              this.#updateFitScale();
             } else {
-              this.#scaleSetting.set(this.calculateFitScale(this.#widthSetting.get(), this.#heightSetting.get()));
+              // Defer fit-scale calculation until setAvailableSize() initializes #preferredSize.
+              this.#autoFitScaleOnInitialize = true;
             }
           } else {
             this.#autoAdjustScaleSetting.set(true);
@@ -775,7 +780,7 @@ export class DeviceModeModel extends Common.ObjectWrapper.ObjectWrapper<EventTyp
   private calculateFitScale(screenWidth: number, screenHeight: number): number {
     let scale = Math.min(screenWidth ? this.#preferredSize.width / screenWidth : 1,
                          screenHeight ? this.#preferredSize.height / screenHeight : 1);
-    scale = Math.min(Math.floor(scale * 100), 100);
+    scale = Math.max(Math.min(Math.floor(scale * 100), 100), 1);
 
     let sharpScale = scale;
     while (sharpScale > scale * 0.7) {

package/front_end/models/har/HARFormat.ts

@@ -2,7 +2,6 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-import * as Common from '../../core/common/common.js';
 import type * as Platform from '../../core/platform/platform.js';
 import * as SDK from '../../core/sdk/sdk.js';
 import type * as Protocol from '../../generated/protocol.js';
@@ -421,9 +420,7 @@ export class HARInitiator extends HARBase {
     super(data);
     this.type = (HARBase.optionalString(data['type']) ?? SDK.NetworkRequest.InitiatorType.OTHER) as
         Protocol.Network.InitiatorType;
-    const url = HARBase.optionalString(data['url']);
-    // Ignore privileged/non-web schemes in imported HARs.
-    this.url = url && Common.ParsedURL.hasWebSafeScheme(url) ? url : undefined;
+    this.url = HARBase.optionalString(data['url']);
     this.lineNumber = HARBase.optionalNumber(data['lineNumber']);
     this.requestId = HARBase.optionalString(data['requestId']) as Protocol.Network.RequestId;
     if (data['stack']) {
@@ -480,9 +477,7 @@ export class HARCallFrame extends HARBase {
 
     this.functionName = HARBase.optionalString(data['functionName']) ?? '';
     this.scriptId = (HARBase.optionalString(data['scriptId']) ?? '') as Protocol.Runtime.ScriptId;
-    const url = HARBase.optionalString(data['url']) ?? '';
-    // Ignore privileged/non-web schemes in imported HARs.
-    this.url = Common.ParsedURL.hasWebSafeScheme(url) ? url : '';
+    this.url = HARBase.optionalString(data['url']) ?? '';
     this.lineNumber = HARBase.optionalNumber(data['lineNumber']) ?? -1;
     this.columnNumber = HARBase.optionalNumber(data['columnNumber']) ?? -1;
   }

package/front_end/models/issues_manager/Issue.ts

@@ -27,7 +27,7 @@ const UIStrings = {
   /**
    * @description A description for a kind of issue we display in the Issues tab.
    */
-  pageErrorIssue: 'A page error issue: the page is not working correctly',
+  pageErrorIssue: 'A page error issue: the page isn’t working correctly',
   /**
    * @description A description for a kind of issue we display in the Issues tab.
    */

package/front_end/models/javascript_metadata/NativeFunctions.ts

@@ -227,7 +227,7 @@ export const NativeFunctions: readonly NativeFunctionValue[] = [
   {
     name: "create",
     signatures: [["?options"]],
-    receivers: ["CredentialsContainer","SemanticEmbedder"]
+    receivers: ["CredentialsContainer","DecisionModel","SemanticEmbedder"]
   },
   {
     name: "defineProperty",
@@ -3859,6 +3859,10 @@ export const NativeFunctions: readonly NativeFunctionValue[] = [
     name: "framebufferTextureMultiviewOVR",
     signatures: [["target","attachment","texture","level","baseViewIndex","numViews"]]
   },
+  {
+    name: "startRendering",
+    signatures: [["?chunkSize"]]
+  },
   {
     name: "convertToBlob",
     signatures: [["?options"]]
@@ -8085,6 +8089,10 @@ export const NativeFunctions: readonly NativeFunctionValue[] = [
     name: "decapsulateBits",
     signatures: [["decapsulationAlgorithm","decapsulationKey","ciphertext"]]
   },
+  {
+    name: "requestFileHandle",
+    signatures: [["hash","?options"]]
+  },
   {
     name: "userAgentAllowsProtocol",
     signatures: [["protocol"]]
@@ -8301,6 +8309,14 @@ export const NativeFunctions: readonly NativeFunctionValue[] = [
     name: "WorkletAnimation",
     signatures: [["animatorName","effects","?timeline","?options"]]
   },
+  {
+    name: "availability",
+    signatures: [["?options"]]
+  },
+  {
+    name: "decide",
+    signatures: [["input","?options"]]
+  },
   {
     name: "LanguageModelToolCall",
     signatures: [["init"]]
@@ -8313,10 +8329,6 @@ export const NativeFunctions: readonly NativeFunctionValue[] = [
     name: "LanguageModelToolError",
     signatures: [["init"]]
   },
-  {
-    name: "availability",
-    signatures: [["?options"]]
-  },
   {
     name: "embed",
     signatures: [["input","?options"]]
@@ -8395,7 +8407,7 @@ export const NativeFunctions: readonly NativeFunctionValue[] = [
   },
   {
     name: "createParserOptions",
-    signatures: [["input"]]
+    signatures: [["?input","...args"]]
   },
   {
     name: "createPolicy",
@@ -9031,54 +9043,6 @@ export const NativeFunctions: readonly NativeFunctionValue[] = [
     name: "timeout",
     signatures: [["milliseconds"]]
   },
-  {
-    name: "beforeHTML",
-    signatures: [["html","?options"]]
-  },
-  {
-    name: "beforeHTMLUnsafe",
-    signatures: [["html","?options"]]
-  },
-  {
-    name: "afterHTML",
-    signatures: [["html","?options"]]
-  },
-  {
-    name: "afterHTMLUnsafe",
-    signatures: [["html","?options"]]
-  },
-  {
-    name: "replaceWithHTML",
-    signatures: [["html","?options"]]
-  },
-  {
-    name: "replaceWithHTMLUnsafe",
-    signatures: [["html","?options"]]
-  },
-  {
-    name: "streamBeforeHTML",
-    signatures: [["?options"]]
-  },
-  {
-    name: "streamBeforeHTMLUnsafe",
-    signatures: [["?options"]]
-  },
-  {
-    name: "streamAfterHTML",
-    signatures: [["?options"]]
-  },
-  {
-    name: "streamAfterHTMLUnsafe",
-    signatures: [["?options"]]
-  },
-  {
-    name: "streamReplaceWithHTML",
-    signatures: [["?options"]]
-  },
-  {
-    name: "streamReplaceWithHTMLUnsafe",
-    signatures: [["?options"]]
-  },
   {
     name: "Comment",
     signatures: [["?data"]]
@@ -9191,6 +9155,54 @@ export const NativeFunctions: readonly NativeFunctionValue[] = [
     name: "MutationObserver",
     signatures: [["callback"]]
   },
+  {
+    name: "beforeHTML",
+    signatures: [["html","?options"]]
+  },
+  {
+    name: "beforeHTMLUnsafe",
+    signatures: [["html","?options"]]
+  },
+  {
+    name: "afterHTML",
+    signatures: [["html","?options"]]
+  },
+  {
+    name: "afterHTMLUnsafe",
+    signatures: [["html","?options"]]
+  },
+  {
+    name: "replaceWithHTML",
+    signatures: [["html","?options"]]
+  },
+  {
+    name: "replaceWithHTMLUnsafe",
+    signatures: [["html","?options"]]
+  },
+  {
+    name: "streamBeforeHTML",
+    signatures: [["?options"]]
+  },
+  {
+    name: "streamBeforeHTMLUnsafe",
+    signatures: [["?options"]]
+  },
+  {
+    name: "streamAfterHTML",
+    signatures: [["?options"]]
+  },
+  {
+    name: "streamAfterHTMLUnsafe",
+    signatures: [["?options"]]
+  },
+  {
+    name: "streamReplaceWithHTML",
+    signatures: [["?options"]]
+  },
+  {
+    name: "streamReplaceWithHTMLUnsafe",
+    signatures: [["?options"]]
+  },
   {
     name: "Observable",
     signatures: [["callback"]]

package/front_end/models/persistence/Automapping.ts

@@ -194,7 +194,13 @@ export class Automapping {
       if (this.#sourceCodeToProcessingPromiseMap.get(networkSourceCode) !== createBindingPromise) {
         return null;
       }
-      if (status.network.contentType().isFromSourceMap() || !status.fileSystem.contentType().isTextType()) {
+      const target = Bindings.NetworkProject.NetworkProject.targetForUISourceCode(status.network);
+      const isNodeTarget = target?.type() === SDK.Target.Type.NODE;
+      // Source-map-derived URLs and contents are controlled by the inspected page.
+      // Only trust them without content validation when debugging Node.js.
+      const skipContentValidation = !status.fileSystem.contentType().isTextType() ||
+          (status.network.contentType().isFromSourceMap() && isNodeTarget);
+      if (skipContentValidation) {
         return status;
       }
 
@@ -231,10 +237,9 @@ export class Automapping {
         return null;
       }
 
-      const target = Bindings.NetworkProject.NetworkProject.targetForUISourceCode(status.network);
       let isValid = false;
       const fileContent = fileSystemContent.content;
-      if (target && target.type() === SDK.Target.Type.NODE) {
+      if (isNodeTarget) {
         if (networkContent.content) {
           const rewrappedNetworkContent =
               PersistenceImpl.rewrapNodeJSContent(status.fileSystem, fileContent, networkContent.content);

package/front_end/models/trace/Processor.ts

@@ -202,6 +202,13 @@ export class TraceProcessor extends EventTarget {
 
     options.logger?.start('parse:handleEvent');
 
+    // In bundled builds each handler is a module namespace object whose exports are getters, and every handler has a
+    // different shape. V8 can't optimise a `handler.handleEvent` read that sees all these shapes, so reading it inside the
+    // loop costs a slow getter call for every event and handler.
+    // On a 1M-event trace that is about 30M getter calls per parse. Reading each `handleEvent` once here cut the
+    // handleEvent loop from ~2.9s to ~2.1s (-25%) and total parse time by ~10%, so keep this read outside the loop.
+    const handleEventFns = sortedHandlers.map(([, handler]) => handler.handleEvent);
+
     // Handle each event.
     for (let i = 0; i < traceEvents.length; ++i) {
       // Every so often we take a break just to render.
@@ -213,9 +220,8 @@ export class TraceProcessor extends EventTarget {
         await new Promise(resolve => setTimeout(resolve, 0));
       }
       const event = traceEvents[i];
-      for (let j = 0; j < sortedHandlers.length; ++j) {
-        const [, handler] = sortedHandlers[j];
-        handler.handleEvent(event);
+      for (let j = 0; j < handleEventFns.length; ++j) {
+        handleEventFns[j](event);
       }
     }
 

package/front_end/models/trace/extras/TraceTree.ts

@@ -497,7 +497,7 @@ export class BottomUpRootNode extends Node {
 
     function onEndEvent(event: Types.Events.Event): void {
       const id = idStack.pop();
-      if (!id) {
+      if (id === undefined) {
         return;
       }