From 776f0192abe51dbe53acedecf0b8304409502a5a Mon Sep 17 00:00:00 2001 From: Lars den Bakker Date: Sat, 1 Aug 2020 00:45:29 +0200 Subject: [PATCH] fix(chromium): remove Debugger.paused event listener on coverage stop (#3252) --- src/chromium/crCoverage.ts | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/chromium/crCoverage.ts b/src/chromium/crCoverage.ts index 53e8d74b6d..41c7ed980b 100644 --- a/src/chromium/crCoverage.ts +++ b/src/chromium/crCoverage.ts @@ -79,8 +79,8 @@ class JSCoverage { this._eventListeners = [ helper.addEventListener(this._client, 'Debugger.scriptParsed', this._onScriptParsed.bind(this)), helper.addEventListener(this._client, 'Runtime.executionContextsCleared', this._onExecutionContextsCleared.bind(this)), + helper.addEventListener(this._client, 'Debugger.paused', this._onDebuggerPaused.bind(this)), ]; - this._client.on('Debugger.paused', () => this._client.send('Debugger.resume')); await Promise.all([ this._client.send('Profiler.enable'), this._client.send('Profiler.startPreciseCoverage', { callCount: true, detailed: true }), @@ -89,6 +89,10 @@ class JSCoverage { ]); } + _onDebuggerPaused() { + this._client.send('Debugger.resume'); + } + _onExecutionContextsCleared() { if (!this._resetOnNavigation) return;