Skip to content

Commit 0c31a7d

Browse files
Lightning00BladeDevtools-frontend LUCI CQ
authored andcommitted
[cleanup] Remove any
Using unknown satisfies TypeScript as value are `any`. Bug: none Change-Id: I676b6c9208ef96d3618969d420889b6c8b39da79 Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/6177278 Auto-Submit: Nikolay Vitkov <[email protected]> Reviewed-by: Benedikt Meurer <[email protected]> Commit-Queue: Benedikt Meurer <[email protected]>
1 parent 289e948 commit 0c31a7d

File tree

2 files changed

+4
-12
lines changed

2 files changed

+4
-12
lines changed

front_end/models/formatter/FormatterWorkerPool.ts

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -89,9 +89,7 @@ export class FormatterWorkerPool {
8989
methodName: string, params: {
9090
[x: string]: string,
9191
},
92-
// TODO(crbug.com/1172300) Ignored during the jsdoc to ts migration
93-
// eslint-disable-next-line @typescript-eslint/no-explicit-any
94-
callback: (arg0: boolean, arg1: any) => void): void {
92+
callback: (arg0: boolean, arg1: unknown) => void): void {
9593
const task = new Task(methodName, params, onData, true);
9694
this.taskQueue.push(task);
9795
this.processNextTask();

front_end/ui/legacy/Infobar.ts

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -42,9 +42,7 @@ export class Infobar {
4242
private readonly infoMessage: HTMLElement;
4343
private infoText: HTMLElement;
4444
private readonly actionContainer: HTMLElement;
45-
// TODO(crbug.com/1172300) Ignored during the jsdoc to ts migration
46-
// eslint-disable-next-line @typescript-eslint/no-explicit-any
47-
private readonly disableSetting: Common.Settings.Setting<any>|null;
45+
private readonly disableSetting: Common.Settings.Setting<boolean>|null;
4846
private readonly closeContainer: HTMLElement;
4947
private readonly toggleElement: Buttons.Button.Button;
5048
private readonly closeButton: DevToolsCloseButton;
@@ -53,9 +51,7 @@ export class Infobar {
5351
private parentView?: Widget;
5452

5553
constructor(
56-
// TODO(crbug.com/1172300) Ignored during the jsdoc to ts migration
57-
// eslint-disable-next-line @typescript-eslint/no-explicit-any
58-
type: Type, text: string, actions?: InfobarAction[], disableSetting?: Common.Settings.Setting<any>,
54+
type: Type, text: string, actions?: InfobarAction[], disableSetting?: Common.Settings.Setting<boolean>,
5955
/* TODO(crbug.com/1354548) Remove with JS Profiler deprecation */ isCloseable: boolean = true,
6056
jslogContext?: string) {
6157
this.element = document.createElement('div');
@@ -155,9 +151,7 @@ export class Infobar {
155151
}
156152

157153
static create(
158-
// TODO(crbug.com/1172300) Ignored during the jsdoc to ts migration
159-
// eslint-disable-next-line @typescript-eslint/no-explicit-any
160-
type: Type, text: string, actions?: InfobarAction[], disableSetting?: Common.Settings.Setting<any>,
154+
type: Type, text: string, actions?: InfobarAction[], disableSetting?: Common.Settings.Setting<boolean>,
161155
jslogContext?: string): Infobar|null {
162156
if (disableSetting && disableSetting.get()) {
163157
return null;

0 commit comments

Comments
 (0)