Closed Bug 329581 Opened 14 years ago Closed 14 years ago
Most observers are not removed on shutdown
Much of the calendar code requires observers of one kind or another. Although all components which implement an addObserver function also implement a removeObserver function, most callers do not take advantage of this. The easy cases to fix are the front-end cases, like the views, which can just use destructors or onunload listeners to tidy up (most don't even do this!). The tougher cases are where backend (component) code is adding observers. These are probably going to need shutdown() methods added to them. (possibly incomplete) list of unremoved observers: calendar-multiday-view's mObserver calendar-month-view's mObserver agendaTreeView's calendarObserver calendar-todo-list's mOperationListener ltnCompositeCalendarObserver ltnCalendarManagerObserver calCalendarManagerObserver All error-announcer observers All helper-observers in the composite calendar ics calendar's observer on the memory calendar For the most part, I'm working off of http://landfill.mozilla.org/mxr-test/mozilla/search?string=addObserver&find=calendar&filter=
The bugspam monkeys have struck again. They are currently chewing on default assignees for Calendar. Be afraid for your sanity!
Assignee: base → nobody
I'll ask for review here once Lightning is unbroken and I can test there too. These are some major leaks on Mac and on Firefox Lightning that I'd like to fix.
Assignee: nobody → jminta
Status: NEW → ASSIGNED
Unbitrots the previous patch. Now tested on Lightning.
Comment on attachment 234954 [details] [diff] [review] unbitrotted Yay! Less leaking! r1=lilmatt
Attachment #234954 - Flags: first-review?(mattwillis) → first-review+
Comment on attachment 234954 [details] [diff] [review] unbitrotted r=dmose
Attachment #234954 - Flags: second-review?(dmose) → second-review+
patch checked in.
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.