1331512 - Many XUL chrome mochitests are mis-including the mochikit test.css style sheet

Status: RESOLVED DUPLICATE of bug 1333164

(Core :: General, defect)

Description

[:Gijs (he/him)]

+++ This bug was initially created as a clone of Bug #1330123 +++

> 1122 WARNING: NS_ENSURE_SUCCESS(rv, rv) failed with result 0x80520012: file layout/style/Loader.cpp, line 2159

This warning [1] shows up in the following test suites:

> <elide devtools>

>     25 - test-linux64/debug-mochitest-chrome-3 c3

This and all the other mochitest-chrome hits are because some tests do this:

<?xml-stylesheet type="text/css" href="/tests/SimpleTest/test.css"?>

(cf. https://dxr.mozilla.org/mozilla-central/source/js/xpconnect/tests/chrome/test_bug793433.xul?q=path%3Atest_bug793433.xul&redirect_type=single#3 )

This doesn't work, because the / resolves relative to the chrome:// package which is chrome://mochitests/, and it misses out the requisite "content" path fragment.

We should mass-rewrite these to the correct URL, or remove them altogether (as the tests are clearly passing and introducing the new CSS may in principle break them).

DXR finds 42 hits, which I'm sure is a freak accident:

https://dxr.mozilla.org/mozilla-central/search?q=path%3A.xul+%27%22%2Ftests%2FSimpleTest%2Ftest.css&redirect=false

Comment 1

[Eric Rahm [:erahm]]

Bug 1333164 is probably a dup of this, but it gives a clearer breakdown.