On 2014-03-19, 10:25 PM, Boris Zbarsky wrote:
On 3/19/14 9:41 PM, Justin Dolske wrote:
It uses a weak reference with the observer service, plus a dummy strong
reference (via addEventListener()) to automatically manage the
lifetime... When the node/document does away, so does the event listener.

This is sort of ok for notifications that fire a lot, but for
notifications that fire rarely, or never, this effectively leaks a word
of memory in the observer service.  And then they all get touched at
shudown.  We've had bugs with hundreds of thousands of dead weak
observers all hanging out in the observer service making shutdown take
minutes.  :(

Why do we have to touch that list on shutdown?

_______________________________________________
dev-platform mailing list
dev-platform@lists.mozilla.org
https://lists.mozilla.org/listinfo/dev-platform

Reply via email to