WebKit Bugzilla
New
Browse
Search+
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
REOPENED
263305
[meta] Implement close requests and close watchers
https://bugs.webkit.org/show_bug.cgi?id=263305
Summary
[meta] Implement close requests and close watchers
Domenic Denicola
Reported
2023-10-17 23:26:31 PDT
This includes both the new CloseWatcher API, and hooking it up to <dialog> and popover="" so that they have the same behavior. (Including in edge cases like many of them being opened simultaneously.) See: *
https://html.spec.whatwg.org/#close-requests-and-close-watchers
*
https://github.com/WICG/close-watcher
*
https://github.com/whatwg/html/pull/9462
*
https://github.com/web-platform-tests/wpt/tree/master/close-watcher
*
https://source.chromium.org/chromium/chromium/src/+/main:third_party/blink/renderer/core/html/closewatcher/
*
https://source.chromium.org/chromium/chromium/src/+/main:content/browser/closewatcher/
Attachments
Add attachment
proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1
2023-10-18 14:00:43 PDT
<
rdar://problem/117163294
>
Luke Warlow
Comment 2
2023-12-17 11:07:57 PST
Pull request:
https://github.com/WebKit/WebKit/pull/21939
Domenic Denicola
Comment 3
2024-04-21 23:29:07 PDT
Small spec change coming here to improve web compat:
https://github.com/whatwg/html/pull/10291
Karl Dubost
Comment 4
2024-11-20 23:18:25 PST
Safari is failing that Firefox and Chrome are passing.
https://wpt.fyi/results/close-watcher/user-activation/y-dialog-disconnected.html
Domenic Denicola
Comment 5
2024-11-20 23:21:34 PST
I mean, Safari is failing all the close watcher tests, right? Not just that one.
https://wpt.fyi/results/close-watcher?label=experimental&label=master&aligned
EWS
Comment 6
2025-01-29 15:53:01 PST
Committed
289529@main
(74e0b308c997): <
https://commits.webkit.org/289529@main
> Reviewed commits have been landed. Closing PR #21939 and removing active labels.
Luke Warlow
Comment 7
2025-01-29 16:22:35 PST
Reopening so this can be used to track the full implementation that PR should have used a sub issue rather than the meta one.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug