chrome-devtools-frontend 1.0.1522145 → 1.0.1524741

package/front_end/core/platform/StringUtilities.ts

@@ -251,7 +251,7 @@ export const stripLineBreaks = (inputStr: string): string => {
 const EXTENDED_KEBAB_CASE_REGEXP = /^([a-z0-9]+(?:-[a-z0-9]+)*\.)*[a-z0-9]+(?:-[a-z0-9]+)*$/;
 
 /**
- * Tests if the `inputStr` is following the extended Kebab Case naming convetion,
+ * Tests if the `inputStr` is following the extended Kebab Case naming convention,
  * where words are separated with either a dash (`-`) or a dot (`.`), and all
  * characters must be lower-case alphanumeric.
  *
@@ -386,7 +386,7 @@ export const escapeForRegExp = (str: string): string => {
 
 export const naturalOrderComparator = (a: string, b: string): number => {
   const chunk = /^\d+|^\D+/;
-  let chunka, chunkb, anum, bnum;
+  let chunkA, chunkB, numA, numB;
   while (true) {
     if (a) {
       if (!b) {
@@ -398,32 +398,32 @@ export const naturalOrderComparator = (a: string, b: string): number => {
       }
       return 0;
     }
-    chunka = (a.match(chunk) as string[])[0];
-    chunkb = (b.match(chunk) as string[])[0];
-    anum = !Number.isNaN(Number(chunka));
-    bnum = !Number.isNaN(Number(chunkb));
-    if (anum && !bnum) {
+    chunkA = (a.match(chunk) as string[])[0];
+    chunkB = (b.match(chunk) as string[])[0];
+    numA = !Number.isNaN(Number(chunkA));
+    numB = !Number.isNaN(Number(chunkB));
+    if (numA && !numB) {
       return -1;
     }
-    if (bnum && !anum) {
+    if (numB && !numA) {
       return 1;
     }
-    if (anum && bnum) {
-      const diff = Number(chunka) - Number(chunkb);
+    if (numA && numB) {
+      const diff = Number(chunkA) - Number(chunkB);
       if (diff) {
         return diff;
       }
-      if (chunka.length !== chunkb.length) {
-        if (!Number(chunka) && !Number(chunkb)) {  // chunks are strings of all 0s (special case)
-          return chunka.length - chunkb.length;
+      if (chunkA.length !== chunkB.length) {
+        if (!Number(chunkA) && !Number(chunkB)) {  // chunks are strings of all 0s (special case)
+          return chunkA.length - chunkB.length;
         }
-        return chunkb.length - chunka.length;
+        return chunkB.length - chunkA.length;
       }
-    } else if (chunka !== chunkb) {
-      return (chunka < chunkb) ? -1 : 1;
+    } else if (chunkA !== chunkB) {
+      return (chunkA < chunkB) ? -1 : 1;
     }
-    a = a.substring(chunka.length);
-    b = b.substring(chunkb.length);
+    a = a.substring(chunkA.length);
+    b = b.substring(chunkB.length);
   }
 };
 
@@ -499,19 +499,22 @@ export const toLowerCaseString = function(input: string): LowerCaseString {
   return input.toLowerCase() as LowerCaseString;
 };
 
+/**
+ * 1: two or more consecutive uppercase letters. This is useful for identifying acronyms
+ * 2: lookahead assertion that matches a word boundary
+ * 3: numeronym: single letter followed by number and another letter
+ * 4: word starting with an optional uppercase letter
+ * 5: single digit followed by word to handle '3D' or '2px' (this might be controverial)
+ * 6: single uppercase letter or number
+ * 7: a dot character. We extract it into a separate word and remove dashes around it later.
+ * This is makes more sense conceptually and allows accounting for all possible word variants.
+ * Making dot a part of a word prevent us from handling acronyms or numeronyms after the word
+ * correctly without making the RegExp prohibitively complicated.
+ * https://regex101.com/r/FhMVKc/1
+ *            <---1---><------------2-----------> <---------3--------> <-----4----> <------5-----> <-----6----> <7>
+ */
 const WORD = /[A-Z]{2,}(?=[A-Z0-9][a-z0-9]+|\b|_)|[A-Za-z][0-9]+[a-z]?|[A-Z]?[a-z]+|[0-9][A-Za-z]+|[A-Z]|[0-9]+|[.]/g;
-//            <---1---><------------2-----------> <---------3--------> <-----4----> <------5-----> <-----6----> <7>
-// 1: two or more consecutive uppercase letters. This is useful for identifying acronyms
-// 2: lookahead assertion that matches a word boundary
-// 3: numeronym: single letter followed by number and another letter
-// 4: word starting with an optional uppercase letter
-// 5: single digit followed by word to handle '3D' or '2px' (this might be controverial)
-// 6: single uppercase letter or number
-// 7: a dot character. We extract it into a separate word and remove dashes around it later.
-//    This is makes more sense conceptually and allows accounting for all possible word variants.
-//    Making dot a part of a word prevent us from handling acronyms or numeronyms after the word
-//    correctly without making the RegExp prohibitively complicated.
-// https://regex101.com/r/FhMVKc/1
+
 export const toKebabCase = function(input: string): Lowercase<string> {
   return (input.match?.(WORD)?.map(w => w.toLowerCase()).join('-').replaceAll('-.-', '.') || input) as
       Lowercase<string>;
@@ -553,7 +556,7 @@ export function toSnakeCase(text: string): string {
   return result;
 }
 
-// Replaces the last ocurrence of parameter `search` with parameter `replacement` in `input`
+/** Replaces the last occurrence of parameter `search` with parameter `replacement` in `input` **/
 export const replaceLast = function(input: string, search: string, replacement: string): string {
   const replacementStartIndex = input.lastIndexOf(search);
   if (replacementStartIndex === -1) {

package/front_end/core/root/Runtime.ts

@@ -303,7 +303,7 @@ export class Experiment {
   }
 }
 
-// This must be constructed after the query parameters have been parsed.
+/** This must be constructed after the query parameters have been parsed. **/
 export const experiments = new ExperimentsSupport();
 
 /**

package/front_end/core/sdk/CSSMetadata.ts

@@ -362,7 +362,7 @@ export const CubicBezierKeywordValues = new Map([
   ['ease-out', 'cubic-bezier(0, 0, 0.58, 1)'],
 ]);
 
-// CSS-wide keywords.
+/** CSS-wide keywords. **/
 export const enum CSSWideKeyword {
   INHERIT = 'inherit',
   INITIAL = 'initial',
@@ -370,8 +370,10 @@ export const enum CSSWideKeyword {
   REVERT_LAYER = 'revert-layer',
   UNSET = 'unset',
 }
-// Spec: https://drafts.csswg.org/css-cascade/#defaulting-keywords
-// https://drafts.csswg.org/css-cascade-5/#revert-layer
+/**
+ * Spec: https://drafts.csswg.org/css-cascade/#defaulting-keywords
+ * https://drafts.csswg.org/css-cascade-5/#revert-layer
+ **/
 export const CSSWideKeywords: CSSWideKeyword[] = [
   CSSWideKeyword.INHERIT,
   CSSWideKeyword.INITIAL,
@@ -380,7 +382,7 @@ export const CSSWideKeywords: CSSWideKeyword[] = [
   CSSWideKeyword.UNSET,
 ];
 
-// https://www.w3.org/TR/css-anchor-position-1/#typedef-try-size
+/** https://www.w3.org/TR/css-anchor-position-1/#typedef-try-size **/
 export const enum PositionTryOrderKeyword {
   NORMAL = 'normal',
   MOST_HEIGHT = 'most-height',

package/front_end/core/sdk/CSSPropertyParser.ts

@@ -309,11 +309,11 @@ class ComputedTextChunk {
     return this.#cachedComputedText;
   }
 
-  // If the match is top-level, i.e. is an outermost subexpression in the property value, count the number of outermost
-  // subexpressions after applying any potential substitutions.
+  // If the match is top-level, i.e. is an outermost sub-expression in the property value, count the number of outermost
+  // sub-expressions after applying any potential substitutions.
   get topLevelValueCount(): number {
     if (this.match.node.parent?.name !== 'Declaration') {
-      // Not a top-level matchh.
+      // Not a top-level match.
       return 0;
     }
     const computedText = this.computedText;
@@ -332,12 +332,14 @@ class ComputedTextChunk {
   }
 }
 
-// This class constructs the "computed" text from the input property text, i.e., it will strip comments and substitute
-// var() functions if possible. It's intended for use during the bottom-up tree matching process. The original text is
-// not modified. Instead, computed text slices are produced on the fly. During bottom-up matching, the sequence of
-// top-level comments and var() matches will be recorded. This produces an ordered sequence of text pieces that need to
-// be substituted into the original text. When a computed text slice is requested, it is generated by piecing together
-// original and computed slices as required.
+/**
+ * This class constructs the "computed" text from the input property text, i.e., it will strip comments and substitute
+ * var() functions if possible. It's intended for use during the bottom-up tree matching process. The original text is
+ * not modified. Instead, computed text slices are produced on the fly. During bottom-up matching, the sequence of
+ * top-level comments and var() matches will be recorded. This produces an ordered sequence of text pieces that need to
+ * be substituted into the original text. When a computed text slice is requested, it is generated by piecing together
+ * original and computed slices as required.
+ **/
 export class ComputedText {
   readonly #chunks: ComputedTextChunk[] = [];
   readonly text: string;
@@ -486,10 +488,12 @@ export class ComputedText {
   }
 }
 
-// This function determines whether concatenating two pieces of text requires any spacing inbetween. For example, there
-// shouldn't be any space between 'var' and '(', but there should be a space between '1px' and 'solid'. The node
-// sequences that make up the pieces of text may contain non-text nodes/trees. Any such element inbetween the texts is
-// ignored for the spacing requirement.
+/**
+ * This function determines whether concatenating two pieces of text requires any spacing in between. For example, there
+ * shouldn't be any space between 'var' and '(', but there should be a space between '1px' and 'solid'. The node
+ * sequences that make up the pieces of text may contain non-text nodes/trees. Any such element in between the texts is
+ * ignored for the spacing requirement.
+ **/
 export function requiresSpace(a: string, b: string): boolean;
 export function requiresSpace(a: Node[], b: Node[]): boolean;
 export function requiresSpace(a: Node[]|string|undefined, b: Node[]|string|undefined): boolean {

package/front_end/core/sdk/CSSPropertyParserMatchers.ts

@@ -178,13 +178,15 @@ function getCssEvaluationElement(): HTMLElement {
   return cssEvaluationElement;
 }
 
-// These functions use an element in the frontend to evaluate CSS. The advantage
-// of this is that it is synchronous and doesn't require a CDP method. The
-// disadvantage is it lacks context that would allow substitutions such as
-// `var()` and `calc()` to be resolved correctly, and if the user is doing
-// remote debugging there is a possibility that the CSS behavior is different
-// between the two browser versions. We use it for type checking after
-// substitutions (but not for actual evaluation) and for applying units.
+/**
+ * These functions use an element in the frontend to evaluate CSS. The advantage
+ * of this is that it is synchronous and doesn't require a CDP method. The
+ * disadvantage is it lacks context that would allow substitutions such as
+ * `var()` and `calc()` to be resolved correctly, and if the user is doing
+ * remote debugging there is a possibility that the CSS behavior is different
+ * between the two browser versions. We use it for type checking after
+ * substitutions (but not for actual evaluation) and for applying units.
+ **/
 export function localEvalCSS(value: string, type: string): string|null {
   const element = getCssEvaluationElement();
   element.setAttribute('data-value', value);
@@ -192,9 +194,11 @@ export function localEvalCSS(value: string, type: string): string|null {
   return element.computedStyleMap().get('--evaluation')?.toString() ?? null;
 }
 
-// It is important to establish whether a type is valid, because if it is not,
-// the current behavior of blink is to ignore the fallback and parse as a
-// raw string, returning '' if the attribute is not set.
+/**
+ * It is important to establish whether a type is valid, because if it is not,
+ * the current behavior of blink is to ignore the fallback and parse as a
+ * raw string, returning '' if the attribute is not set.
+ **/
 export function isValidCSSType(type: string): boolean {
   const element = getCssEvaluationElement();
   element.setAttribute('data-custom-expr', `attr(data-nonexistent ${type}, "good")`);
@@ -1133,8 +1137,10 @@ export class GridTemplateMatcher extends matcherBase(GridTemplateMatch) {
     // `needClosingLineNames` tracks if the current row can still consume an optional LineNames,
     // which will decide if we should start a new line or not when a LineNames is encountered.
     let needClosingLineNames = false;
-    // Gather row definitions of [<line-names>? <string> <track-size>? <line-names>?], which
-    // be rendered into separate lines.
+    /**
+     * Gather row definitions of [<line-names>? <string> <track-size>? <line-names>?], which
+     * be rendered into separate lines.
+     **/
     function parseNodes(nodes: CodeMirror.SyntaxNode[], varParsingMode = false): void {
       for (const curNode of nodes) {
         if (matching.getMatch(curNode) instanceof BaseVariableMatch) {
@@ -1247,7 +1253,7 @@ export class AnchorFunctionMatcher extends matcherBase(AnchorFunctionMatch) {
   }
 }
 
-// For linking `position-anchor: --anchor-name`.
+/** For linking `position-anchor: --anchor-name`. **/
 export class PositionAnchorMatch implements Match {
   constructor(readonly text: string, readonly matching: BottomUpTreeMatching, readonly node: CodeMirror.SyntaxNode) {
   }

package/front_end/core/sdk/ChildTargetManager.ts

@@ -12,6 +12,8 @@ import type * as ProtocolClient from '../protocol_client/protocol_client.js';
 import {ParallelConnection} from './Connections.js';
 import {PrimaryPageChangeType, ResourceTreeModel} from './ResourceTreeModel.js';
 import {SDKModel} from './SDKModel.js';
+import {SecurityOriginManager} from './SecurityOriginManager.js';
+import {StorageKeyManager} from './StorageKeyManager.js';
 import {Capability, type Target, Type} from './Target.js';
 import {Events as TargetManagerEvents, TargetManager} from './TargetManager.js';
 
@@ -204,6 +206,39 @@ export class ChildTargetManager extends SDKModel<EventTypes> implements Protocol
     if (waitingForDebugger) {
       void target.runtimeAgent().invoke_runIfWaitingForDebugger();
     }
+
+    // For top-level workers (those not attached to a frame), we need to
+    // initialize their storage context manually. The `Capability.STORAGE` is
+    // only granted in `Target.ts` to workers that are not parented by a frame,
+    // which makes this check safe. Frame-associated workers have their storage
+    // managed by ResourceTreeModel.
+    if (type !== Type.FRAME && target.hasAllCapabilities(Capability.STORAGE)) {
+      await this.initializeStorage(target);
+    }
+  }
+
+  private async initializeStorage(target: Target): Promise<void> {
+    const storageAgent = target.storageAgent();
+    const response = await storageAgent.invoke_getStorageKey({});
+
+    const storageKey = response.storageKey;
+    if (response.getError() || !storageKey) {
+      console.error(`Failed to get storage key for target ${target.id()}: ${response.getError()}`);
+      return;
+    }
+
+    const storageKeyManager = target.model(StorageKeyManager);
+    if (storageKeyManager) {
+      storageKeyManager.setMainStorageKey(storageKey);
+      storageKeyManager.updateStorageKeys(new Set([storageKey]));
+    }
+
+    const securityOriginManager = target.model(SecurityOriginManager);
+    if (securityOriginManager) {
+      const origin = new URL(storageKey).origin;
+      securityOriginManager.setMainSecurityOrigin(origin, '');
+      securityOriginManager.updateSecurityOrigins(new Set([origin]));
+    }
   }
 
   detachedFromTarget({sessionId}: Protocol.Target.DetachedFromTargetEvent): void {

package/front_end/core/sdk/DOMModel.ts

@@ -49,7 +49,7 @@ import {SDKModel} from './SDKModel.js';
 import {Capability, type Target} from './Target.js';
 import {TargetManager} from './TargetManager.js';
 
-// Keep this list in sync with https://w3c.github.io/aria/#state_prop_def
+/** Keep this list in sync with https://w3c.github.io/aria/#state_prop_def **/
 export const ARIA_ATTRIBUTES = new Set<string>([
   'role',
   'aria-activedescendant',

package/front_end/core/sdk/EnhancedTracesParser.ts

@@ -35,11 +35,10 @@ export interface RundownScriptCompiled extends EventBase {
       frameType: 'page'|'iframe',
       url: string,
       /**
-       * isolate is a `uint64_t`, which is too much for JS number to represent exactly.
-       * TODO: consider adjusting the trace event impl to either string or reduced precision. see https://crrev.com/c/6300647
-       * This applies for all `isolate` in trace events we consume.
+       * Older traces were a number, but this is an unsigned 64 bit value, so that was a bug.
+       * New traces use string instead. See https://crbug.com/447654178.
        */
-      isolate: number,
+      isolate: string|number,
       /** AKA V8ContextToken. https://source.chromium.org/chromium/chromium/src/+/main:third_party/blink/renderer/core/inspector/inspector_trace_events.cc;l=1229;drc=3c88f61e18b043e70c225d8d57c77832a85e7f58 */
       v8context: string,
       origin: string,
@@ -60,7 +59,11 @@ export interface RundownScript extends EventBase {
   name: 'ScriptCatchup';
   args: {
     data: {
-      isolate: number,
+      /**
+       * Older traces were a number, but this is an unsigned 64 bit value, so that was a bug.
+       * New traces use string instead. See https://crbug.com/447654178.
+       */
+      isolate: string|number,
       executionContextId: Protocol.Runtime.ExecutionContextId,
       scriptId: number,
       isModule: boolean,
@@ -119,7 +122,11 @@ interface FunctionCall extends EventBase {
     data: {
       frame: Protocol.Page.FrameId,
       scriptId: Protocol.Runtime.ScriptId,
-      isolate?: number,
+      /**
+       * Older traces were a number, but this is an unsigned 64 bit value, so that was a bug.
+       * New traces use string instead. See https://crbug.com/447654178.
+       */
+      isolate?: string|number,
     },
   };
 }

package/front_end/core/sdk/EventBreakpointsModel.ts

@@ -51,8 +51,10 @@ export class EventBreakpointsModel extends SDKModel<void> {
   }
 }
 
-// This implementation (as opposed to similar class in DOMDebuggerModel) is for
-// instrumentation breakpoints in targets that run JS but do not have a DOM.
+/**
+ * This implementation (as opposed to similar class in DOMDebuggerModel) is for
+ * instrumentation breakpoints in targets that run JS but do not have a DOM.
+ **/
 class EventListenerBreakpoint extends CategorizedBreakpoint {
   override setEnabled(enabled: boolean): void {
     if (this.enabled() === enabled) {

package/front_end/core/sdk/HttpReasonPhraseStrings.ts

@@ -4,8 +4,10 @@
 
 import * as i18n from '../i18n/i18n.js';
 
-// Reason phrase sources
-// See https://www.rfc-editor.org/rfc/rfc9110.html#name-status-codes
+/**
+ * Reason phrase sources
+ * See https://www.rfc-editor.org/rfc/rfc9110.html#name-status-codes
+ **/
 export function getStatusText(statusCode: number): string {
   const statusTextLookup: Record<number, string> = {
     100: 'Continue',

package/front_end/core/sdk/NetworkManager.ts

@@ -23,8 +23,6 @@ import {
   type IncludedCookieWithReason,
   type NameValue,
   NetworkRequest,
-  type WebBundleInfo,
-  type WebBundleInnerRequestInfo
 } from './NetworkRequest.js';
 import {SDKModel} from './SDKModel.js';
 import {Capability, type Target} from './Target.js';
@@ -1502,44 +1500,20 @@ export class NetworkDispatcher implements ProtocolProxyApi.NetworkDispatcher {
     request.setTrustTokenOperationDoneEvent(event);
   }
 
-  subresourceWebBundleMetadataReceived({requestId, urls}: Protocol.Network.SubresourceWebBundleMetadataReceivedEvent):
-      void {
-    const extraInfoBuilder = this.getExtraInfoBuilder(requestId);
-    extraInfoBuilder.setWebBundleInfo({resourceUrls: urls as Platform.DevToolsPath.UrlString[]});
-    const finalRequest = extraInfoBuilder.finalRequest();
-    if (finalRequest) {
-      this.updateNetworkRequest(finalRequest);
-    }
+  subresourceWebBundleMetadataReceived(): void {
+    // TODO: remove implementation after deleting this methods from definition in Network.pdl
   }
 
-  subresourceWebBundleMetadataError({requestId, errorMessage}: Protocol.Network.SubresourceWebBundleMetadataErrorEvent):
-      void {
-    const extraInfoBuilder = this.getExtraInfoBuilder(requestId);
-    extraInfoBuilder.setWebBundleInfo({errorMessage});
-    const finalRequest = extraInfoBuilder.finalRequest();
-    if (finalRequest) {
-      this.updateNetworkRequest(finalRequest);
-    }
+  subresourceWebBundleMetadataError(): void {
+    // TODO: remove implementation after deleting this methods from definition in Network.pdl
   }
 
-  subresourceWebBundleInnerResponseParsed({innerRequestId, bundleRequestId}:
-                                              Protocol.Network.SubresourceWebBundleInnerResponseParsedEvent): void {
-    const extraInfoBuilder = this.getExtraInfoBuilder(innerRequestId);
-    extraInfoBuilder.setWebBundleInnerRequestInfo({bundleRequestId});
-    const finalRequest = extraInfoBuilder.finalRequest();
-    if (finalRequest) {
-      this.updateNetworkRequest(finalRequest);
-    }
+  subresourceWebBundleInnerResponseParsed(): void {
+    // TODO: remove implementation after deleting this methods from definition in Network.pdl
   }
 
-  subresourceWebBundleInnerResponseError({innerRequestId, errorMessage}:
-                                             Protocol.Network.SubresourceWebBundleInnerResponseErrorEvent): void {
-    const extraInfoBuilder = this.getExtraInfoBuilder(innerRequestId);
-    extraInfoBuilder.setWebBundleInnerRequestInfo({errorMessage});
-    const finalRequest = extraInfoBuilder.finalRequest();
-    if (finalRequest) {
-      this.updateNetworkRequest(finalRequest);
-    }
+  subresourceWebBundleInnerResponseError(): void {
+    // TODO: remove implementation after deleting this methods from definition in Network.pdl
   }
 
   reportingApiReportAdded(data: Protocol.Network.ReportingApiReportAddedEvent): void {
@@ -2151,8 +2125,6 @@ class ExtraInfoBuilder {
   #responseExtraInfos: Array<ExtraResponseInfo|null> = [];
   #responseEarlyHintsHeaders: NameValue[] = [];
   #finished = false;
-  #webBundleInfo: WebBundleInfo|null = null;
-  #webBundleInnerRequestInfo: WebBundleInnerRequestInfo|null = null;
 
   addRequest(req: NetworkRequest): void {
     this.#requests.push(req);
@@ -2189,16 +2161,6 @@ class ExtraInfoBuilder {
     this.updateFinalRequest();
   }
 
-  setWebBundleInfo(info: WebBundleInfo): void {
-    this.#webBundleInfo = info;
-    this.updateFinalRequest();
-  }
-
-  setWebBundleInnerRequestInfo(info: WebBundleInnerRequestInfo): void {
-    this.#webBundleInnerRequestInfo = info;
-    this.updateFinalRequest();
-  }
-
   finished(): void {
     this.#finished = true;
     // We may have missed responseReceived event in case of failure.
@@ -2260,8 +2222,6 @@ class ExtraInfoBuilder {
       return;
     }
     const finalRequest = this.finalRequest();
-    finalRequest?.setWebBundleInfo(this.#webBundleInfo);
-    finalRequest?.setWebBundleInnerRequestInfo(this.#webBundleInnerRequestInfo);
     finalRequest?.setEarlyHintsHeaders(this.#responseEarlyHintsHeaders);
   }
 }

package/front_end/core/sdk/NetworkRequest.ts

@@ -236,8 +236,6 @@ export class NetworkRequest extends Common.ObjectWrapper.ObjectWrapper<EventType
   #initialPriority: Protocol.Network.ResourcePriority|null = null;
   #currentPriority: Protocol.Network.ResourcePriority|null = null;
   #signedExchangeInfo: Protocol.Network.SignedExchangeInfo|null = null;
-  #webBundleInfo: WebBundleInfo|null = null;
-  #webBundleInnerRequestInfo: WebBundleInnerRequestInfo|null = null;
   #resourceType: Common.ResourceType.ResourceType = Common.ResourceType.resourceTypes.Other;
   #contentData: Promise<TextUtils.ContentData.ContentDataOrError>|null = null;
   #streamingContentData: Promise<TextUtils.StreamingContentData.StreamingContentDataOrError>|null = null;
@@ -1500,22 +1498,6 @@ export class NetworkRequest extends Common.ObjectWrapper.ObjectWrapper<EventType
     return this.#signedExchangeInfo;
   }
 
-  setWebBundleInfo(info: WebBundleInfo|null): void {
-    this.#webBundleInfo = info;
-  }
-
-  webBundleInfo(): WebBundleInfo|null {
-    return this.#webBundleInfo;
-  }
-
-  setWebBundleInnerRequestInfo(info: WebBundleInnerRequestInfo|null): void {
-    this.#webBundleInnerRequestInfo = info;
-  }
-
-  webBundleInnerRequestInfo(): WebBundleInnerRequestInfo|null {
-    return this.#webBundleInnerRequestInfo;
-  }
-
   async populateImageSource(image: HTMLImageElement): Promise<void> {
     const contentData = await this.requestContentData();
     if (TextUtils.ContentData.ContentData.isError(contentData)) {
@@ -2207,16 +2189,6 @@ export interface EarlyHintsInfo {
   responseHeaders: NameValue[];
 }
 
-export interface WebBundleInfo {
-  resourceUrls?: Platform.DevToolsPath.UrlString[];
-  errorMessage?: string;
-}
-
-export interface WebBundleInnerRequestInfo {
-  bundleRequestId?: string;
-  errorMessage?: string;
-}
-
 export type OverrideType = 'content'|'headers';
 
 export enum DirectSocketType {

package/front_end/core/sdk/PageResourceLoader.ts

@@ -59,7 +59,7 @@ export interface PageResource {
   duration: number|null;
 }
 
-// Used for revealing a resource.
+/** Used for revealing a resource. **/
 export class ResourceKey {
   readonly key: string;
 

package/front_end/core/sdk/PreloadingModel.ts

@@ -23,11 +23,13 @@ export interface WithId<I, V> {
   value: V;
 }
 
-// Holds preloading related information.
-//
-// - SpeculationRule rule sets
-// - Preloading attempts
-// - Relationship between rule sets and preloading attempts
+/**
+ * Holds preloading related information.
+ *
+ * - SpeculationRule rule sets
+ * - Preloading attempts
+ * - Relationship between rule sets and preloading attempts
+ **/
 export class PreloadingModel extends SDKModel<EventTypes> {
   private agent: ProtocolProxyApi.PreloadApi;
   private loaderIds: Protocol.Network.LoaderId[] = [];
@@ -410,19 +412,21 @@ class RuleSetRegistry {
   }
 }
 
-// Protocol.Preload.PreloadingStatus|'NotTriggered'
-//
-// A renderer sends SpeculationCandidate to the browser process and the
-// browser process checks eligibilities, and starts PreloadingAttempt.
-//
-// In the frontend, "NotTriggered" is used to denote that a
-// PreloadingAttempt is waiting for at trigger event (eg:
-// mousedown/mouseover). All PreloadingAttempts will start off as
-// "NotTriggered", but "eager" preloading attempts (attempts not
-// actually waiting for any trigger) will be processed by the browser
-// immediately, and will not stay in this state for long.
-//
-// TODO(https://crbug.com/1384419): Add NotEligible.
+/**
+ * Protocol.Preload.PreloadingStatus|'NotTriggered'
+ *
+ * A renderer sends SpeculationCandidate to the browser process and the
+ * browser process checks eligibilities, and starts PreloadingAttempt.
+ *
+ * In the frontend, "NotTriggered" is used to denote that a
+ * PreloadingAttempt is waiting for at trigger event (eg:
+ * mousedown/mouseover). All PreloadingAttempts will start off as
+ * "NotTriggered", but "eager" preloading attempts (attempts not
+ * actually waiting for any trigger) will be processed by the browser
+ * immediately, and will not stay in this state for long.
+ *
+ * TODO(https://crbug.com/1384419): Add NotEligible.
+ **/
 export const enum PreloadingStatus {
   NOT_TRIGGERED = 'NotTriggered',
   PENDING = 'Pending',

package/front_end/core/sdk/RehydratingConnection.ts

@@ -210,7 +210,7 @@ export class RehydratingConnection implements ProtocolClient.InspectorBackend.Co
   }
 }
 
-// Default rehydrating session with default responses.
+/** Default rehydrating session with default responses. **/
 class RehydratingSessionBase {
   connection: RehydratingConnectionInterface|null = null;
 

package/front_end/core/sdk/RehydratingObject.ts

@@ -4,7 +4,7 @@
 
 import type * as Protocol from '../../generated/protocol.js';
 
-// This object is emitted to ScriptParsed and also used in the RehydratingConnection
+/** This object is emitted to ScriptParsed and also used in the RehydratingConnection **/
 export interface RehydratingScript extends Protocol.Debugger.ScriptParsedEvent {
   sourceText?: string;
   executionContextAuxData?: RehydratingExecutionContextAuxData;

package/front_end/core/sdk/RemoteObject.ts

@@ -9,7 +9,7 @@ import type {DOMPinnedWebIDLProp, DOMPinnedWebIDLType} from '../common/JavaScrip
 import type {DebuggerModel, FunctionDetails} from './DebuggerModel.js';
 import type {RuntimeModel} from './RuntimeModel.js';
 
-// This cannot be an interface due to "instanceof RemoteObject" checks in the code.
+/** This cannot be an interface due to "instanceof RemoteObject" checks in the code. **/
 export abstract class RemoteObject {
   static fromLocalObject(value: unknown): RemoteObject {
     return new LocalJSONObject(value);