chrome-devtools-frontend 1.0.1713292 → 1.0.1716465

package/AUTHORS

@@ -53,6 +53,7 @@ Han Xu <suyanhanx@gmail.com>
 Helmut Januschka <helmut@januschka.com>
 Henry Lim <henry@limhenry.xyz>
 Hojeong Park <parkhj062@gmail.com>
+Hubert Gajewski <hubert@hubertgajewski.com>
 Ivan Akulov <gxoptg@gmail.com>
 Jacky Hu <flameddd@gmail.com>
 Jake Mulhern <mulherje@gmail.com>

package/DIR_METADATA

@@ -4,11 +4,9 @@ buganizer_public: {
 }
 agentic_review: {
   capability_groups: {
-    name: "core-review"
     review_capability_skills: [".agents/skills/code-review/SKILL.md"]
   }
   capability_groups: {
-    name: "typescript-and-models"
     path_expression: ["front_end/.*\\.ts", ".*BUILD\\.gn", ".*\\.gni"]
     path_expression_exclusion: [".*\\.test\\.ts"]
     review_capability_skills: [
@@ -20,7 +18,6 @@ agentic_review: {
     ]
   }
   capability_groups: {
-    name: "ui-and-widgets"
     path_expression: ["front_end/(panels|ui)/.*\\.(ts|css)"]
     path_expression_exclusion: [".*\\.test\\.ts"]
     review_capability_skills: [
@@ -29,7 +26,6 @@ agentic_review: {
     ]
   }
   capability_groups: {
-    name: "tests"
     path_expression: [".*\\.test\\.ts", "test/.*\\.ts"]
     review_capability_skills: [
       ".agents/skills/devtools-testing-guidance/SKILL.md",

package/eslint.config.mjs

@@ -61,6 +61,7 @@ export default defineConfig([
     'scripts/deps/tests/fixtures',
     'scripts/gn_deps_verifier/tests/fixtures',
     'test/**/fixtures/',
+    'test/ai_evals/eval_data/',
     'test/e2e/**/*.js',
     'test/shared/**/*.js',
     'extensions/cxx_debugging/tests/inputs/',
@@ -375,6 +376,7 @@ export default defineConfig([
         'error',
         {
           argsIgnorePattern: '^_',
+          ignoreUsingDeclarations: true,
         },
       ],
 

package/front_end/core/host/Platform.ts

@@ -69,3 +69,40 @@ export function fontFamily(): string {
 export function setFontFamilyForTests(family: string): void {
   _fontFamily = family;
 }
+
+let _isRemoteDevtoolsFrontend: boolean|undefined;
+let _locationForTest: string|undefined;
+
+export function setLocationForTest(url: string|undefined): void {
+  _locationForTest = url;
+  _isRemoteDevtoolsFrontend = undefined;
+  _isCustomDevtoolsFrontend = undefined;
+}
+
+export function isRemoteDevtoolsFrontend(): boolean {
+  if (typeof _isRemoteDevtoolsFrontend === 'undefined') {
+    const location = (globalThis as unknown as {location?: {toString(): string}}).location;
+    const url = _locationForTest ?? (location ? location.toString() : '');
+    _isRemoteDevtoolsFrontend = url.startsWith('devtools://devtools-remote/');
+  }
+  return _isRemoteDevtoolsFrontend ?? false;
+}
+
+let _isCustomDevtoolsFrontend: boolean|undefined;
+
+export function isCustomDevtoolsFrontend(): boolean {
+  if (typeof _isCustomDevtoolsFrontend === 'undefined') {
+    const location = (globalThis as unknown as {location?: {toString(): string}}).location;
+    const url = _locationForTest ?? (location ? location.toString() : '');
+    _isCustomDevtoolsFrontend = url.startsWith('devtools://devtools/custom/');
+  }
+  return _isCustomDevtoolsFrontend ?? false;
+}
+
+export function setIsRemoteDevtoolsFrontendForTest(isRemote: boolean|undefined): void {
+  _isRemoteDevtoolsFrontend = isRemote;
+}
+
+export function setIsCustomDevtoolsFrontendForTest(isCustom: boolean|undefined): void {
+  _isCustomDevtoolsFrontend = isCustom;
+}

package/front_end/core/host/UserMetricsEnums.ts

@@ -633,9 +633,12 @@ export enum IssueCreated {
   'CorsIssue::LocalNetworkAccessPermissionDenied' = 111,
   'SRIMessageSignatureIssue::ValidationFailedIntegrityMismatch' = 112,
   'ElementAccessibilityIssue::InteractiveContentSummaryDescendant' = 113,
-  'CorsIssue::InvalidLocalNetworkAccess' = 114,
+  'GenericIssue::AutofillAndManualTextPolicyControlledFeaturesInfo' = 114,
+  'GenericIssue::AutofillPolicyControlledFeatureInfo' = 115,
+  'GenericIssue::ManualTextPolicyControlledFeatureInfo' = 116,
+  'CorsIssue::InvalidLocalNetworkAccess' = 118,
   /* eslint-enable @typescript-eslint/naming-convention */
-  MAX_VALUE = 115,
+  MAX_VALUE = 119,
 }
 
 export enum DeveloperResourceLoaded {

package/front_end/core/platform/browser/HostRuntime.ts

@@ -95,17 +95,17 @@ async function saveScreenshot(options: Api.HostRuntime.ScreenshotOptions): Promi
     const contentTop = options.clip.visiblePageRectTop * scale;
 
     canvas = new OffscreenCanvas(
-        Math.floor(screenRectWidth),
+        Math.round(screenRectWidth),
         // Cap the height to not hit the GPU limit.
         // https://crbug.com/1260828
-        Math.min(1 << 14, Math.floor(screenRectHeight)),
+        Math.min(1 << 14, Math.round(screenRectHeight)),
     );
     const ctx = canvas.getContext('2d', {willReadFrequently: true});
     if (!ctx) {
       throw new Error('Could not get 2d context from canvas.');
     }
     ctx.imageSmoothingEnabled = false;
-    ctx.drawImage(pageImage, Math.floor(contentLeft), Math.floor(contentTop));
+    ctx.drawImage(pageImage, Math.round(contentLeft), Math.round(contentTop));
   } else {
     canvas = new OffscreenCanvas(
         pageImage.naturalWidth,

package/front_end/core/sdk/CSSProperty.ts

@@ -20,6 +20,7 @@ import {
 } from './CSSPropertyParser.js';
 import {CSSWideKeywordMatcher} from './CSSPropertyParserMatchers.js';
 import type {CSSStyleDeclaration} from './CSSStyleDeclaration.js';
+import {textEditorIndentSettingDescriptor} from './SDKSettings.js';
 
 export const enum Events {
   LOCAL_VALUE_UPDATED = 'localValueUpdated',
@@ -221,7 +222,7 @@ export class CSSProperty extends Common.ObjectWrapper.ObjectWrapper<EventTypes>
     const range = this.range.relativeTo(this.ownerStyle.range.startLine, this.ownerStyle.range.startColumn);
     const indentation = this.ownerStyle.cssText ?
         this.detectIndentation(this.ownerStyle.cssText) :
-        this.ownerStyle.cssModel().target().targetManager().settings.moduleSetting('text-editor-indent').get();
+        this.ownerStyle.cssModel().target().targetManager().settings.resolve(textEditorIndentSettingDescriptor).get();
     const endIndentation = this.ownerStyle.cssText ? indentation.substring(0, this.ownerStyle.range.endColumn) : '';
     const text = new TextUtils.Text.Text(this.ownerStyle.cssText || '');
     const newStyleText = text.replaceRange(range, Platform.StringUtilities.sprintf(';%s;', propertyText));

package/front_end/core/sdk/CSSPropertyParserMatchers.ts

@@ -85,7 +85,10 @@ export class BaseVariableMatcher extends BaseVariableMatcherBase {
       return null;
     }
     const nameNode = args[0][0];
-    const fallback = args.length === 2 ? args[1] : undefined;
+    // Skip the first comma and closing parenthesis, preserving commas in the fallback.
+    const fallback = args.length > 1 ?
+        Array.from(ASTUtils.stripComments(ASTUtils.siblings(nameNode.nextSibling))).slice(1, -1) :
+        undefined;
 
     if (!isVariableNameNode(nameNode, matching.ast)) {
       return null;

package/front_end/core/sdk/DebuggerModel.ts

@@ -134,10 +134,14 @@ export const enum StepMode {
   STEP_OVER = 'StepOver',
 }
 
-/** A CDP step command. `ranges` is passed as `skipList` to stepInto/stepOver, and ignored for stepOut. */
+/**
+ * A CDP step command. `ranges` is passed as `skipList` to stepInto/stepOver, and ignored for stepOut. `enterRanges` is
+ * passed as `enterRanges` to stepOver: functions within them are entered as if by stepInto.
+ */
 export interface AutoStep {
   readonly command: StepMode;
   readonly ranges: readonly LocationRange[];
+  readonly enterRanges?: readonly LocationRange[];
 }
 
 /**
@@ -478,20 +482,25 @@ export class DebuggerModel extends SDKModel<EventTypes> {
     this.#issueStep(step, breakOnAsyncCall);
   }
 
-  #issueStep({command, ranges}: AutoStep, breakOnAsyncCall = false): void {
-    const skipList =
-        sortAndMergeRanges(ranges.map(({start, end}) => ({
-                                        scriptId: start.scriptId,
-                                        start: {lineNumber: start.lineNumber, columnNumber: start.columnNumber},
-                                        end: {lineNumber: end.lineNumber, columnNumber: end.columnNumber},
-                                      })));
+  #issueStep({command, ranges, enterRanges}: AutoStep, breakOnAsyncCall = false): void {
+    const toProtocolRange = ({start, end}: LocationRange): Protocol.Debugger.LocationRange => ({
+      scriptId: start.scriptId,
+      start: {lineNumber: start.lineNumber, columnNumber: start.columnNumber},
+      end: {lineNumber: end.lineNumber, columnNumber: end.columnNumber},
+    });
+    const skipList = sortAndMergeRanges(ranges.map(toProtocolRange));
     switch (command) {
       case StepMode.STEP_INTO:
         void this.agent.invoke_stepInto({breakOnAsyncCall, skipList});
         break;
-      case StepMode.STEP_OVER:
-        void this.agent.invoke_stepOver({skipList});
+      case StepMode.STEP_OVER: {
+        const request: Protocol.Debugger.StepOverRequest = {skipList};
+        if (enterRanges?.length) {
+          request.enterRanges = sortAndMergeRanges(enterRanges.map(toProtocolRange));
+        }
+        void this.agent.invoke_stepOver(request);
         break;
+      }
       case StepMode.STEP_OUT:
         void this.agent.invoke_stepOut();
         break;
@@ -500,6 +509,11 @@ export class DebuggerModel extends SDKModel<EventTypes> {
 
   resume(): void {
     this.#stepContext = null;
+    this.#resumeBackend();
+  }
+
+  /** Resumes without touching the step context. */
+  #resumeBackend(): void {
     void this.agent.invoke_resume({terminateOnResume: false});
     this.#isPausing = false;
   }
@@ -682,7 +696,8 @@ export class DebuggerModel extends SDKModel<EventTypes> {
       if (this.#synchronizeBreakpointsCallback && script) {
         await this.#synchronizeBreakpointsCallback(script);
       }
-      this.resume();
+      // V8 continues an in-progress step after an instrumentation pause, so keep the step context.
+      this.#resumeBackend();
       return;
     }
 

package/front_end/core/sdk/PageFunctions.ts

@@ -109,7 +109,12 @@ export function toggleClassAndInjectStyleRule(this: Element, pseudoElementName:
   const classNamePrefix = '__web-inspector-hide';
   const classNameSuffix = '-shortcut__';
   const styleTagId = '__web-inspector-hide-shortcut-style__';
-  const pseudoElementNameEscaped = pseudoElementName ? pseudoElementName.replace(/[^a-zA-Z0-9_-]/g, '_') : '';
+  let pseudoElementNameEscaped = '';
+  if (pseudoElementName) {
+    pseudoElementNameEscaped = pseudoElementName.replace(/[^a-zA-Z0-9-]/g, function(c) {
+      return '_' + c.charCodeAt(0).toString(16) + '_';
+    });
+  }
   const className = classNamePrefix + pseudoElementNameEscaped + classNameSuffix;
   this.classList.toggle(className, hidden);
 
@@ -117,7 +122,7 @@ export function toggleClassAndInjectStyleRule(this: Element, pseudoElementName:
   while (localRoot.parentNode) {
     localRoot = (localRoot.parentNode as Element);
   }
-  if (localRoot.nodeType === Node.DOCUMENT_NODE) {
+  if (localRoot.nodeType === 9 /* Node.DOCUMENT_NODE */) {
     localRoot = document.head;
   }
 

package/front_end/core/sdk/ResourceTreeModel.ts

@@ -1097,6 +1097,15 @@ export class ResourceTreeFrame {
     return response.originTrials;
   }
 
+  async getSpellCheckCustomDictionary(): Promise<string[]> {
+    const response =
+        await this.resourceTreeModel().target().pageAgent().invoke_getSpellCheckCustomDictionary({frameId: this.#id});
+    if (response.getError()) {
+      return [];
+    }
+    return response.words;
+  }
+
   setCreationStackTrace(creationStackTraceData: {
     creationStackTrace: Protocol.Runtime.StackTrace|null,
     creationStackTraceTarget: Target,

package/front_end/core/sdk/RuntimeModel.ts

@@ -20,7 +20,7 @@ import {
   ScopeRemoteObject,
 } from './RemoteObject.js';
 import {SDKModel} from './SDKModel.js';
-import {customFormattersSettingDescriptor} from './SDKSettings.js';
+import {customFormattersSettingDescriptor, textEditorIndentSettingDescriptor} from './SDKSettings.js';
 import {Capability, type Target, Type} from './Target.js';
 
 export class RuntimeModel extends SDKModel<EventTypes> {
@@ -303,8 +303,11 @@ export class RuntimeModel extends SDKModel<EventTypes> {
       return;
     }
 
-    const indent =
-        this.target().targetManager().context.get(Common.Settings.Settings).moduleSetting('text-editor-indent').get();
+    const indent = this.target()
+                       .targetManager()
+                       .context.get(Common.Settings.Settings)
+                       .resolve(textEditorIndentSettingDescriptor)
+                       .get();
     void object
         .callFunctionJSON(toStringForClipboard, [{
                             value: {

package/front_end/core/sdk/SDKSettings.ts

@@ -346,3 +346,10 @@ export const preserveNetworkLogSettingDescriptor: Common.Settings.SettingDescrip
   type: Common.Settings.SettingType.BOOLEAN,
   defaultValue: false,
 };
+
+export const textEditorIndentSettingDescriptor: Common.Settings.SettingDescriptor<string> = {
+  name: 'text-editor-indent',
+  type: Common.Settings.SettingType.ENUM,
+  defaultValue: '    ',
+  storageType: Common.Settings.SettingStorageType.SYNCED,
+};

package/front_end/core/sdk/ServerSentEventsProtocol.ts

@@ -95,8 +95,10 @@ export class ServerSentEventsParser {
       this.#data += '\n';
     }
     if (fieldName === 'id') {
-      // We should do a check here whether the id field contains "\0" and ignore it.
-      this.#id = this.#line.substring(fieldValueStart);
+      const id = this.#line.substring(fieldValueStart);
+      if (!id.includes('\0')) {
+        this.#id = id;
+      }
     }
     // Ignore all other fields. Also ignore "retry", we won't forward that to the backend.
   }

package/front_end/core/sdk/SourceMap.ts

@@ -900,11 +900,6 @@ export class SourceMap {
                                         [];
   }
 
-  /** See {@link SourceMapScopesInfo.hasOutlinedFunctions}. False without encoded scopes. */
-  hasOutlinedFunctions(): boolean {
-    return this.hasEncodedScopeInfo() && (this.#scopesInfo?.hasOutlinedFunctions() ?? false);
-  }
-
   /** See {@link SourceMapScopesInfo.outlinedFunctionRanges}. Empty without encoded scopes. */
   outlinedFunctionRanges(generatedLine: number, generatedColumn: number): PositionRange[] {
     return this.hasEncodedScopeInfo() ? this.#scopesInfo?.outlinedFunctionRanges(generatedLine, generatedColumn) ?? [] :

package/front_end/core/sdk/SourceMapFunctionRanges.ts

@@ -77,8 +77,11 @@ function validateStartBeforeEnd(ranges: NamedFunctionRange[]): void {
 }
 
 function createScopeFrom(range: NamedFunctionRange, parent?: ScopesCodec.OriginalScope): ScopesCodec.OriginalScope {
+  // Copy only known fields: ranges can come from extensions and may carry extra properties.
   return {
-    ...range,
+    start: range.start,
+    end: range.end,
+    name: typeof range.name === 'string' ? range.name : String(range.name ?? ''),
     kind: 'Function',
     isStackFrame: true,
     parent,

package/front_end/core/sdk/SourceMapScopesInfo.ts

@@ -613,21 +613,11 @@ export class SourceMapScopesInfo {
     return result;
   }
 
-  /**
-   * @returns true, iff any generated function is outlined, i.e. marked as "hidden" but with a definition (see
-   *          {@link GeneratedFrameKind.OUTLINED}). Hidden functions without a definition are compiler helpers.
-   */
-  hasOutlinedFunctions(): boolean {
-    const hasOutlined = (ranges: ScopesCodec.GeneratedRange[]): boolean =>
-        ranges.some(range => (range.isStackFrame && range.isHidden && range.originalScope !== undefined) ||
-                        hasOutlined(range.children));
-    return hasOutlined(this.#generatedRanges);
-  }
-
   /**
    * @returns the outlined parts of the authored function at the position: the hidden generated functions whose
-   *          original scope lies within that function's original scope (see {@link GeneratedFrameKind.OUTLINED}).
-   *          Includes the outlined part the position itself is in, if any. Sorted by start position.
+   *          original scope lies within that function's original scope (see {@link GeneratedFrameKind.OUTLINED}),
+   *          but not within a nested authored function. Includes the outlined part the position itself is in, if
+   *          any. Sorted by start position.
    */
   outlinedFunctionRanges(generatedLine: number, generatedColumn: number): PositionRange[] {
     const rangeChain = this.#findGeneratedRangeChain(generatedLine, generatedColumn);
@@ -635,17 +625,17 @@ export class SourceMapScopesInfo {
     if (!functionScope) {
       return [];
     }
-    const isWithinFunction = (scope: ScopesCodec.OriginalScope|undefined): boolean =>
-        scope !== undefined && (scope === functionScope || isWithinFunction(scope.parent));
     const result: PositionRange[] = [];
-    (function walk(ranges: ScopesCodec.GeneratedRange[]) {
+    const walk = (ranges: ScopesCodec.GeneratedRange[]): void => {
       for (const range of ranges) {
-        if (range.isStackFrame && range.isHidden && isWithinFunction(range.originalScope)) {
+        if (range.isStackFrame && range.isHidden && range.originalScope &&
+            this.#findFunctionScopeInOriginalScopeChain(range.originalScope) === functionScope) {
           result.push({start: range.start, end: range.end});
         }
         walk(range.children);
       }
-    })(this.#generatedRanges);
+    };
+    walk(this.#generatedRanges);
     return result;
   }
 

package/front_end/devtools_compatibility.js

@@ -1006,8 +1006,11 @@
       'CorsIssue::LocalNetworkAccessPermissionDenied': 111,
       'SRIMessageSignatureIssue::ValidationFailedIntegrityMismatch': 112,
       'ElementAccessibilityIssue::InteractiveContentSummaryDescendant': 113,
-      'CorsIssue::InvalidLocalNetworkAccess': 114,
-      MAX_VALUE: 115,
+      'GenericIssue::AutofillAndManualTextPolicyControlledFeaturesInfo': 114,
+      'GenericIssue::AutofillPolicyControlledFeatureInfo': 115,
+      'GenericIssue::ManualTextPolicyControlledFeatureInfo': 116,
+      'CorsIssue::InvalidLocalNetworkAccess': 118,
+      MAX_VALUE: 119,
     },
     DeveloperResourceLoaded: {
       LOAD_THROUGH_PAGE_VIA_TARGET: 0,

package/front_end/entrypoints/formatter_worker/CSSFormatter.ts

@@ -13,6 +13,7 @@ const cssTrimEnd = (tokenValue: string): string => {
 
 export class CSSFormatter {
   readonly #builder: FormattedContentBuilder;
+  #text!: string;
   #toOffset!: number;
   #fromOffset!: number;
   #lineEndings!: number[];
@@ -29,6 +30,7 @@ export class CSSFormatter {
   }
 
   format(text: string, lineEndings: number[], fromOffset: number, toOffset: number): void {
+    this.#text = text;
     this.#lineEndings = lineEndings;
     this.#fromOffset = fromOffset;
     this.#toOffset = toOffset;
@@ -41,6 +43,48 @@ export class CSSFormatter {
     this.#builder.setEnforceSpaceBetweenWords(oldEnforce);
   }
 
+  #isSelectorPseudoColon(colonOffset: number): boolean {
+    let parenDepth = 0;
+    for (let i = colonOffset + 1; i < this.#toOffset; ++i) {
+      const ch = this.#text[i];
+      if (ch === '/' && this.#text[i + 1] === '*') {
+        const commentEnd = this.#text.indexOf('*/', i + 2);
+        if (commentEnd === -1) {
+          break;
+        }
+        i = commentEnd + 1;
+        continue;
+      }
+      if (ch === '\'' || ch === '"') {
+        const quote = ch;
+        for (++i; i < this.#toOffset; ++i) {
+          if (this.#text[i] === '\\') {
+            ++i;
+          } else if (this.#text[i] === quote) {
+            break;
+          }
+        }
+        continue;
+      }
+      if (ch === '(') {
+        ++parenDepth;
+      } else if (ch === ')') {
+        if (parenDepth === 0) {
+          return false;
+        }
+        --parenDepth;
+      } else if (parenDepth === 0) {
+        if (ch === '{') {
+          return true;
+        }
+        if (ch === ';' || ch === '}') {
+          return false;
+        }
+      }
+    }
+    return false;
+  }
+
   #tokenCallback(token: string, type: string|null, startPosition: number): void {
     // startPosition is relative to the start of the CSS block.
     // Convert it to an absolute document offset to match this.#lineEndings.
@@ -87,14 +131,20 @@ export class CSSFormatter {
       this.#builder.decreaseNestingLevel();
       this.#state.afterClosingBrace = true;
       this.#state.inPropertyValue = false;
-    } else if (token === ':' && !this.#state.inPropertyValue && this.#state.seenProperty) {
-      this.#builder.addToken(token, startPosition);
-      this.#builder.addSoftSpace();
-      this.#state.eatWhitespace = true;
-      this.#state.inPropertyValue = true;
       this.#state.seenProperty = false;
-      return;
+    } else if (token === ':' && !this.#state.inPropertyValue && this.#state.seenProperty) {
+      if (this.#isSelectorPseudoColon(startPosition)) {
+        this.#state.seenProperty = false;
+      } else {
+        this.#builder.addToken(token, startPosition);
+        this.#builder.addSoftSpace();
+        this.#state.eatWhitespace = true;
+        this.#state.inPropertyValue = true;
+        this.#state.seenProperty = false;
+        return;
+      }
     } else if (token === '{') {
+      this.#state.seenProperty = false;
       this.#builder.addSoftSpace();
       this.#builder.addToken(token, startPosition);
       this.#builder.addNewLine();

package/front_end/entrypoints/formatter_worker/HTMLFormatter.ts

@@ -102,7 +102,26 @@ export class HTMLFormatter {
     this.#builder.addNewLine();
   }
 
-  #afterCloseTag(_element: FormatterElement): void {
+  #afterCloseTag(element: FormatterElement): void {
+    if (!this.#model) {
+      return;
+    }
+
+    if (!element.children.length && element !== this.#model.document()) {
+      const nextToken = this.#model.peekToken();
+      if (nextToken) {
+        if (isTextToken(nextToken)) {
+          return;
+        }
+        const followingToken = this.#model.peekToken(1);
+        if (Platform.StringUtilities.isWhitespace(nextToken.value) && !/[\r\n]/.test(nextToken.value) &&
+            followingToken && isTextToken(followingToken)) {
+          this.#model.nextToken();
+          this.#builder.addToken(nextToken.value, nextToken.startOffset);
+          return;
+        }
+      }
+    }
     this.#builder.addNewLine();
   }
 
@@ -239,6 +258,12 @@ function hasTokenInSet(tokenTypes: Set<string>, type: string): boolean {
   return tokenTypes.has(type) || tokenTypes.has(`xml-${type}`);
 }
 
+function isTextToken(token: Token): boolean {
+  return !Platform.StringUtilities.isWhitespace(token.value) && !hasTokenInSet(token.type, 'tag') &&
+      !hasTokenInSet(token.type, 'attribute') && !hasTokenInSet(token.type, 'bracket') &&
+      !hasTokenInSet(token.type, 'comment') && !hasTokenInSet(token.type, 'meta');
+}
+
 export class HTMLModel {
   #state: ParseState = ParseState.INITIAL;
   readonly #document: FormatterElement;
@@ -275,9 +300,7 @@ export class HTMLModel {
       if (element && (hasTokenInSet(token.type, 'comment') || hasTokenInSet(token.type, 'meta'))) {
         element.hasComments = true;
       }
-      if (element && !Platform.StringUtilities.isWhitespace(token.value) && !hasTokenInSet(token.type, 'tag') &&
-          !hasTokenInSet(token.type, 'attribute') && !hasTokenInSet(token.type, 'bracket') &&
-          !hasTokenInSet(token.type, 'comment') && !hasTokenInSet(token.type, 'meta')) {
+      if (element && isTextToken(token)) {
         element.hasContent = true;
       }
       if (element && (element.name === 'script' || element.name === 'style') &&
@@ -497,8 +520,8 @@ export class HTMLModel {
     this.#stack.push(newElement);
   }
 
-  peekToken(): Token|null {
-    return this.#tokenIndex < this.#tokens.length ? this.#tokens[this.#tokenIndex] : null;
+  peekToken(offset = 0): Token|null {
+    return this.#tokenIndex + offset < this.#tokens.length ? this.#tokens[this.#tokenIndex + offset] : null;
   }
 
   nextToken(): Token|null {

package/front_end/entrypoints/main/MainImpl.ts

@@ -144,15 +144,6 @@ const WINDOW_LOCAL_STORAGE: Common.Settings.SettingsBackingStore = {
   clear: () => window.localStorage.clear(),
 };
 
-let isCustomDevtoolsFrontendInternal: boolean;
-
-function isCustomDevtoolsFrontend(): boolean {
-  if (typeof isCustomDevtoolsFrontendInternal === 'undefined') {
-    isCustomDevtoolsFrontendInternal = window.location.toString().startsWith('devtools://devtools/custom/');
-  }
-  return isCustomDevtoolsFrontendInternal;
-}
-
 export class MainImpl {
   #readyForTestPromise = Promise.withResolvers<void>();
   #veStartPromise!: Promise<void>;
@@ -309,11 +300,12 @@ export class MainImpl {
   } {
     this.#initializeExperiments();
     let storagePrefix = '';
-    if (isCustomDevtoolsFrontend()) {
+    if (Host.Platform.isCustomDevtoolsFrontend()) {
       storagePrefix = '__custom__';
-    } else if (
-        !Root.Runtime.Runtime.queryParam('can_dock') && Boolean(Root.Runtime.Runtime.queryParam('debugFrontend')) &&
-        !Host.InspectorFrontendHost.isUnderTest()) {
+    } else if (Host.Platform.isRemoteDevtoolsFrontend()) {
+      storagePrefix = '__remote__';
+    } else if (!Root.Runtime.Runtime.queryParam('can_dock') &&
+               Boolean(Root.Runtime.Runtime.queryParam('debugFrontend')) && !Host.InspectorFrontendHost.isUnderTest()) {
       storagePrefix = '__bundled__';
     }
 
@@ -669,7 +661,7 @@ export class MainImpl {
 
   #redispatchClipboardEvent(event: Event): void {
     const eventCopy = new CustomEvent('clipboard-' + event.type, {bubbles: true, composed: true});
-    // @ts-expect-error Used in ElementsTreeOutline
+    // @ts-expect-error Used in DOMTreeWidget
     eventCopy['original'] = event;
     const document = event.target && (event.target as HTMLElement).ownerDocument;
     const target = document ? UI.DOMUtilities.deepActiveElement(document) : null;

package/front_end/generated/Deprecation.ts

@@ -242,6 +242,10 @@ export const UIStrings = {
    * @description A deprecation warning shown in the DevTools Issues tab. It's shown when a video conferencing website uses a non-standard API for controlling the crypto method used, but is not having an effect because the desired behavior is already enabled-by-default.
    */
   RTCConstraintEnableDtlsSrtpTrue: "The constraint `DtlsSrtpKeyAgreement` is removed. You have specified a `true` value for this constraint, which had no effect, but you can remove this constraint for tidiness.",
+  /**
+   * @description Warning displayed to developers when the non-standard RTCPeerConnection.createDTMFSender() method is used. Developers should use the standard RTCRtpSender.dtmf attribute instead.
+   */
+  RTCPeerConnectionCreateDTMFSender: "RTCPeerConnection.createDTMFSender() is deprecated and will be removed. Please use RTCRtpSender.dtmf instead.",
   /**
    * @description WebRTC is set of JavaScript APIs for sending and receiving data, audio and video. getStats() is a method used to obtain network and quality metrics. There are two versions of this method, one is being deprecated because it is non-standard.
    */
@@ -254,10 +258,6 @@ export const UIStrings = {
    * @description A deprecation warning shown in the DevTools Issues tab. The placeholder is always the noun 'SharedArrayBuffer' which refers to a JavaScript construct.
    */
   SharedArrayBufferConstructedWithoutIsolation: "`SharedArrayBuffer` will require cross-origin isolation. See https://developer.chrome.com/blog/enabling-shared-array-buffer/ for more details.",
-  /**
-   * @description A deprecation warning shown in the DevTools Issues tab. It's shown when one of the Shared Storage APIs like `sharedStorage.set()`, `sharedStorage.worklet.addModule()`, `sharedStorage.selectURL()`, etc., along with `<img sharedstoragewritable>`, `<iframe sharedstoragewritable>`, or `fetch(url, {sharedStorageWritable: true})` are used.
-   */
-  SharedStorage: "The Shared Storage API is deprecated and will be removed in a future release.",
   /**
    * @description A deprecation warning shown in the DevTools Issues tab. It's shown when the `document.requestStorageAccessFor` API is called. The placeholder will always be the string `document.requestStorageAccessFor`.
    */
@@ -427,6 +427,10 @@ export const DEPRECATIONS_METADATA: Partial<Record<string, DeprecationDescriptor
   "RTCConstraintEnableDtlsSrtpTrue": {
     "milestone": 97
   },
+  "RTCPeerConnectionCreateDTMFSender": {
+    "chromeStatusFeature": 5160724576993280,
+    "milestone": 160
+  },
   "RTCPeerConnectionGetStatsLegacyNonCompliant": {
     "chromeStatusFeature": 4631626228695040,
     "milestone": 117
@@ -444,9 +448,6 @@ export const DEPRECATIONS_METADATA: Partial<Record<string, DeprecationDescriptor
   "SharedArrayBufferConstructedWithoutIsolation": {
     "milestone": 106
   },
-  "SharedStorage": {
-    "chromeStatusFeature": 5076349064708096
-  },
   "StorageAccessAPI_requestStorageAccessFor_Method": {
     "chromeStatusFeature": 5162221567082496
   },