Created attachment 302630 [details] [diff] [review] Patch, v1 Working on making the cycle collector free closed DOM windows I found that the download status panel listener is never removed when the browser closes. This keeps all DOM windows open until the app shuts down and the download manager clears its listener list. We need to figure out a generic way to prevent this sort of thing, but that's for another bug.
Attachment #302630 - Flags: review?(mano)
Wouldn't it be better to just unregister the listener on window unload?
11 years ago
onunload="BrowserShutdown()", so I think this is doing what you're talking about?
right, sorry. Missed that.
Comment on attachment 302630 [details] [diff] [review] Patch, v1 I would rather add an unit method to the listener object, instead of exposing isListening.
Attachment #302630 - Flags: review?(mano) → review-
Created attachment 303159 [details] [diff] [review] Patch, v2 Ok, with uninit.
Comment on attachment 303159 [details] [diff] [review] Patch, v2 nit: remove the braces in uninit. r=mano otherwise.
Attachment #303159 - Flags: review?(mano) → review+
Flags: blocking-firefox3? → blocking-firefox3+
Priority: -- → P1
Status: NEW → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → FIXED
*** VERIFIED I even 100% remember this bug, it's fixed now. Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9b5pre) Gecko/2008030607 Minefield/3.0b5pre -Mike
You need to log in before you can comment on or make changes to this bug.