Closed
Bug 1345906
Opened 7 years ago
Closed 7 years ago
fuzzing.enabled pref needs ReloadPrefsCallback to work with the JS engine
Categories
(Core :: JavaScript Engine, defect)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
mozilla55
Tracking | Status | |
---|---|---|
firefox55 | --- | fixed |
People
(Reporter: decoder, Assigned: decoder)
Details
(Keywords: sec-want, Whiteboard: [adv-main55-])
Attachments
(1 file)
The fuzzing.enabled pref currently only works when it is followed by any other preference that starts with "javascript." because that triggers a preference reload for the JS engine (yes, that is insane to debug because the pref will not be used when you put it at the end of the prefs.js file, but will be used after the first start if you use other javascript options because of the alphabetic reordering of prefs). Thanks to jandem for debugging this together with me :) The solution here is to add a ReloadPrefsCallback for the fuzzing.enabled pref as well.
Comment hidden (mozreview-request) |
Comment 2•7 years ago
|
||
mozreview-review |
Comment on attachment 8845473 [details] Bug 1345906 - Add ReloadPrefsCallback for fuzzing.enabled pref. https://reviewboard.mozilla.org/r/118642/#review120536
Attachment #8845473 -
Flags: review?(jdemooij) → review+
Pushed by choller@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/eba81b98b048 Add ReloadPrefsCallback for fuzzing.enabled pref. r=jandem
Comment 4•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/eba81b98b048
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
Updated•7 years ago
|
Whiteboard: [adv-main55-]
You need to log in
before you can comment on or make changes to this bug.
Description
•