Summary: | Hit-testing does not account for clip-path on <iframe> | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Antoine Quint <graouts> | ||||||||
Component: | Layout and Rendering | Assignee: | Antoine Quint <graouts> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | bfulgham, changseok, esprehn+autocc, ews-watchlist, glenn, koivisto, kondapallykalyan, pdr, simon.fraser, webkit-bug-importer, zalan | ||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||
Version: | WebKit Nightly Build | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
Bug Depends on: | |||||||||||
Bug Blocks: | 226499 | ||||||||||
Attachments: |
|
Description
Antoine Quint
2021-05-28 09:43:09 PDT
Created attachment 430021 [details]
Patch
Created attachment 430097 [details]
Patch
Created attachment 430241 [details]
Patch
Comment on attachment 430241 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=430241&action=review > LayoutTests/css3/masking/clip-path-iframe.html:9 > + if (window.eventSender) > + eventSender.mouseMoveTo(50, 50); This won't work on iOS. It would be better if the test was constructed so that all platforms are covered (and nothing happens case wouldn't pass). You can exercise hit testing code directly in platform independent manner by using Document.elementFromPoint. Comment on attachment 430241 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=430241&action=review > LayoutTests/css3/masking/clip-path-iframe.html:15 > +iframe { I think this also affects replaced elements like <img>. It would be good to have a test for them too. Committed r278343 (238375@main): <https://commits.webkit.org/238375@main> |