1904406 - xpcshell tests crash when loading about:certificate due to common.css missing file error

Status: NEW

(Fenix :: Crash Reporting, defect)

Description

[Jared Hirsch [:jhirsch] (he/him) (Needinfo please)]

In bug 1845340 an xpcshell test was added (test_out_of_process_use) that attempts to verify that OHTTP config is not accessible from a content process. As a setup step, this test loads the about:certificate page. On Android, attempting to load this page from the test crashes with the error message:

Mozilla crash reason: Missing chrome or resource URLs: chrome://global/skin/in-content/common.css

Perhaps it's worth noting that the specific test utility method that triggers this line is:

let page = await XPCShellContentUtils.loadContentPage("about:certificate", {
  remote: true,
});

and I only see the error message if I change the argument from remote: true to remote: false.

Now, I am able to load the about:certificate page in the same local emulator that crashed when I tried to load about:certificate in an xpcshell test, so I'd guess the bug here might be that some necessary resources aren't made available to the tests.

Comment 1

[:Gijs (out for now; he/him)]

(In reply to Jared Hirsch [:jhirsch] (he/him) (Needinfo please) from comment #0)

Now, I am able to load the about:certificate page in the same local emulator that crashed when I tried to load about:certificate in an xpcshell test, so I'd guess the bug here might be that some necessary resources aren't made available to the tests.

Apologies for not being clearer about this in bug 1845340, but the crashing is (deliberately) restricted to automation. So the same behaviour (missing CSS file) is still happening in the emulator, it's just not causing crashes. https://searchfox.org/mozilla-central/rev/64ddb621a0d3905fc2e3df475517d4163d377b22/toolkit/themes/shared/desktop-jar.inc.mn#59 and https://searchfox.org/mozilla-central/search?q=common.css&path=jar&regexp=false indicate we just don't ship these CSS files at all on Android.

I'm not sure if you were hoping to use this bug to purely fix this specific test to not rely on about:certificate, or to fix about:certificate on Android (by unshipping it, not relying on those CSS files, or shipping the CSS files and other required resources on Android). We should probably morph it in either direction and/or file a separate bug for the other part of the work :-)

Comment 2

[BugBot [:suhaib / :marco/ :calixte]]

The severity field is not set for this bug.
:aaronmt, could you have a look please?

For more information, please visit BugBot documentation.

Comment 3

[BugBot [:suhaib / :marco/ :calixte]]

The severity field is not set for this bug.
:royang, could you have a look please?

For more information, please visit BugBot documentation.

Comment 4

[BugBot [:suhaib / :marco/ :calixte]]

The Bugbug bot thinks this bug should belong to the 'Fenix::Crash Reporting' component, and is moving the bug to that component. Please correct in case you think the bot is wrong.