Closed Bug 1658621 Opened 4 years ago Closed 4 years ago

Remove JS_WEB_PARSER_COMPILE_LAZY_AFTER_MS/JS_PRIVILEGED_PARSER_COMPILE_LAZY_AFTER_MS probes

Categories

(Core :: JavaScript Engine, task)

task

Tracking

()

RESOLVED FIXED
81 Branch
Tracking Status
firefox81 --- fixed

People

(Reporter: arai, Assigned: arai)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

JS_WEB_PARSER_COMPILE_LAZY_AFTER_MS probe [1] and JS_PRIVILEGED_PARSER_COMPILE_LAZY_AFTER_MS probe [2] has been expired.
We should remove the probe and related code around ScriptSource and compilation.

This helps simplifying stencil instantiation and make it separate from bytecode compilation.

[1] https://searchfox.org/mozilla-central/rev/8df04ff073d0fa70f692935c5dcddc0e23cb0117/toolkit/components/telemetry/Histograms.json#1156-1167
[2] https://searchfox.org/mozilla-central/rev/8df04ff073d0fa70f692935c5dcddc0e23cb0117/toolkit/components/telemetry/Histograms.json#1144-1155

Yoric, is there any chance the probe/code is still somehow used?
If not, I'll remove them.

Flags: needinfo?(dteller)
Summary: Remove JS_WEB_PARSER_COMPILE_LAZY_AFTER_MS"/JS_PRIVILEGED_PARSER_COMPILE_LAZY_AFTER_MS probes → Remove JS_WEB_PARSER_COMPILE_LAZY_AFTER_MS/JS_PRIVILEGED_PARSER_COMPILE_LAZY_AFTER_MS probes

Good idea.

Flags: needinfo?(dteller)
Pushed by arai_a@mac.com:
https://hg.mozilla.org/integration/autoland/rev/f56158e0fd57
Remove JS_WEB_PARSER_COMPILE_LAZY_AFTER_MS/JS_PRIVILEGED_PARSER_COMPILE_LAZY_AFTER_MS probes. r=Yoric
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 81 Branch
Blocks: stencil-backlog
No longer blocks: stencil
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: