Closed Bug 1225861 Opened 4 years ago Closed 4 years ago

Fix collection of LOW_MEMORY_EVENTS_{PHYSICAL,VIRTUAL} histograms

Categories

(Toolkit :: Telemetry, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla45
Tracking Status
firefox45 --- fixed

People

(Reporter: poiru, Assigned: poiru)

References

Details

Attachments

(1 file)

Bug 913260 unintentionally changed[0] LOW_MEMORY_EVENTS_{PHYSICAL,VIRTUAL} to MEMORY_EVENTS_{PHYSICAL,VIRTUAL}.

[0]: http://hg.mozilla.org/mozilla-central/diff/9e162ee7d4d1/toolkit/components/telemetry/TelemetryPing.js
Attachment #8689005 - Flags: review?(n.nethercote)
Comment on attachment 8689005 [details] [diff] [review]
Fix collection of LOW_MEMORY_EVENTS_{PHYSICAL,VIRTUAL} histograms

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

Good catch. This means we haven't been collecting these values for telemetry for over 2 years and nobody noticed until now. That's a decent argument for removing them altogether, though this is fine in the meantime...
Attachment #8689005 - Flags: review?(n.nethercote) → review+
https://hg.mozilla.org/mozilla-central/rev/d0c158b50036
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla45
You need to log in before you can comment on or make changes to this bug.