From f4bb2ff0dc135a4088dd10978768d7f758e8b2d1 Mon Sep 17 00:00:00 2001 From: Benedikt Meurer Date: Thu, 4 Jun 2020 15:09:37 +0200 Subject: [PATCH] [wasm] Properly terminate the wasmparser worker. Before we had been leaking all the wasmparser workers. Bug: chromium:1071432 Change-Id: Ice230c7aa4355facd85202aec02ed01d7fe9c447 Fixed: chromium:1091230 Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/2230505 Commit-Queue: Benedikt Meurer Commit-Queue: Paul Lewis Auto-Submit: Benedikt Meurer Reviewed-by: Paul Lewis --- front_end/source_frame/SourceFrame.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/front_end/source_frame/SourceFrame.js b/front_end/source_frame/SourceFrame.js index c844d47cbf..ffc5a5fbee 100644 --- a/front_end/source_frame/SourceFrame.js +++ b/front_end/source_frame/SourceFrame.js @@ -330,8 +330,8 @@ export class SourceFrameImpl extends UI.View.SimpleView { worker.onerror = reject; }); worker.postMessage({method: 'disassemble', params: {content}}); - const {source, offsets, functionBodyOffsets} = await promise; + worker.terminate(); this._rawContent = source; this._wasmDisassembly = new Common.WasmDisassembly.WasmDisassembly(offsets, functionBodyOffsets); }