Marionette is reported enabled when it fails to initialise

RESOLVED FIXED in Firefox 61

Status

defect
RESOLVED FIXED
Last year
Last year

People

(Reporter: ato, Assigned: ato)

Tracking

Trunk
mozilla62
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox61 fixed, firefox62 fixed)

Details

Attachments

(2 attachments)

When Marionette component initialisation fails, we fail to prevent
Marionette from reporting itself as enabled:

> [task 2018-06-06T17:20:02.966Z] 17:20:02     INFO -  1528305602959	Marionette	FATAL	Remote protocol server failed to start: [Exception... "File error: Not found"  nsresult: "0x80520012 (NS_ERROR_FILE_NOT_FOUND)"  location: "JS frame :: jar:file:///builds/worker/workspace/build/application/firefox/omni.ja!/components/marionette.js :: init/< :: line 444"  data: no] Stack trace: init/<()@jar:file:///builds/worker/workspace/build/application/firefox/omni.ja!/components/marionette.js:444
> [task 2018-06-06T17:20:03.028Z] 17:20:03     INFO -  1528305603021	Marionette	DEBUG	Remote service is active
Assignee: nobody → ato
Status: NEW → ASSIGNED
Comment on attachment 8983887 [details]
Bug 1467215 - Abort initialisation on fatal error.

https://reviewboard.mozilla.org/r/249732/#review255944

::: commit-message-04cc9:5
(Diff revision 1)
> +Bug 1467215 - Abort initialisation on fatal error. r?maja_zf
> +
> +When the Marionette component fails to initialise we fail to bail
> +out.  This causes Marionette to report itself as enabled through
> +an environment variable, an observer nortification, and a log message.

typo notification
Attachment #8983887 - Flags: review?(mjzffr) → review+
Pushed by atolfsen@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/0b9c59172bc1
Abort initialisation on fatal error. r=maja_zf
https://hg.mozilla.org/integration/autoland/rev/19607304a6b7
Reset prefs possibly set by failed init. r=maja_zf
https://hg.mozilla.org/mozilla-central/rev/0b9c59172bc1
https://hg.mozilla.org/mozilla-central/rev/19607304a6b7
Status: ASSIGNED → RESOLVED
Closed: Last year
Resolution: --- → FIXED
Target Milestone: --- → mozilla62
Sheriffs: Uplift to beta if possible, please.
Whiteboard: [checkin-needed-beta]
You need to log in before you can comment on or make changes to this bug.