Closed
Bug 1263484
Opened 8 years ago
Closed 8 years ago
Remove MEMORY_JS_MAIN_RUNTIME_TEMPORARY_PEAK telemetry measurement
Categories
(Toolkit :: Telemetry, defect)
Toolkit
Telemetry
Tracking
()
RESOLVED
FIXED
mozilla48
Tracking | Status | |
---|---|---|
firefox48 | --- | fixed |
People
(Reporter: n.nethercote, Assigned: n.nethercote)
References
Details
Attachments
(1 file)
4.08 KB,
patch
|
chutten
:
review+
|
Details | Diff | Splinter Review |
MEMORY_JS_MAIN_RUNTIME_TEMPORARY_PEAK was added a long time ago when it could get very high -- e.g. 100s of MiBs -- during JS parsing. But things improved and now it rarely exceeds a handful of MiBs, and it's just not really an interesting thing to report via telemetry any more.
Assignee | ||
Comment 1•8 years ago
|
||
I think I found all the spots that needed changing...
Attachment #8739813 -
Flags: review?(chutten)
Assignee | ||
Updated•8 years ago
|
Assignee: nobody → n.nethercote
Status: NEW → ASSIGNED
Comment 2•8 years ago
|
||
Comment on attachment 8739813 [details] [diff] [review] Remove the JS_MAIN_RUNTIME_TEMPORARY_PEAK telemetry measurement Review of attachment 8739813 [details] [diff] [review]: ----------------------------------------------------------------- That is indeed all the Telemetry stuff. A casual dxr for JSMainRuntimeTemporaryPeak comes up with 34 results which mostly seem to be self-referential. Can the whole mechanism be removed as well? (Out of scope for this bug of course, but maybe a good first project for someone you want to have learn the internals of JS memory reporting?)
Attachment #8739813 -
Flags: review?(chutten) → review+
Assignee | ||
Comment 3•8 years ago
|
||
Thank you for the fast review.
> A casual dxr for JSMainRuntimeTemporaryPeak comes up with 34 results which
> mostly seem to be self-referential. Can the whole mechanism be removed as
> well? (Out of scope for this bug of course, but maybe a good first project
> for someone you want to have learn the internals of JS memory reporting?)
Good question. I think it's still a useful thing to show in about:memory, but it's not useful enough to report via telemetry.
Assignee | ||
Comment 4•8 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/9ff4ef78ee4fccb7129a14b0853d58fc690522b1 Bug 1263484 - Remove the JS_MAIN_RUNTIME_TEMPORARY_PEAK telemetry measurement. r=chutten.
Comment 5•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/9ff4ef78ee4f
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox48:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
You need to log in
before you can comment on or make changes to this bug.
Description
•