734249 - test_bug378518.xul: "ASSERTION: Invalid extends value: '!nsXBLService::IsChromeOrResourceURI(doc->GetDocumentURI())'"

Status: RESOLVED WONTFIX

(Core :: XBL, defect)

Description

[Serge Gautherie (:sgautherie)]

https://tbpl.mozilla.org/php/getParsedLog.php?id=9916100&tree=Firefox&full=1
Rev3 WINNT 5.1 mozilla-central debug test mochitest-other on 2012-03-08 09:58:31 PST for push 9ddf749edcf2
+
http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1331231261.1331235914.4197.gz&fulltext=1
WINNT 5.2 comm-central-trunk debug test mochitest-other on 2012/03/08 10:27:41
{
1757 INFO TEST-START | chrome://mochitests/content/chrome/content/xbl/test/test_bug378518.xul
###!!! ASSERTION: Invalid extends value: '!nsXBLService::IsChromeOrResourceURI(doc->GetDocumentURI())', file e:/builds/moz2_slave/m-cen-w32-dbg/build/content/xbl/src/nsXBLPrototypeBinding.cpp, line 2391
xul!mozilla::scache::PathifyURI+0x000000000041D2FD
[...]
}

Comment 1

[[e-mail]]

The test uses a binding with extends="xul:checkbox" although nowadays the only allowed elements are autorepeatbutton, box, browser, button, hbox, image, menu, menubar, menuitem, menupopup, row, slider, spacer, splitter, text and tree.

I don't know if the test is at fault or whether checkbox should be allowed.

Comment 2

[Neil Deakin]

The test was written for the bug that removed support, so it seems like there's no bug here.

Comment 3

[Serge Gautherie (:sgautherie)]

(In reply to Neil Deakin from comment #2)

Nothing was explicitly written about that assertion in the test or in the bug.
If it is actually on purpose, then it deserves to be documented, doesn't it?

Comment 4

[Neil Deakin]

Adding a note in the bug would be good.

Comment 5

[Neil Deakin]

I meant adding a note in the test.

Comment 6

[Boris Zbarsky [:bzbarsky]]

XBL is now disabled in Firefox (Bug 1583314) and is in the process of being removed from Gecko (Bug 1566221), so closing bugs requesting changes to its implementation as wontfix.