tl;dr - If you update a test harness pref in build/automation.py.in, please also update it in testing/profiles/prefs_general.js. This is temporary and I am working to remove the automation.py.in location in bug 830430[1]

---

Currently prefs that are common to our test harnesses are hardcoded in a string of Javascript inside of automation.py. This is bad for many reasons:

1) It isn't obvious to find - as a result prefs for various harnesses have been littered throughout the tree 2) Most of the newer harnesses don't use automation.py - these have no way of taking advantage of these prefs and must duplicate them 3) Storing Javascript in a python string and writing to a file is just yucky in general - it should start out in a js file

Instead we have decided to create a single canonical directory that will store all prefs (and other profile information). This location is testing/profiles.

So, if you update a test harness pref in build/automation.py.in, please also update it in testing/profiles/prefs_general.js. Having two locations for prefs is bad, and I'm working to remove the automation.py location in bug 830430[1]. I'll make sure the two locations are in sync before doing so.

If you have any questions, concerns or suggestions feel free to contact me or post in the bug.

Thanks,
Andrew

[1] https://bugzilla.mozilla.org/show_bug.cgi?id=830430

_______________________________________________
dev-platform mailing list
dev-platform@lists.mozilla.org
https://lists.mozilla.org/listinfo/dev-platform

Reply via email to