1896593 - Make our refresh driver tick match the HTML spec more closely.

Status: RESOLVED FIXED

(Core :: Layout, task)

Comment 1

[Emilio Cobos Álvarez [:emilio]]

Attachment: Bug 1896593 - Make our refresh driver tick match the HTML spec more closely. r=smaug,#layout

This doesn't change behavior intentionally, though it annotates places
where our code doesn't match the spec right now, and adds comments and
quotes the HTML spec in the relevant places.

Some things that this highlights:

• Resize steps are fired at the wrong time.
• Some of our animation stuff seems sketchy / out of place too to me
  (AnimationTimeline observes the refresh driver at the style phase).
• Focus fix-up is out of order (that's known, bug 1788741 for tests).
• Our layout flush stuff should probably be better integrated / closer
  to the resize observer loop.
• Display observers are run before stuff like ResizeObserver runs. That
  might be a problem for accessibility's NotificationController?

Again, these are intentionally not addressed in this patch.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/e88bed5f3358
Make our refresh driver tick match the HTML spec more closely. r=smaug

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/7b31a5c8c0f9
Fix a devtools frequent intermittent.

Comment 4

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/e88bed5f3358
https://hg.mozilla.org/mozilla-central/rev/7b31a5c8c0f9