Throttle HUD memory collection

RESOLVED FIXED in Firefox 42

Status

defect
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: rnicoletti, Assigned: janx)

Tracking

unspecified
FxOS-S2 (10Jul)
ARM
Gonk (Firefox OS)
Dependency tree / graph

Firefox Tracking Flags

(firefox42 fixed)

Details

Attachments

(2 attachments, 2 obsolete attachments)

Currently when the HUD is observing memory usage there is a performance degradation observed, especially time-to-load. It would be beneficial to be able to throttle either the frequency with which the memory is observed and/or the frequency of the advanced telemetry memory events sent to the system app.
I agree that HUD memory collection should be significantly throttled.

It was set to poll somewhat aggressively in order to display precise memory graphs, but sub-second precision is probably overkill for this. Also, the memory metric is mostly used by widgets and telemetry, rather than live graphs, and these metric consumers have even less need for high-frequency.
Assignee: nobody → janx
Summary: Throttle advanced telemetry memory events → Throttle HUD memory collection
Drive-by fix.
Depends on: 1178512
Less frequent polling is better (see comment #1).
Attachment #8626638 - Attachment is obsolete: true
Attachment #8628845 - Flags: review?(jryans)
Drive-by ES6 style refactor.
Attachment #8626636 - Attachment is obsolete: true
Attachment #8628846 - Flags: review?(jryans)
Attachment #8628845 - Flags: review?(jryans) → review+
Attachment #8628846 - Flags: review?(jryans) → review+
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/c539b6f091fc
https://hg.mozilla.org/mozilla-central/rev/b45dd8808e4c
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → FxOS-S2 (10Jul)
You need to log in before you can comment on or make changes to this bug.