Closed Bug 1497962 Opened 2 years ago Closed 2 years ago

Ensure that extra_prefs are passed as a list

Categories

(Testing :: web-platform-tests, enhancement)

Version 3
enhancement
Not set
normal

Tracking

(firefox64 fixed)

RESOLVED FIXED
mozilla64
Tracking Status
firefox64 --- fixed

People

(Reporter: KWierso, Assigned: KWierso)

Details

Attachments

(1 file)

I was trying to set a preference via extra_prefs for some work in bug 1496627 (later figured out I didn't need that preference, so the patch in there doesn't include it) and was having a hard time figuring out why I was getting weird errors. 

Turns out, wptcommandline.py expects extra_prefs to be passed as a python list (eg ["pref=value"] or ["pref1=value","pref2=value2"]), so but it will happily attempt to use the string "pref=value" and then fail miserably.
Pushed by james@hoppipolla.co.uk:
https://hg.mozilla.org/integration/autoland/rev/43470b75b0d4
Ensure extra_prefs is passed as a list r=jgraham
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/13476 for changes under testing/web-platform/tests
https://hg.mozilla.org/mozilla-central/rev/43470b75b0d4
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla64
You need to log in before you can comment on or make changes to this bug.