chrome-devtools-frontend 1.0.950768 → 1.0.951864

package/front_end/core/sdk/CSSRule.ts

@@ -73,7 +73,7 @@ export class CSSRule {
   getStyleSheetHeader(styleSheetId: Protocol.CSS.StyleSheetId): CSSStyleSheetHeader {
     const styleSheetHeader = this.cssModelInternal.styleSheetHeaderForId(styleSheetId);
     console.assert(styleSheetHeader !== null);
-    return /** @type {!CSSStyleSheetHeader} */ styleSheetHeader as CSSStyleSheetHeader;
+    return styleSheetHeader as CSSStyleSheetHeader;
   }
 }
 

package/front_end/core/sdk/DOMModel.ts

@@ -408,11 +408,7 @@ export class DOMNode {
     return this.localName();
   }
 
-  setNodeName(
-      name: string,
-      // TODO(crbug.com/1172300) Ignored during the jsdoc to ts migration
-      // eslint-disable-next-line @typescript-eslint/no-explicit-any
-      callback?: ((arg0: string|null, arg1: DOMNode|null) => any)): void {
+  setNodeName(name: string, callback?: ((arg0: string|null, arg1: DOMNode|null) => void)): void {
     this.#agent.invoke_setNodeName({nodeId: this.id, name}).then(response => {
       if (!response.getError()) {
         this.#domModelInternal.markUndoableState();
@@ -435,9 +431,7 @@ export class DOMNode {
     this.nodeValueInternal = nodeValue;
   }
 
-  // TODO(crbug.com/1172300) Ignored during the jsdoc to ts migration
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
-  setNodeValue(value: string, callback?: ((arg0: string|null) => any)): void {
+  setNodeValue(value: string, callback?: ((arg0: string|null) => void)): void {
     this.#agent.invoke_setNodeValue({nodeId: this.id, value}).then(response => {
       if (!response.getError()) {
         this.#domModelInternal.markUndoableState();
@@ -453,11 +447,7 @@ export class DOMNode {
     return attr ? attr.value : undefined;
   }
 
-  setAttribute(
-      name: string, text: string,
-      // TODO(crbug.com/1172300) Ignored during the jsdoc to ts migration
-      // eslint-disable-next-line @typescript-eslint/no-explicit-any
-      callback?: ((arg0: string|null) => any)): void {
+  setAttribute(name: string, text: string, callback?: ((arg0: string|null) => void)): void {
     this.#agent.invoke_setAttributesAsText({nodeId: this.id, text, name}).then(response => {
       if (!response.getError()) {
         this.#domModelInternal.markUndoableState();
@@ -468,11 +458,7 @@ export class DOMNode {
     });
   }
 
-  setAttributeValue(
-      name: string, value: string,
-      // TODO(crbug.com/1172300) Ignored during the jsdoc to ts migration
-      // eslint-disable-next-line @typescript-eslint/no-explicit-any
-      callback?: ((arg0: string|null) => any)): void {
+  setAttributeValue(name: string, value: string, callback?: ((arg0: string|null) => void)): void {
     this.#agent.invoke_setAttributeValue({nodeId: this.id, name, value}).then(response => {
       if (!response.getError()) {
         this.#domModelInternal.markUndoableState();
@@ -520,9 +506,7 @@ export class DOMNode {
     return outerHTML;
   }
 
-  // TODO(crbug.com/1172300) Ignored during the jsdoc to ts migration
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
-  setOuterHTML(html: string, callback?: ((arg0: string|null) => any)): void {
+  setOuterHTML(html: string, callback?: ((arg0: string|null) => void)): void {
     this.#agent.invoke_setOuterHTML({nodeId: this.id, outerHTML: html}).then(response => {
       if (!response.getError()) {
         this.#domModelInternal.markUndoableState();
@@ -533,10 +517,7 @@ export class DOMNode {
     });
   }
 
-  removeNode(callback?: (
-      // TODO(crbug.com/1172300) Ignored during the jsdoc to ts migration
-      // eslint-disable-next-line @typescript-eslint/no-explicit-any
-      (arg0: string|null, arg1?: Protocol.DOM.NodeId|undefined) => any)): Promise<void> {
+  removeNode(callback?: ((arg0: string|null, arg1?: Protocol.DOM.NodeId|undefined) => void)): Promise<void> {
     return this.#agent.invoke_removeNode({nodeId: this.id}).then(response => {
       if (!response.getError()) {
         this.#domModelInternal.markUndoableState();
@@ -738,11 +719,8 @@ export class DOMNode {
     this.#attributesInternal.delete(name);
   }
 
-  copyTo(
-      targetNode: DOMNode, anchorNode: DOMNode|null,
-      // TODO(crbug.com/1172300) Ignored during the jsdoc to ts migration
-      // eslint-disable-next-line @typescript-eslint/no-explicit-any
-      callback?: ((arg0: string|null, arg1: DOMNode|null) => any)): void {
+  copyTo(targetNode: DOMNode, anchorNode: DOMNode|null, callback?: ((arg0: string|null, arg1: DOMNode|null) => void)):
+      void {
     this.#agent
         .invoke_copyTo(
             {nodeId: this.id, targetNodeId: targetNode.id, insertBeforeNodeId: anchorNode ? anchorNode.id : undefined})
@@ -756,11 +734,8 @@ export class DOMNode {
         });
   }
 
-  moveTo(
-      targetNode: DOMNode, anchorNode: DOMNode|null,
-      // TODO(crbug.com/1172300) Ignored during the jsdoc to ts migration
-      // eslint-disable-next-line @typescript-eslint/no-explicit-any
-      callback?: ((arg0: string|null, arg1: DOMNode|null) => any)): void {
+  moveTo(targetNode: DOMNode, anchorNode: DOMNode|null, callback?: ((arg0: string|null, arg1: DOMNode|null) => void)):
+      void {
     this.#agent
         .invoke_moveTo(
             {nodeId: this.id, targetNodeId: targetNode.id, insertBeforeNodeId: anchorNode ? anchorNode.id : undefined})
@@ -1076,11 +1051,11 @@ export class DOMModel extends SDKModel<EventTypes> {
   }
 
   cssModel(): CSSModel {
-    return /** @type {!CSSModel} */ this.target().model(CSSModel) as CSSModel;
+    return this.target().model(CSSModel) as CSSModel;
   }
 
   overlayModel(): OverlayModel {
-    return /** @type {!OverlayModel} */ this.target().model(OverlayModel) as OverlayModel;
+    return this.target().model(OverlayModel) as OverlayModel;
   }
 
   static cancelSearch(): void {

package/front_end/core/sdk/OverlayModel.ts

@@ -59,27 +59,13 @@ export class OverlayModel extends SDKModel<EventTypes> implements ProtocolProxyA
   #hideHighlightTimeout: number|null;
   #defaultHighlighter: Highlighter;
   #highlighter: Highlighter;
-  // TODO(crbug.com/1172300) Ignored during the jsdoc to ts migration
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
-  #showPaintRectsSetting: Common.Settings.Setting<any>;
-  // TODO(crbug.com/1172300) Ignored during the jsdoc to ts migration
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
-  #showLayoutShiftRegionsSetting: Common.Settings.Setting<any>;
-  // TODO(crbug.com/1172300) Ignored during the jsdoc to ts migration
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
-  #showAdHighlightsSetting: Common.Settings.Setting<any>;
-  // TODO(crbug.com/1172300) Ignored during the jsdoc to ts migration
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
-  #showDebugBordersSetting: Common.Settings.Setting<any>;
-  // TODO(crbug.com/1172300) Ignored during the jsdoc to ts migration
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
-  #showFPSCounterSetting: Common.Settings.Setting<any>;
-  // TODO(crbug.com/1172300) Ignored during the jsdoc to ts migration
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
-  #showScrollBottleneckRectsSetting: Common.Settings.Setting<any>;
-  // TODO(crbug.com/1172300) Ignored during the jsdoc to ts migration
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
-  #showWebVitalsSetting: Common.Settings.Setting<any>;
+  #showPaintRectsSetting: Common.Settings.Setting<boolean>;
+  #showLayoutShiftRegionsSetting: Common.Settings.Setting<boolean>;
+  #showAdHighlightsSetting: Common.Settings.Setting<boolean>;
+  #showDebugBordersSetting: Common.Settings.Setting<boolean>;
+  #showFPSCounterSetting: Common.Settings.Setting<boolean>;
+  #showScrollBottleneckRectsSetting: Common.Settings.Setting<boolean>;
+  #showWebVitalsSetting: Common.Settings.Setting<boolean>;
   #registeredListeners: Common.EventTarget.EventDescriptor[];
   #showViewportSizeOnResize: boolean;
   #persistentHighlighter: OverlayPersistentHighlighter|null;
@@ -113,14 +99,15 @@ export class OverlayModel extends SDKModel<EventTypes> implements ProtocolProxyA
     this.#defaultHighlighter = new DefaultHighlighter(this);
     this.#highlighter = this.#defaultHighlighter;
 
-    this.#showPaintRectsSetting = Common.Settings.Settings.instance().moduleSetting('showPaintRects');
-    this.#showLayoutShiftRegionsSetting = Common.Settings.Settings.instance().moduleSetting('showLayoutShiftRegions');
-    this.#showAdHighlightsSetting = Common.Settings.Settings.instance().moduleSetting('showAdHighlights');
-    this.#showDebugBordersSetting = Common.Settings.Settings.instance().moduleSetting('showDebugBorders');
-    this.#showFPSCounterSetting = Common.Settings.Settings.instance().moduleSetting('showFPSCounter');
+    this.#showPaintRectsSetting = Common.Settings.Settings.instance().moduleSetting<boolean>('showPaintRects');
+    this.#showLayoutShiftRegionsSetting =
+        Common.Settings.Settings.instance().moduleSetting<boolean>('showLayoutShiftRegions');
+    this.#showAdHighlightsSetting = Common.Settings.Settings.instance().moduleSetting<boolean>('showAdHighlights');
+    this.#showDebugBordersSetting = Common.Settings.Settings.instance().moduleSetting<boolean>('showDebugBorders');
+    this.#showFPSCounterSetting = Common.Settings.Settings.instance().moduleSetting<boolean>('showFPSCounter');
     this.#showScrollBottleneckRectsSetting =
-        Common.Settings.Settings.instance().moduleSetting('showScrollBottleneckRects');
-    this.#showWebVitalsSetting = Common.Settings.Settings.instance().moduleSetting('showWebVitals');
+        Common.Settings.Settings.instance().moduleSetting<boolean>('showScrollBottleneckRects');
+    this.#showWebVitalsSetting = Common.Settings.Settings.instance().moduleSetting<boolean>('showWebVitals');
 
     this.#registeredListeners = [];
     this.#showViewportSizeOnResize = true;
@@ -178,18 +165,15 @@ export class OverlayModel extends SDKModel<EventTypes> implements ProtocolProxyA
     return this.#domModel;
   }
 
-  // TODO(crbug.com/1172300) Ignored during the jsdoc to ts migration
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
-  highlightRect({x, y, width, height, color, outlineColor}: HighlightRect): Promise<any> {
+  highlightRect({x, y, width, height, color, outlineColor}: HighlightRect):
+      Promise<Protocol.ProtocolResponseWithError> {
     const highlightColor = color || {r: 255, g: 0, b: 255, a: 0.3};
     const highlightOutlineColor = outlineColor || {r: 255, g: 0, b: 255, a: 0.5};
     return this.overlayAgent.invoke_highlightRect(
         {x, y, width, height, color: highlightColor, outlineColor: highlightOutlineColor});
   }
 
-  // TODO(crbug.com/1172300) Ignored during the jsdoc to ts migration
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
-  clearHighlight(): Promise<any> {
+  clearHighlight(): Promise<Protocol.ProtocolResponseWithError> {
     return this.overlayAgent.invoke_hideHighlight();
   }
 

package/front_end/core/sdk/OverlayPersistentHighlighter.ts

@@ -360,15 +360,9 @@ export class OverlayPersistentHighlighter {
   }
 }
 
-/**
- * @interface
- */
 export interface DOMModel {
   nodeForId(nodeId: Protocol.DOM.NodeId): void;
 }
-/**
- * @interface
- */
 export interface OverlayAgent {
   // TODO(crbug.com/1172300) Ignored during the jsdoc to ts migration
   // eslint-disable-next-line @typescript-eslint/naming-convention
@@ -415,16 +409,10 @@ export interface OverlayAgent {
   }): void;
 }
 
-/**
- * @interface
- */
 export interface Target {
   overlayAgent(): OverlayAgent;
 }
 
-/**
- * @interface
- */
 export interface OverlayModel {
   getDOMModel(): DOMModel;
 

package/front_end/core/sdk/RuntimeModel.ts

@@ -644,7 +644,6 @@ export class ExecutionContext {
       return this.evaluateGlobal(options, userGesture, awaitPromise);
     }
 
-    /** @type {!EvaluationResult} */
     if (this.runtimeModel.hasSideEffectSupport() !== false) {
       await this.runtimeModel.checkSideEffectSupport();
       if (this.runtimeModel.hasSideEffectSupport()) {

package/front_end/core/sdk/SourceMap.ts

@@ -303,7 +303,7 @@ export class TextSourceMap implements SourceMap {
       this.eachSection(this.parseMap.bind(this));
       this.#json = null;
     }
-    return /** @type {!Array<!SourceMapEntry>} */ this.#mappingsInternal as SourceMapEntry[];
+    return this.#mappingsInternal;
   }
 
   private reversedMappings(sourceURL: string): number[] {

package/front_end/core/sdk/TracingManager.ts

@@ -94,9 +94,6 @@ export class TracingManager extends SDKModel<void> {
   }
 }
 
-/**
- * @interface
- */
 export interface TracingManagerClient {
   traceEventsCollected(events: EventPayload[]): void;
 

package/front_end/entrypoints/heap_snapshot_worker/AllocationProfile.ts

@@ -190,14 +190,14 @@ export class AllocationProfile {
   }
 
   private ensureBottomUpNode(nodeId: number): BottomUpAllocationNode {
-    let node: (BottomUpAllocationNode|null) = this.idToNode[nodeId];
+    let node = this.idToNode[nodeId];
     if (!node) {
       const functionInfo = this.collapsedTopNodeIdToFunctionInfo[nodeId];
       node = functionInfo.bottomUpRoot();
       delete this.collapsedTopNodeIdToFunctionInfo[nodeId];
       this.idToNode[nodeId] = node;
     }
-    return /** @type {!BottomUpAllocationNode} */ node as BottomUpAllocationNode;
+    return node as BottomUpAllocationNode;
   }
 
   private serializeCaller(node: BottomUpAllocationNode): HeapSnapshotModel.HeapSnapshotModel.SerializedAllocationNode {
@@ -331,7 +331,7 @@ export class FunctionAllocationInfo {
     if (!this.bottomUpTree) {
       this.buildAllocationTraceTree();
     }
-    return /** @type {!BottomUpAllocationNode} */ this.bottomUpTree as BottomUpAllocationNode;
+    return this.bottomUpTree as BottomUpAllocationNode;
   }
 
   private buildAllocationTraceTree(): void {

package/front_end/entrypoints/heap_snapshot_worker/HeapSnapshot.ts

@@ -2280,9 +2280,7 @@ export class HeapSnapshotNodesProvider extends HeapSnapshotItemProvider {
     const ascending1 = comparator.ascending1 ? 1 : -1;
     const ascending2 = comparator.ascending2 ? 1 : -1;
 
-    // TODO(crbug.com/1172300) Ignored during the jsdoc to ts migration)
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    function sortByNodeField(fieldAccessor: () => any, ascending: number): number {
+    function sortByNodeField(fieldAccessor: () => void, ascending: number): number {
       const valueA = fieldAccessor.call(nodeA);
       const valueB = fieldAccessor.call(nodeB);
       return valueA < valueB ? -ascending : (valueA > valueB ? ascending : 0);
@@ -2616,8 +2614,7 @@ export class JSHeapSnapshot extends HeapSnapshot {
   }
 
   getStatistics(): HeapSnapshotModel.HeapSnapshotModel.Statistics {
-    return /** @type {!HeapSnapshotModel.HeapSnapshotModel.Statistics} */ this.statistics as
-        HeapSnapshotModel.HeapSnapshotModel.Statistics;
+    return this.statistics as HeapSnapshotModel.HeapSnapshotModel.Statistics;
   }
 }
 

package/front_end/entrypoints/heap_snapshot_worker/HeapSnapshotWorkerDispatcher.ts

@@ -58,7 +58,7 @@ export class HeapSnapshotWorkerDispatcher {
     for (let i = 0; i < path.length; ++i) {
       result = result[path[i]];
     }
-    return /** @type {!Function} */ result as Function;
+    return result as Function;
   }
 
   // TODO(crbug.com/1172300) Ignored during the jsdoc to ts migration)

package/front_end/models/bindings/CSSWorkspaceBinding.ts

@@ -152,9 +152,6 @@ export class CSSWorkspaceBinding implements SDK.TargetManager.SDKModelObserver<S
   }
 }
 
-/**
- * @interface
- */
 export interface SourceMapping {
   rawLocationToUILocation(rawLocation: SDK.CSSModel.CSSLocation): Workspace.UISourceCode.UILocation|null;
 

package/front_end/models/bindings/DebuggerLanguagePlugins.ts

@@ -677,8 +677,7 @@ class SourceScopeRemoteObject extends SDK.RemoteObject.RemoteObjectImpl {
       properties.push(makeProperty(namespace, (namespaces[namespace] as SDK.RemoteObject.RemoteObject)));
     }
 
-    return /** @type {!SDK.RemoteObject.GetPropertiesResult} */ {properties: properties, internalProperties: []} as
-        SDK.RemoteObject.GetPropertiesResult;
+    return {properties: properties, internalProperties: []};
   }
 }
 

package/front_end/models/bindings/DebuggerWorkspaceBinding.ts

@@ -421,7 +421,7 @@ class ModelData {
     uiLocation = uiLocation || this.resourceMapping.rawLocationToUILocation(rawLocation);
     uiLocation = uiLocation || ResourceMapping.instance().jsLocationToUILocation(rawLocation);
     uiLocation = uiLocation || this.#defaultMapping.rawLocationToUILocation(rawLocation);
-    return /** @type {!Workspace.UISourceCode.UILocation} */ uiLocation as Workspace.UISourceCode.UILocation;
+    return uiLocation;
   }
 
   uiLocationToRawLocations(
@@ -561,9 +561,6 @@ export interface RawLocationRange {
   end: SDK.DebuggerModel.Location;
 }
 
-/**
- * @interface
- */
 export interface DebuggerSourceMapping {
   rawLocationToUILocation(rawLocation: SDK.DebuggerModel.Location): Workspace.UISourceCode.UILocation|null;
 

package/front_end/models/bindings/FileUtils.ts

@@ -31,9 +31,6 @@
 import type * as Common from '../../core/common/common.js';
 import * as Workspace from '../workspace/workspace.js';
 
-/**
- * @interface
- */
 export interface ChunkedReader {
   fileSize(): number;
 
@@ -217,7 +214,6 @@ export class FileOutputStream implements Common.StringOutputStream.OutputStream
 
   async open(fileName: string): Promise<boolean> {
     this.#closed = false;
-    /** @type {!Array<function():void>} */
     this.#writeCallbacks = [];
     this.#fileName = fileName;
     const saveResponse = await Workspace.FileManager.FileManager.instance().save(this.#fileName, '', true);

package/front_end/models/bindings/IgnoreListManager.ts

@@ -83,8 +83,7 @@ export class IgnoreListManager implements SDK.TargetManager.SDKModelObserver<SDK
   }
 
   private getSkipStackFramesPatternSetting(): Common.Settings.RegExpSetting {
-    return /** @type {!Common.Settings.RegExpSetting} */ Common.Settings.Settings.instance().moduleSetting(
-               'skipStackFramesPattern') as Common.Settings.RegExpSetting;
+    return Common.Settings.Settings.instance().moduleSetting('skipStackFramesPattern') as Common.Settings.RegExpSetting;
   }
 
   private setIgnoreListPatterns(debuggerModel: SDK.DebuggerModel.DebuggerModel): Promise<boolean> {

package/front_end/models/bindings/NetworkProject.ts

@@ -180,7 +180,6 @@ export class NetworkProject {
       return [];
     }
     const frames = Array.from(attribution.keys()).map(frameId => resourceTreeModel.frameForId(frameId));
-    return /** @type {!Array<!SDK.ResourceTreeModel.ResourceTreeFrame>} */ frames.filter(frame => Boolean(frame)) as
-        SDK.ResourceTreeModel.ResourceTreeFrame[];
+    return frames.filter(frame => Boolean(frame)) as SDK.ResourceTreeModel.ResourceTreeFrame[];
   }
 }

package/front_end/models/extensions/ExtensionAPI.ts

@@ -304,8 +304,9 @@ namespace APIImpl {
   // We cannot use the stronger `unknown` type in place of `any` in the following type definition. The type is used as
   // the right-hand side of `extends` in a few places, which doesn't narrow `unknown`. Without narrowing, overload
   // resolution and meaningful type inference of arguments break, for example.
+  // TODO(crbug.com/1172300) Ignored during the jsdoc to ts migration
   // eslint-disable-next-line @typescript-eslint/no-explicit-any
-  export type Callable = (...args: any) => any;
+  export type Callable = (...args: any) => void;
 
   export interface EventSink<ListenerT extends Callable> extends PublicAPI.Chrome.DevTools.EventSink<ListenerT> {
     _type: string;
@@ -640,7 +641,6 @@ self.injectedExtensionAPI = function(
   };
 
   function LanguageServicesAPIImpl(this: APIImpl.LanguageExtensions): void {
-    /** @type {!Map<*, !MessagePort>} */
     this._plugins = new Map();
   }
 

package/front_end/models/extensions/ExtensionTraceProvider.ts

@@ -42,9 +42,6 @@ export class ExtensionTraceProvider {
 // eslint-disable-next-line @typescript-eslint/naming-convention
 let _lastSessionId = 0;
 
-/**
- * @interface
- */
 export interface TracingSession {
   complete(url: string, timeOffsetMicroseconds: number): void;
 }

package/front_end/models/issues_manager/SameSiteCookieIssue.ts

@@ -67,8 +67,7 @@ export class SameSiteCookieIssue extends Issue {
   static createIssuesFromSameSiteDetails(
       sameSiteDetails: Protocol.Audits.SameSiteCookieIssueDetails,
       issuesModel: SDK.IssuesModel.IssuesModel): SameSiteCookieIssue[] {
-    /** @type {!Array<!Issue>} */
-    const issues = [];
+    const issues: SameSiteCookieIssue[] = [];
 
     // Exclusion reasons have priority. It means a cookie was blocked. Create an issue
     // for every exclusion reason but ignore warning reasons if the cookie was blocked.

package/front_end/models/timeline_model/TimelineModel.ts

@@ -122,7 +122,7 @@ export class TimelineModelImpl {
   static forEachEvent(
       events: SDK.TracingModel.Event[], onStartEvent: (arg0: SDK.TracingModel.Event) => void,
       onEndEvent: (arg0: SDK.TracingModel.Event) => void,
-      onInstantEvent?: ((arg0: SDK.TracingModel.Event, arg1: SDK.TracingModel.Event|null) => any), startTime?: number,
+      onInstantEvent?: ((arg0: SDK.TracingModel.Event, arg1: SDK.TracingModel.Event|null) => void), startTime?: number,
       endTime?: number, filter?: ((arg0: SDK.TracingModel.Event) => boolean)): void {
     startTime = startTime || 0;
     endTime = endTime || Infinity;

package/front_end/models/timeline_model/TracingLayerTree.ts

@@ -178,7 +178,6 @@ export class TracingLayer implements SDK.LayerTreeBase.Layer {
         payload.compositing_reason_ids || (payload.debug_info && payload.debug_info.compositing_reason_ids) || [];
     this.drawsContentInternal = Boolean(payload.draws_content);
     this.gpuMemoryUsageInternal = payload.gpu_memory_usage;
-    /** @type {!Array<!LayerPaintEvent>} */
     this.paints = [];
   }
 

package/front_end/panels/accessibility/ARIAMetadata.ts

@@ -39,7 +39,6 @@ export class ARIAMetadata {
       this.attributes.set(attributeConfig.name, new Attribute(attributeConfig));
     }
 
-    /** @type {!Array<string>} */
     this.roleNames = config['roles'].map(roleConfig => roleConfig.name);
   }
 

package/front_end/panels/accessibility/AXBreadcrumbsPane.ts

@@ -499,11 +499,11 @@ export class AXBreadcrumb {
   }
 
   element(): HTMLElement {
-    return /** @type {!HTMLElement} */ this.elementInternal as HTMLElement;
+    return this.elementInternal;
   }
 
   nodeElement(): HTMLElement {
-    return /** @type {!HTMLElement} */ this.nodeElementInternal as HTMLElement;
+    return this.nodeElementInternal;
   }
 
   appendChild(breadcrumb: AXBreadcrumb): void {

package/front_end/panels/accessibility/AccessibilityNodeView.ts

@@ -203,8 +203,7 @@ export class AXNodeSubPane extends AccessibilitySubPane {
       };
       addProperty(roleProperty);
     }
-    for (const property of /** @type {!Array.<!Protocol.Accessibility.AXProperty>} */ axNode.properties() as
-         Protocol.Accessibility.AXProperty[]) {
+    for (const property of axNode.properties() as Protocol.Accessibility.AXProperty[]) {
       addProperty(property);
     }
 

package/front_end/panels/animation/AnimationTimeline.ts

@@ -240,7 +240,6 @@ export class AnimationTimeline extends UI.Widget.VBox implements SDK.TargetManag
     UI.ARIAUtils.markAsListBox(playbackRateControl);
     UI.ARIAUtils.setAccessibleName(playbackRateControl, i18nString(UIStrings.playbackRates));
 
-    /** @type {!Array<!HTMLElement>} */
     this.#playbackRateButtons = [];
     for (const playbackRate of GlobalPlaybackRates) {
       const button = (playbackRateControl.createChild('button', 'animation-playback-rate-button') as HTMLElement);

package/front_end/panels/application/ApplicationPanelCacheSection.ts

@@ -6,9 +6,9 @@ import type * as Common from '../../core/common/common.js';
 import * as i18n from '../../core/i18n/i18n.js';
 import * as SDK from '../../core/sdk/sdk.js';
 import * as UI from '../../ui/legacy/legacy.js';
+import * as ApplicationComponents from './components/components.js';
 
 import {ApplicationPanelTreeElement, ExpandableApplicationPanelTreeElement} from './ApplicationPanelTreeElement.js';
-import {BackForwardCacheView} from './BackForwardCacheView.js';
 import type {ResourcesPanel} from './ResourcesPanel.js';
 import {ServiceWorkerCacheView} from './ServiceWorkerCacheViews.js';
 
@@ -125,7 +125,6 @@ export class SWCacheTreeElement extends ApplicationPanelTreeElement {
     super(resourcesPanel, cache.cacheName + ' - ' + cache.securityOrigin, false);
     this.model = model;
     this.cache = cache;
-    /** @type {?} */
     this.view = null;
     const icon = UI.Icon.Icon.create('mediumicon-table', 'resource-tree-item');
     this.setLeadingIcons([icon]);
@@ -175,7 +174,7 @@ export class SWCacheTreeElement extends ApplicationPanelTreeElement {
 }
 
 export class BackForwardCacheTreeElement extends ApplicationPanelTreeElement {
-  private view?: BackForwardCacheView;
+  private view?: ApplicationComponents.BackForwardCacheView.BackForwardCacheViewWrapper;
 
   constructor(resourcesPanel: ResourcesPanel) {
     super(resourcesPanel, i18nString(UIStrings.backForwardCache), false);
@@ -190,7 +189,7 @@ export class BackForwardCacheTreeElement extends ApplicationPanelTreeElement {
   onselect(selectedByUser?: boolean): boolean {
     super.onselect(selectedByUser);
     if (!this.view) {
-      this.view = new BackForwardCacheView();
+      this.view = new ApplicationComponents.BackForwardCacheView.BackForwardCacheViewWrapper();
     }
     this.showView(this.view);
     return false;

package/front_end/panels/application/ServiceWorkerUpdateCycleView.ts

@@ -105,7 +105,6 @@ export class ServiceWorkerUpdateCycleView {
         }
         state = state.previousState;
       }
-      /** @type {Array <ServiceWorkerUpdateRange>} */
       const ranges: Array<ServiceWorkerUpdateRange> = [];
       addNormalizedRanges(
           ranges, version.id, beginInstallTime, endInstallTime, beginActivateTime, endActivateTime, currentStatus);
@@ -158,7 +157,6 @@ export class ServiceWorkerUpdateCycleView {
     this.selectedRowIndex = -1;
     this.removeRows();
     this.createTimingTableHead();
-    /** @type {!Array<ServiceWorkerUpdateRange>} */
     const timeRangeArray = timeRanges;
     if (timeRangeArray.length === 0) {
       return;

package/front_end/panels/application/StorageView.ts

@@ -178,9 +178,7 @@ export class StorageView extends UI.ThrottledWidget.ThrottledWidget {
 
     this.reportView.element.classList.add('clear-storage-header');
     this.reportView.show(this.contentElement);
-    /** @type {?SDK.Target.Target} */
     this.target = null;
-    /** @type {?string} */
     this.securityOrigin = null;
 
     this.settings = new Map();
@@ -213,7 +211,6 @@ export class StorageView extends UI.ThrottledWidget.ThrottledWidget {
     quotaOverrideCheckboxRow.appendChild(this.quotaOverrideCheckbox);
     this.quotaOverrideCheckbox.checkboxElement.addEventListener('click', this.onClickCheckbox.bind(this), false);
     this.quotaOverrideControlRow = quota.appendRow();
-    /** @type {!HTMLInputElement} */
     this.quotaOverrideEditor =
         this.quotaOverrideControlRow.createChild('input', 'quota-override-notification-editor') as HTMLInputElement;
     this.quotaOverrideControlRow.appendChild(UI.UIUtils.createLabel(i18nString(UIStrings.mb)));
@@ -446,7 +443,7 @@ export class StorageView extends UI.ThrottledWidget.ThrottledWidget {
       return;
     }
 
-    const securityOrigin = /** @type {string} */ (this.securityOrigin);
+    const securityOrigin = this.securityOrigin;
     const response = await this.target.storageAgent().invoke_getUsageAndQuota({origin: securityOrigin});
     this.quotaRow.textContent = '';
     if (response.getError()) {
@@ -475,8 +472,7 @@ export class StorageView extends UI.ThrottledWidget.ThrottledWidget {
 
     if (this.quotaUsage === null || this.quotaUsage !== response.usage) {
       this.quotaUsage = response.usage;
-      /** @type {!Array<!PerfUI.PieChart.Slice>} */
-      const slices = [];
+      const slices: PerfUI.PieChart.Slice[] = [];
       for (const usageForType of response.usageBreakdown.sort((a, b) => b.usage - a.usage)) {
         const value = usageForType.usage;
         if (!value) {

package/front_end/panels/application/{BackForwardCacheStrings.ts → components/BackForwardCacheStrings.ts}

@@ -2,7 +2,7 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-import * as i18n from '../../core/i18n/i18n.js';
+import * as i18n from '../../../core/i18n/i18n.js';
 
 const UIStrings = {
   /**
@@ -422,7 +422,7 @@ const UIStrings = {
   contentMediaPlay: 'A media player was playing upon navigating away.',
 };
 
-const str_ = i18n.i18n.registerUIStrings('panels/application/BackForwardCacheStrings.ts', UIStrings);
+const str_ = i18n.i18n.registerUIStrings('panels/application/components/BackForwardCacheStrings.ts', UIStrings);
 const i18nLazyString = i18n.i18n.getLazilyComputedLocalizedString.bind(undefined, str_);
 
 export const NotRestoredReasonDescription = {