Bug 249547 - Do not lay out in Document::setFocusedElement
Summary: Do not lay out in Document::setFocusedElement
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: UI Events (show other bugs)
Version: Safari Technology Preview
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: BrowserCompat, InRadar
Depends on:
Blocks:
 
Reported: 2022-12-17 16:12 PST by Ahmad Saleem
Modified: 2023-05-24 16:51 PDT (History)
5 users (show)

See Also:


Attachments
Safari 16.2, STP160, Chrome Canary 111 and Firefox 110 result (255.02 KB, image/png)
2022-12-18 05:09 PST, Ahmad Saleem
no flags Details
Testcase (1.26 KB, text/html)
2023-05-24 16:51 PDT, Ahmad Saleem
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Ahmad Saleem 2022-12-17 16:12:56 PST
Hi Team,

Going through Blink's commit, came across another failing test in Safari Technology Preview 160:

Test Case - https://jsfiddle.net/zLbes8pt/

^ passes in Chrome Canary 110 and Firefox Nightly 110.

Blink Commit - https://chromium.googlesource.com/chromium/src.git/+/15acc9dd530608bc5bef752dd6aacee07c7c8377

I haven't looked into whether it is 1-1 merge but it is still a bug, which we should fix.

Thanks!
Comment 1 Tim Nguyen (:ntim) 2022-12-17 22:48:41 PST
The testcase seems to fail in Chrome Canary for me.
Comment 2 Ahmad Saleem 2022-12-18 05:09:25 PST
Created attachment 464095 [details]
Safari 16.2, STP160, Chrome Canary 111 and Firefox 110 result

I am getting following on JSFiddle test across all browsers.
Comment 3 Radar WebKit Bug Importer 2022-12-24 16:13:14 PST
<rdar://problem/103689534>
Comment 4 Ahmad Saleem 2023-05-24 16:51:36 PDT
Created attachment 466488 [details]
Testcase

Removing this testcase from JSFiddle to confirm, it is genuine bug and not something hidden due to iframe focus issue with JSFiddle.