Add bytecode cache preferences to all.js

RESOLVED FIXED in Firefox 55

Status

()

RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: nbp, Assigned: nbp)

Tracking

unspecified
mozilla55
Points:
---

Firefox Tracking Flags

(firefox55 fixed)

Details

Attachments

(1 attachment)

Comment hidden (empty)
(Assignee)

Comment 1

2 years ago
Created attachment 8866839 [details] [diff] [review]
List JS bytecode cache preferences in all.js.

In addition to listing the preferences in about:config, I also extracted the
force option, renamed (bytecode_cache.eager) it and moved it into its own
function.
Attachment #8866839 - Flags: review?(mrbkap)
(Assignee)

Updated

2 years ago
Component: JavaScript Engine → DOM
Comment on attachment 8866839 [details] [diff] [review]
List JS bytecode cache preferences in all.js.

Review of attachment 8866839 [details] [diff] [review]:
-----------------------------------------------------------------

::: modules/libpref/init/all.js
@@ +210,5 @@
>  pref("dom.compartment_per_addon", true);
>  
> +// Whether to enable the JavaScript start-up cache. This causes one of the first
> +// execution to record the bytecode of the JavaScript function used, and save it
> +// to next to the existing cache entry. On the following loads of the same

"and save it in the existing cache entry"
Attachment #8866839 - Flags: review?(mrbkap) → review+

Comment 3

2 years ago
Pushed by npierron@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/031089f91a7c
List JS bytecode cache preferences in all.js. r=mrbkap

Comment 4

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/031089f91a7c
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
status-firefox55: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in before you can comment on or make changes to this bug.