1568747 - Massive Mozmill failure on 2019-07-25 - Write window broken - Port bug 1550801 - Load XUL documents as XHTML documents

Status: RESOLVED FIXED

(Thunderbird :: General, defect)

Description

[Jorg K (CEST = GMT+2)]

https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=1da5a37de0fd1695e96a9d5e7f59fbd84d8759b2&tochange=922be4adb708aee5ab59602b38fbb19f37c2de53

I haven't looked, but it could be
fd2b93ff0abecc11bcfbcd806262c79f5de50b9b Brendan Dahl — Bug 1550801 - Load XUL documents as XHTML documents. r=smaug

That would be something for our sadly not so pro-active de-XBL team. Geoff, have you looked?

Comment 1

[Geoff Lankow (:darktrojan)]

I've looked, and you're right.

https://treeherder.mozilla.org/#/jobs?repo=try-comm-central&revision=94e732a2778e94d301171a13a58e84c8a56391b5

Comment 2

[Jorg K (CEST = GMT+2)]

That try is looking good, sadly you don't have a patch but rather a backout :-(
+GECKO_HEAD_REPOSITORY: https://hg.mozilla.org/try
+GECKO_HEAD_REF: ab6d3e5e6d1ad4b4744cd1234983ced7172359ec

https://hg.mozilla.org/try/graph/ab6d3e5e6d1ad4b4744cd1234983ced7172359ec

which has
Backed out changeset fd2b93ff0abe (bug 1550801)

Comment 3

[Jorg K (CEST = GMT+2)]

Looks like our composition window is broken, so all related tests fail, of course. There are also test failures in mail account setup and heaps of Calendar failures.

Looks like I'll cancel Dailies for a while.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/71dbc5c422f8
No bug - Disable Nightly builds . a=jorgk CLOSED TREE

Comment 5

[Pulsebot]

Backout by [e-mail]:
https://hg.mozilla.org/comm-central/rev/1177eb495b94
Backed out changeset 71dbc5c422f8 since it not only breaks shippable/daily/nightly builds. a=backout CLOSED TREE

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/33aa95186ee7
Fix overlay loader for HTML documents and stop waiting for menulist CE; rs=bustage-fix CLOSED TREE

Comment 7

[Geoff Lankow (:darktrojan)]

That should fix most stuff, if not all. My Try run is greener than I expected.

Comment 8

[Magnus Melin [:mkmelin] (away until Aug 10)]

Thx, Geoff!

Comment 9

[Jorg K (CEST = GMT+2)]

Can someone review the landed patch? Is this the final solution? Not being able to wait for "menulist" seems to need a follow-up:

10:18:28 - darktrojan: oh cool, so one of the things you can't do as an xhtml document is wait for custom elements without a hyphen in the name
10:18:40 - darktrojan: like menulist
10:18:43 - darktrojan: :/

Comment 10

[Magnus Melin [:mkmelin] (away until Aug 10)]

Custom elements are required to have a dash in their name. That it hasn't been enforced was by design, but nevertheless a non standard feature.

Comment 11

[Jorg K (CEST = GMT+2)]

So why are there custom elements without a hyphen which are bound to break?

Please note the difference between dash and hyphen:
https://www.ef.com/wwen/english-resources/english-grammar/hyphens-and-dashes/

Comment 12

[Magnus Melin [:mkmelin] (away until Aug 10)]

AFAIK they were/are only allowed (internally?) to cause minimum change requirements during the de-xbl conversion. I'd expect that at some point this allowance will go away, and we've therefore proactively renamed some elements (like minimonth) to something with a hypen instead.

Comment 13

[Geoff Lankow (:darktrojan)]

Attachment: 1568747-xul-html-bustage.diff

Comment 14

[Magnus Melin [:mkmelin] (away until Aug 10)]

Comment on attachment 9081515 [details] [diff] [review]
1568747-xul-html-bustage.diff

Review of attachment 9081515 [details] [diff] [review]:
-----------------------------------------------------------------

LGTM, r=mkmelin