1522235 - Disallow XUL element in HTML for real or fix it in the debug build

Status: NEW

(Core :: XUL, enhancement, P3)

Description

[Tim Guan-tin Chien [:timdream] (please needinfo)]

XUL element in HTML has been deprecated in bug 1027299, and is already broken in debug build since bug 1008455 (non-whitelisted usage will hit an assertion)

The check has since changed and moved in bug 1444193 and bug 1446831. The current whitelisted elements are here:

https://searchfox.org/mozilla-central/rev/39265dd58298c40bed029617ad408bf806cce761/dom/xul/nsXULElement.cpp#659-666

I will remove videocontrols and datetimebox in bug 1507895.

The usage of <xul:label> element was actually removed in bug 1495153. However when I tried to remove that from the list, dom/xul/test/test_import_xul_to_content.xul failed, which is the test check-in in bug 1027299. This test didn't fail solely because label is still on the list.

I guess we should use this bug to decide and implement disallowing XBL element in HTML for real and/or make the behavior of the debug and opt build consistent.

Comment 1

[Tim Guan-tin Chien [:timdream] (please needinfo)]

There is another debug assertion: bug 1426492 limits the allowed XBL bindings in the content process.