818806 - NR_ASYNC_SCHEDULE() causes leaks if the timers are still active at shutdown

Status: NEW

(Core :: WebRTC: Networking, defect)

Description

[Randell Jesup [:jesup] (needinfo me)]

See bug 816640.  Looking at that, I realized there's no way to cancel these timers, and since they hold a ref to the timer the timer will never be deleted IF we shutdown while the timer is still active.  The callback will get Released() in timer thread shutdown (though hopefully it doesn't need anything significant to be running to die), so we'd likely leak only the Timer.

It's likely to be better to Cancel the timers when the PeerConnection dies.

Note: it only applies if we shutdown with it still running.

If this triggers automation issues due to the possible leak, it may need to be upgraded

Comment 1

[Randell Jesup [:jesup] (needinfo me)]

likely moot as we ensure we stop things before shutdown gets that far.