Closed
Bug 1239363
Opened 8 years ago
Closed 8 years ago
Failing to parse marionette.logging preference when it is not a string
Categories
(Remote Protocol :: Marionette, defect)
Remote Protocol
Marionette
Tracking
(firefox46 fixed)
RESOLVED
FIXED
mozilla46
Tracking | Status | |
---|---|---|
firefox46 | --- | fixed |
People
(Reporter: ato, Assigned: ato)
Details
(Keywords: pi-marionette-server)
Attachments
(1 file)
Marionette fails to start when the marionette.logging preference isn’t a string: 06:44:13 INFO - JavaScript error: resource://gre/components/marionettecomponent.js, line 63: TypeError: s.toLowerCase is not a function
Assignee | ||
Updated•8 years ago
|
Assignee | ||
Comment 1•8 years ago
|
||
The marionette.logging preference used to have a boolean type. To maintain backwards compatibility with earlier clients we keep accepting boolean types as input. Previously setting it to true meant you wanted all the logs, and this change respects that. Review commit: https://reviewboard.mozilla.org/r/30741/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/30741/
Attachment #8707510 -
Flags: review?(dburns)
Updated•8 years ago
|
Attachment #8707510 -
Flags: review?(dburns) → review+
Comment 2•8 years ago
|
||
Comment on attachment 8707510 [details] MozReview Request: Bug 1239363 - Accept non-string types for marionette.logging pref; r=AutomatedTester https://reviewboard.mozilla.org/r/30741/#review27543
Updated•8 years ago
|
Severity: normal → major
Comment hidden (Intermittent Failures Robot) |
Comment 5•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/b28a5678774d
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox46:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla46
Comment hidden (Intermittent Failures Robot) |
Assignee | ||
Comment 7•8 years ago
|
||
FWIW the OrangeFactor issues here should be resolved by bug 1239371, which is now on inbound.
Comment hidden (Intermittent Failures Robot) |
Updated•1 year ago
|
Product: Testing → Remote Protocol
You need to log in
before you can comment on or make changes to this bug.
Description
•