From 721cc64e7c50d64caf4eb7f9d71bf1f01beb8efd Mon Sep 17 00:00:00 2001 From: Nataniel Borges Date: Mon, 16 Jan 2023 15:01:49 +0000 Subject: [PATCH] Fix layer trace entry builder API Previously the viewer had all methods in the builder because of name mangling. With jsName annotation this issue was solved and we can now use the correct API Test: open winscope and take a SF dump Change-Id: I4c1ad3dada5ce994540adb45ad04041e02a4ad39 --- .../trace/flickerlib/layers/LayerTraceEntry.ts | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/tools/winscope/src/trace/flickerlib/layers/LayerTraceEntry.ts b/tools/winscope/src/trace/flickerlib/layers/LayerTraceEntry.ts index 51577aa3d..362a7dc78 100644 --- a/tools/winscope/src/trace/flickerlib/layers/LayerTraceEntry.ts +++ b/tools/winscope/src/trace/flickerlib/layers/LayerTraceEntry.ts @@ -40,14 +40,14 @@ LayerTraceEntry.fromProto = ( ): LayerTraceEntry => { const layers = protos.map((it) => Layer.fromProto(it, excludesCompositionState)); const displays = (displayProtos || []).map((it) => newDisplay(it)); - const builder = new LayerTraceEntryBuilder( - `${elapsedTimestamp}`, - layers, - displays, - vSyncId, - hwcBlob, - where, - `${realToElapsedTimeOffsetNs ?? 0}` + const builder = new LayerTraceEntryBuilder() + .setElapsedTimestamp(`${elapsedTimestamp}`) + .setLayers(layers) + .setDisplays(displays) + .setVSyncId(`${vSyncId}`) + .setHwcBlob(hwcBlob) + .setWhere(where) + .setRealToElapsedTimeOffsetNs(`${realToElapsedTimeOffsetNs ?? 0}` ); const entry: LayerTraceEntry = builder.build();