WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
249602
Update EventHandler::nodeWillBeRemoved to cover ShadowDOM as well
https://bugs.webkit.org/show_bug.cgi?id=249602
Summary
Update EventHandler::nodeWillBeRemoved to cover ShadowDOM as well
Ahmad Saleem
Reported
2022-12-19 12:56:10 PST
Hi Team, Based on discussion with Ryosuke on Slack, it seems that the function should have "containsIncludingShadowDOM" rather than just "contain".
https://github.com/WebKit/WebKit/blob/af67e80850c1e752b05548a41f4d507272c8e7bf/Source/WebCore/page/EventHandler.cpp#L431
This bug is just to fix this and have ShadowDOM coverage. Thanks!
Attachments
Add attachment
proposed patch, testcase, etc.
EWS
Comment 1
2022-12-19 22:06:48 PST
Committed
258122@main
(3499e2bc4f43): <
https://commits.webkit.org/258122@main
> Reviewed commits have been landed. Closing PR #7868 and removing active labels.
Radar WebKit Bug Importer
Comment 2
2022-12-19 22:07:18 PST
<
rdar://problem/103544698
>
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