From dbd124d84b734f30ad2f57256f492c72b9b58819 Mon Sep 17 00:00:00 2001 From: Pavel Feldman Date: Mon, 7 Feb 2022 19:21:58 -0800 Subject: [PATCH] chore: disable page.pause() when JS debugger is attached (#11926) --- packages/playwright-core/src/client/page.ts | 5 ++++- .../src/dispatchers/browserContextDispatcher.ts | 2 +- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/packages/playwright-core/src/client/page.ts b/packages/playwright-core/src/client/page.ts index 0558dbb8a7..cda775ada8 100644 --- a/packages/playwright-core/src/client/page.ts +++ b/packages/playwright-core/src/client/page.ts @@ -644,7 +644,10 @@ export class Page extends ChannelOwner implements api.Page } async pause() { - await this.context()._channel.pause(); + if (!!require('inspector').url()) + debugger; // eslint-disable-line no-debugger + else + await this.context()._channel.pause(); } async pdf(options: PDFOptions = {}): Promise { diff --git a/packages/playwright-core/src/dispatchers/browserContextDispatcher.ts b/packages/playwright-core/src/dispatchers/browserContextDispatcher.ts index f3d9baf9bf..738a228ce5 100644 --- a/packages/playwright-core/src/dispatchers/browserContextDispatcher.ts +++ b/packages/playwright-core/src/dispatchers/browserContextDispatcher.ts @@ -179,7 +179,7 @@ export class BrowserContextDispatcher extends Dispatcher {