Closed Bug 389158 Opened 19 years ago Closed 18 years ago

Event Listeners are not removed in todo-list.js

Categories

(Calendar :: Lightning Only, defect)

defect
Not set
minor

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: Fallen, Assigned: Fallen)

Details

Attachments

(1 file)

In http://lxr.mozilla.org/mozilla/source/calendar/lightning/content/todo-list.js#70 some event listeners are added. These are never removed which causes a memory leak.
Attached patch remove listeners - v1 β€” β€” Splinter Review
Assignee: nobody → philipp
Status: NEW → ASSIGNED
Attachment #273299 - Flags: review?(daniel.boelzle)
Comment on attachment 273299 [details] [diff] [review] remove listeners - v1 Since Berend takes care of the today pane, shifting review to him.
Attachment #273299 - Flags: review?(daniel.boelzle) → review?(Berend.Cornelius)
Comment on attachment 273299 [details] [diff] [review] remove listeners - v1 r=dbo
Attachment #273299 - Flags: review?(Berend.Cornelius) → review+
Checked in on HEAD and MOZILLA_1_8_BRANCH -> FIXED
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Target Milestone: --- → 0.7
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: