657155 - foreignObject inline in HTML isn't reliably reflowed when document.elementFromPoint is called

Status: RESOLVED FIXED

(Core :: SVG, defect)

Description

[Jonathan Watt [:jwatt]]

foreignObject inline in HTML isn't reliably reflowed before the document 'load' event is fired, so document.elementFromPoint() fails. See bug 580983 comment 116. Reflow tardiness isn't confirmed to be the problem, but it's almost certainly the case. Someone should enable the disabled test in test_pointer-events-2.xhtml and check.

Boris, what's your opinion on whether people should be able to rely on foreignObject being reflowed before the document 'load' event is fired? Seems to me like elementFromPoint() should work onload, so it should.

Comment 1

[Robert O'Callahan (:roc) (email my personal email if necessary)]

The issue here is not that reflow must happen before "load" runs. There is no such requirement.

The issue here is that document.elementFromPoint flushes reflow, but apparently that reflow flush is not actually causing the foreignObject to be reflowed.

Comment 2

[Boris Zbarsky [:bzbarsky]]

Fwiw, that is _exactly_ what I was going to say.

This sounds like one of the existing "foreignobject doesn't update layout dynamically correctly" bugs...

Comment 3

[Robert Longson [:longsonr]]

Changed summary to reflect comments.

Comment 4

[Boris Zbarsky [:bzbarsky]]

A testcase would be useful too.

Comment 5

[Robert Longson [:longsonr]]

Attachment: Bug 657155 - Correct position of clipPath so we can re-enable testing of foreignObject clipping r=emilio

Comment 6

[Robert Longson [:longsonr]]

Not sure if this was ever a problem or just that the clipPath position was always incorrect in the test.

https://treeherder.mozilla.org/jobs?repo=try&revision=2594c7470d091ce04962bbfab8a1aed791f78e4e

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/9ca2bcf6799b
Correct position of clipPath so we can re-enable testing of foreignObject clipping r=emilio

Comment 8

[Iulian Moraru]

https://hg.mozilla.org/mozilla-central/rev/9ca2bcf6799b