1434547 - Port bug 1434401 to TB: Remove the "root-element" binding from :root and instead create the LightweightThemeConsumer from JS when new windows are created

Status: RESOLVED FIXED

(Thunderbird :: General, enhancement)

Description

[Richard Marti (:Paenglab)]

Bug 1434401 removes the "root-element" binding.

Comment 1

[Richard Marti (:Paenglab)]

Attachment: root-element.patch

I'm going the same way with JS instead of moving the binding. Naturally this needs bug 1434401 applied first to work (I tried it also without and the LW-themes still worked).

Bug 1434401 is not yet landed.

Comment 2

[Richard Marti (:Paenglab)]

Attachment: root-element.patch

Updated after the Cu Ci patches.

Waiting to ask r? until the m-c bug goes further.

Comment 3

[Richard Marti (:Paenglab)]

Comment on attachment 8949113 [details] [diff] [review]
root-element.patch

Bug 1434401 is in autoland now. So it makes sense to review it. I haven't changed like the FX bug to onLoad because we theme not only the main window but also the AB and the composer window. And so it makes sense to still check for the lightweightthemes="true" to apply only on themeable windows.

Comment 4

[Jorg K (CEST = GMT+2)]

Comment on attachment 8949113 [details] [diff] [review]
root-element.patch

OK. I played with this a little bit, and the notification chrome-document-global-created fires a lot. The hasAttribute("lightweightthemes") takes care of things, I tested that, too. So all good to go.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/87cad73c874a
Port bug 1434401 to TB: Remove the "root-element" binding from :root and instead create the LightweightThemeConsumer from JS when new windows are created. r=jorgk