Open Bug 1377585 Opened 7 years ago Updated 2 years ago

Use a timeout to re-register for cc notifications after |gatherMemory| completes

Categories

(Toolkit :: Telemetry, enhancement, P3)

enhancement

Tracking

()

Tracking Status
firefox57 --- fix-optional

People

(Reporter: erahm, Unassigned)

References

Details

From bug 1375281, comment 14 we would like to stop always observing the cc-end notification. Instead we want to:

#1 - Register for notification
#2 - Get notified, unregister
#3 - Dispatch the idle |gatherMemory| request
#4 - Issue a setTimeout to re-register at the end of |gatherMemory|

This will hopefully avoid redundant round trips where we observe the notification, calculate the amount of time that's passed and then ignore the notification.
Component: XPCOM → Telemetry
Product: Core → Toolkit
Instead of improving how to use the notification, could we directly record memory measurements into Telemetry from the CC (at some interval)?
Flags: needinfo?(erahm)
(In reply to Georg Fritzsche [:gfritzsche] from comment #1)
> Instead of improving how to use the notification, could we directly record
> memory measurements into Telemetry from the CC (at some interval)?

We're explicitly trying to avoid adding overhead to CC for telemetry measurements.
Flags: needinfo?(erahm)
Priority: -- → P3
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.