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)
Core
JavaScript Engine
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
Assignee | ||
Comment 1•4 years ago
|
||
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
Assignee | ||
Comment 2•4 years ago
|
||
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
Comment 5•4 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
status-firefox81:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 81 Branch
Updated•4 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•