1808973 - Unused references to StructuredLog.jsm are lying around and break things if referencing missing chrome files is forbidden

Status: RESOLVED FIXED

(Testing :: Mochitest, task)

Description

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

After bug 1801368 the references in the harness files ( testing/mochitest/browser-harness.xhtml and testing/mochitest/harness.xhtml ) were left behind, as well as some comment references. They don't load, however, as the file has been moved and these aren't ChromeUtils.import calls so don't get shimmed by the jsm-to-sys.mjs conversion work - the harness relies on the import at https://searchfox.org/mozilla-central/rev/72d9bf2d4128990cd1f349d4e15003e515277982/testing/mochitest/tests/SimpleTest/TestRunner.js#18-23 to access structured logging.

Comment 1

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

Set release status flags based on info from the regressing bug 1801368

Comment 2

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

Attachment: Bug 1808973 - remove obsolete references to StructuredLog.jsm harness file, r?Standard8

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/bddbe2f74b55
remove obsolete references to StructuredLog.jsm harness file, r=Standard8

Comment 4

[Norisz Fay [:noriszfay]]

https://hg.mozilla.org/mozilla-central/rev/bddbe2f74b55

Comment 5

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

The patch landed in nightly and beta is affected.
:Gijs, is this bug important enough to require an uplift?

• If yes, please nominate the patch for beta approval.
• If no, please set status-firefox109 to wontfix.

For more information, please visit auto_nag documentation.

Comment 6

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

(changing some fields to make the bot happier...)

This isn't worth uplifting, certainly not at this stage of beta.