From d32d466f2f78b9fef91358767a3f7af826072537 Mon Sep 17 00:00:00 2001 From: Andrey Lushnikov Date: Fri, 17 Feb 2023 12:19:46 -0800 Subject: [PATCH] cherry-pick(#20998): fix: stricter condition for Firefox codegen tests workaround MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit This patch fixes the following tests on WebKit @ Darwin: - library/inspector/cli-codegen-2.spec.ts:197:7 › cli codegen › should download files - library/inspector/cli-codegen-2.spec.ts:428:7 › cli codegen › should update hover model on action --- packages/playwright-core/src/server/injected/recorder.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/playwright-core/src/server/injected/recorder.ts b/packages/playwright-core/src/server/injected/recorder.ts index 428deaf7ea..b6382b3839 100644 --- a/packages/playwright-core/src/server/injected/recorder.ts +++ b/packages/playwright-core/src/server/injected/recorder.ts @@ -254,7 +254,7 @@ class Recorder { const activeElement = this._deepActiveElement(document); // Firefox dispatches "focus" event to body when clicking on a backgrounded headed browser window. // We'd like to ignore this stray event. - if (activeElement === document.body) + if (userGesture && activeElement === document.body) return; const result = activeElement ? generateSelector(this._injectedScript, activeElement, this._testIdAttributeName) : null; this._activeModel = result && result.selector ? result : null;