Skip to content

Commit 17794ee

Browse files
Lightning00BladeDevtools-frontend LUCI CQ
authored andcommitted
Update TypeScript to v5.7.3
https://devblogs.microsoft.com/typescript/announcing-typescript-5-7/ Some changes to the ArrayBuffer types was needed. And TypeScript started "panicking" if we included a bundle multiple times, so updated the BUILD.gn files. Bug: none Change-Id: I3277c191c83ea773ef88765ff89a4f4103b5757c Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/6210862 Reviewed-by: Benedikt Meurer <[email protected]> Commit-Queue: Nikolay Vitkov <[email protected]>
1 parent b7a4a52 commit 17794ee

File tree

270 files changed

+145933
-141845
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

270 files changed

+145933
-141845
lines changed

front_end/core/common/Base64.test.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -30,12 +30,12 @@ describe('Base64 decoder', () => {
3030
describe('Base64 encoder', () => {
3131
it('encodes ArrayBuffers correctly', async () => {
3232
const buffer1 = new Uint8Array([0]);
33-
assert.strictEqual(await Common.Base64.encode(buffer1), 'AA==');
33+
assert.strictEqual(await Common.Base64.encode(buffer1.buffer), 'AA==');
3434

3535
const buffer2 = new Uint8Array([0, 1]);
36-
assert.strictEqual(await Common.Base64.encode(buffer2), 'AAE=');
36+
assert.strictEqual(await Common.Base64.encode(buffer2.buffer), 'AAE=');
3737

3838
const buffer3 = new Uint8Array([0, 1, 2]);
39-
assert.strictEqual(await Common.Base64.encode(buffer3), 'AAEC');
39+
assert.strictEqual(await Common.Base64.encode(buffer3.buffer), 'AAEC');
4040
});
4141
});

front_end/entrypoints/wasmparser_worker/WasmParserWorker.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ export function dissambleWASM(
5454
const offsets = [];
5555
const functionBodyOffsets = [];
5656
let chunkSize: number = 128 * 1024;
57-
let buffer: Uint8Array = new Uint8Array(chunkSize);
57+
let buffer = new Uint8Array(chunkSize);
5858
let pendingSize = 0;
5959
let offsetInModule = 0;
6060
for (let i = 0; i < data.length;) {

front_end/models/bindings/FileUtils.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ export class ChunkedFileReader implements ChunkedReader {
5555
#file: File|null;
5656
readonly #fileSizeInternal: number;
5757
#loadedSizeInternal: number;
58-
#streamReader: ReadableStreamDefaultReader<Uint8Array>|null;
58+
#streamReader: ReadableStreamDefaultReader<Uint8Array<ArrayBuffer>>|null;
5959
readonly #chunkSize: number;
6060
readonly #chunkTransferredCallback: ((arg0: ChunkedReader) => void)|undefined;
6161
readonly #decoder: TextDecoder;

front_end/models/persistence/PersistenceAction.test.ts

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -70,8 +70,10 @@ describeWithLocale('ContextMenuProvider', () => {
7070
uiSourceCode.project.returns(stubProject);
7171
const stubWorkspaceBinding = sinon.createStubInstance(Bindings.DebuggerWorkspaceBinding.DebuggerWorkspaceBinding);
7272
sinon.stub(Bindings.DebuggerWorkspaceBinding.DebuggerWorkspaceBinding, 'instance').returns(stubWorkspaceBinding);
73-
const stubWasmScript = sinon.createStubInstance(
74-
SDK.Script.Script, {getWasmBytecode: Promise.resolve(new Uint8Array([1, 2, 3, 4])), isWasm: true});
73+
const stubWasmScript = sinon.createStubInstance(SDK.Script.Script, {
74+
getWasmBytecode: Promise.resolve(new Uint8Array([1, 2, 3, 4]).buffer),
75+
isWasm: true,
76+
});
7577
stubWorkspaceBinding.scriptsForUISourceCode.returns([stubWasmScript]);
7678

7779
menuProvider.appendApplicableItems(event, contextMenu, uiSourceCode);

front_end/panels/elements/components/BUILD.gn

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,6 @@ devtools_module("components") {
5656
"../../../ui/components/buttons:bundle",
5757
"../../../ui/components/helpers:bundle",
5858
"../../../ui/components/icon_button:bundle",
59-
"../../../ui/components/icon_button:bundle",
6059
"../../../ui/components/legacy_wrapper:bundle",
6160
"../../../ui/components/node_text:bundle",
6261
"../../../ui/components/render_coordinator:bundle",

front_end/panels/linear_memory_inspector/LinearMemoryInspectorController.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ class MockRemoteObject extends SDK.RemoteObject.LocalJSONObject {
2727
}
2828
}
2929

30-
function createWrapper(array: Uint8Array) {
30+
function createWrapper(array: Uint8Array<ArrayBuffer>) {
3131
const mockRemoteObj = new MockRemoteObject(array.buffer);
3232
const mockRemoteArrayBuffer = new SDK.RemoteObject.RemoteArrayBuffer(mockRemoteObj);
3333
return new LinearMemoryInspectorController.RemoteArrayBufferWrapper(mockRemoteArrayBuffer);

front_end/panels/linear_memory_inspector/LinearMemoryInspectorController.ts

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ const MEMORY_TRANSFER_MIN_CHUNK_SIZE = 1000;
3131
let controllerInstance: LinearMemoryInspectorController;
3232

3333
export interface LazyUint8Array {
34-
getRange(start: number, end: number): Promise<Uint8Array>;
34+
getRange(start: number, end: number): Promise<Uint8Array<ArrayBuffer>>;
3535
length(): number;
3636
}
3737

@@ -46,7 +46,7 @@ export class RemoteArrayBufferWrapper implements LazyUint8Array {
4646
return this.#remoteArrayBuffer.byteLength();
4747
}
4848

49-
async getRange(start: number, end: number): Promise<Uint8Array> {
49+
async getRange(start: number, end: number): Promise<Uint8Array<ArrayBuffer>> {
5050
const newEnd = Math.min(end, this.length());
5151
if (start < 0 || start > newEnd) {
5252
console.error(`Requesting invalid range of memory: (${start}, ${end})`);
@@ -129,7 +129,7 @@ export class LinearMemoryInspectorController extends SDK.TargetManager.SDKModelO
129129
}
130130

131131
static async getMemoryForAddress(memoryWrapper: LazyUint8Array, address: number):
132-
Promise<{memory: Uint8Array, offset: number}> {
132+
Promise<{memory: Uint8Array<ArrayBuffer>, offset: number}> {
133133
// Provide a chunk of memory that covers the address to show and some before and after
134134
// as 1. the address shown is not necessarily at the beginning of a page and
135135
// 2. to allow for fewer memory requests.
@@ -139,7 +139,8 @@ export class LinearMemoryInspectorController extends SDK.TargetManager.SDKModelO
139139
return {memory, offset: memoryChunkStart};
140140
}
141141

142-
static async getMemoryRange(memoryWrapper: LazyUint8Array, start: number, end: number): Promise<Uint8Array> {
142+
static async getMemoryRange(memoryWrapper: LazyUint8Array, start: number, end: number):
143+
Promise<Uint8Array<ArrayBuffer>> {
143144
// Check that the requested start is within bounds.
144145
// If the requested end is larger than the actual
145146
// memory, it will be automatically capped when

front_end/panels/linear_memory_inspector/LinearMemoryInspectorPane.test.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -16,13 +16,13 @@ function createArray() {
1616

1717
describeWithEnvironment('LinearMemoryInspectorPane', () => {
1818
class Uint8Wrapper {
19-
private array: Uint8Array;
19+
private array: Uint8Array<ArrayBuffer>;
2020

21-
constructor(array: Uint8Array) {
21+
constructor(array: Uint8Array<ArrayBuffer>) {
2222
this.array = array;
2323
}
2424

25-
getRange(start: number, end: number): Promise<Uint8Array> {
25+
getRange(start: number, end: number): Promise<Uint8Array<ArrayBuffer>> {
2626
return Promise.resolve(this.array.slice(start, end));
2727
}
2828
length(): number {

front_end/panels/linear_memory_inspector/components/LinearMemoryInspector.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ const i18nString = i18n.i18n.getLocalizedString.bind(undefined, str_);
5252
// on the 1. memoryOffset (at which index this portion starts),
5353
// and on the 2. outerMemoryLength (length of the original Uint8Array).
5454
export interface LinearMemoryInspectorData {
55-
memory: Uint8Array;
55+
memory: Uint8Array<ArrayBuffer>;
5656
address: number;
5757
memoryOffset: number;
5858
outerMemoryLength: number;

front_end/panels/linear_memory_inspector/components/LinearMemoryViewer.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ import type {HighlightInfo} from './LinearMemoryViewerUtils.js';
1212
const {render, html} = Lit;
1313

1414
export interface LinearMemoryViewerData {
15-
memory: Uint8Array;
15+
memory: Uint8Array<ArrayBuffer>;
1616
address: number;
1717
memoryOffset: number;
1818
focus: boolean;

0 commit comments

Comments
 (0)