Created attachment 322862 [details] test case 1. Open the attached test case. 2. Click on the green rectangle. Result: no alert messages. Expected: an alert message comes up when clicking the green rectangle. NOTES: The green rectangle is an SVGRectElement inside an SVGSVGElement. The SVGRectElement positioned outside the rectangle of the root element. The SVGRectElement is drawn only because the overflow of the SVGSVGElement is set to be visible. Since the SVGRectElement is drawn, it should be accessible through elementFromPoint() and the hit testing.
Created attachment 322866 [details] Patch
<rdar://problem/34788389>
Created attachment 322871 [details] Patch
Comment on attachment 322871 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=322871&action=review > Source/WebCore/ChangeLog:5 > + https://bugs.webkit.org/show_bug.cgi?id=177953 > + Should be a link to the radar here. > Source/WebCore/ChangeLog:13 > + If an SVGElement is positioned outside the rectangle of the root element, > + it can still be drawn if the root element has "overflow: visible" applied. > + But since SVGElement can be drawn in this case, it should be accessible > + through nodeAtPoint() which is used for hit testing. Does this do the right thing if another HTML element overlaps the visual overflow and has higher z-index?
Created attachment 322918 [details] Patch
Comment on attachment 322871 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=322871&action=review >> Source/WebCore/ChangeLog:13 >> + through nodeAtPoint() which is used for hit testing. > > Does this do the right thing if another HTML element overlaps the visual overflow and has higher z-index? Yes it does. I includes this case in the new layout test which is attached to this patch.
Comment on attachment 322918 [details] Patch Clearing flags on attachment: 322918 Committed r222934: <http://trac.webkit.org/changeset/222934>
All reviewed patches have been landed. Closing bug.