715950 - add telemetry for how many timeouts are attached to a window

Status: RESOLVED WORKSFORME

(Core :: DOM: Core & HTML, defect)

Description

[Nathan Froyd [:froydnj]]

Splitting this off from bug 715380.

Also, I'm not entirely sure what the desired effect is here.  Do you want to know the number of pending timeouts when setTimeout is called?  Or something else?

Comment 1

[(dormant account)]

Just need to poll amount of timeouts at a semi-regular intervals. Could do it when memory polling is done in TelemetryPing.js.

Comment 2

[Nathan Froyd [:froydnj]]

Following on to bz's comment in bug 715953, why do we want this?

Comment 3

[(dormant account)]

(In reply to Nathan Froyd (:froydnj) from comment #2)
> Following on to bz's comment in bug 715953, why do we want this?

Need to know potential for interruptions caused by timeouts.

Comment 4

[Nathan Froyd [:froydnj]]

Do we care about this now that we a) know that excessive timers are a problem; and b) those problems are being addressed?

Comment 5

[Lawrence Mandel [:lmandel] (use needinfo)]

(In reply to Nathan Froyd (:froydnj) from comment #4)
> Do we care about this now that we a) know that excessive timers are a
> problem; and b) those problems are being addressed?

In addition to problem determination Telemetry can be useful in monitoring for regressions. I'm not that familiar with this specific issue. Would it be useful to monitor for regressions in this case?

Comment 6

[(dormant account)]

I don't think we care too much about this. Can close it and someone can reopen it if it still matters.

Comment 7

[Nathan Froyd [:froydnj]]

Yay for getting bugs off my list, then.