Bug 249602

Summary:	Update EventHandler::nodeWillBeRemoved to cover ShadowDOM as well
Product:	WebKit	Reporter:	Ahmad Saleem <ahmad.saleem792>
Component:	HTML Editing	Assignee:	Nobody <webkit-unassigned>
Status:	RESOLVED FIXED
Severity:	Normal	CC:	rniwa, webkit-bug-importer, wenson_hsieh
Priority:	P2	Keywords:	InRadar
Version:	Safari Technology Preview
Hardware:	Unspecified
OS:	Unspecified

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

Note You need to log in before you can comment on or make changes to this bug.