1489376 - Fullscreen events should be dispatched in tree order

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, enhancement, P3)

Description

[Xidorn Quan [:xidorn] UTC+10]

Currently, when entering fullscreen, we dispatch the event in tree order, but when exiting fullscreen, we dispatch in the reverse tree order (because it makes more sense that innermost one is the earliest to exit fullscreen).

However, since then the spec has been revised to take multi-process document into account. Since fullscreen events are always synced with flush, their relative order becomes less a problem, and it may be easier to dispatch them in a given order for both directions of fullscreen change.

(I don't think this needs to block shipping unprefixed fullscreen, though.)

Comment 1

[Xidorn Quan [:xidorn] UTC+10]

It seems this would actually make it easier to implement bug 1375319.

Comment 2

[Xidorn Quan [:xidorn] UTC+10]

Attachment: Bug 1489376 - Dispatch fullscreen events in tree order. r=smaug

Comment 3

[Olli Pettay [:smaug][[e-mail]]]

Comment on attachment 9007654 [details]
Bug 1489376 - Dispatch fullscreen events in tree order. r=smaug

Olli Pettay [:smaug] has approved the revision.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/6b1be7db8f0f
Dispatch fullscreen events in tree order. r=smaug

Comment 5

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/6b1be7db8f0f