1662277 - IntersectionObserverEntry objects should have a consistent time with everything else in the same refresh driver tick / update the rendering steps

Status: RESOLVED FIXED

(Core :: Layout, defect)

Description

[Sean Feng [:sefeng211]]

https://searchfox.org/mozilla-central/rev/0c682c4f01442c3de0fa6cd286e9cadc8276b45f/dom/base/Document.cpp#15095 should use the now timestamp which is at the beginning of the refresh driver.

Comment 1

[Emilio Cobos Álvarez [:emilio]]

Attachment: A test-case.

So this does indeed show the same timestamps in rAF callbacks and IntersectionObserver in WebKit trunk, but not in Blink or current Epiphany at least.

Comment 2

[Emilio Cobos Álvarez [:emilio]]

Attachment: Bug 1662277 - Make IntersectionObserver entries have the same timestamp as other timestamp-returning things during the "update the rendering steps". r=smaug

This is per spec, see the link in the test. I wouldn't see much point on
doing this for IntersectionObserver on its own, see:

https://bugzilla.mozilla.org/show_bug.cgi?id=1518999#c28

Though I see the point of doing this for the paint timing API, and I
guess for consistency we should do it here too.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/d23c9b17da6b
Make IntersectionObserver entries have the same timestamp as other timestamp-returning things during the "update the rendering steps". r=sefeng

Comment 4

[Atila Butkovits]

https://hg.mozilla.org/mozilla-central/rev/d23c9b17da6b