1363864 - Add MozBeforeInitialXULLayout event

Status: RESOLVED FIXED

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

Description

[Zibi Braniecki [:zbraniecki][:gandalf]]

In bug 1347798 we're adding a small per-document JS code that enables DOM localization using the new API.

At the moment, we only translate the document on window.onready, but it may be better to do this before layout.

Comment 1

[Zibi Braniecki [:zbraniecki][:gandalf]]

Attachment: mozbeforelayout.diff

Comment 2

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

Could we call the events something like Before/AfterInitialLayout

Comment 3

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

Maybe even Before/AfterInitialXULLayout

Comment 4

[Zibi Braniecki [:zbraniecki][:gandalf]]

Sure! I'm not even yet at the stage where I'd know if I need it. Just wanted to dump my patches into bugzilla.

If I'll get it on track to land I'll rename. Thanks :)

Comment 5

[Zibi Braniecki [:zbraniecki][:gandalf]]

Attachment: Bug 1363864 - Add MozBeforeInitialXULLayout event.

Review commit: https://reviewboard.mozilla.org/r/207930/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/207930/

Comment 6

[Zibi Braniecki [:zbraniecki][:gandalf]]

While profiling Preferences with Fluent we noticed that the localization is applied way too late[0], which brought me back to this event.

With this event we can trigger DOM localization right before layout  is started which prevents FOUCs.

[0] https://searchfox.org/mozilla-central/rev/c8e791091973825680bbba807fc1c4f5bda0f5a1/intl/l10n/l10n.js#17

Comment 7

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

Comment on attachment 8937210 [details]
Bug 1363864 - Add MozBeforeInitialXULLayout event.

https://reviewboard.mozilla.org/r/207930/#review213902

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/16471f5143c7
Add MozBeforeInitialXULLayout event. r=smaug

Comment 9

[Dorel Luca [:dluca]]

https://hg.mozilla.org/mozilla-central/rev/16471f5143c7