Closed Bug 388216 Opened 17 years ago Closed 17 years ago

Event Listeners are not removed in views

Categories

(Calendar :: Calendar Frontend, defect)

defect
Not set
trivial

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: Fallen, Assigned: Fallen)

Details

Attachments

(1 file)

Attached patch Fix listeners - v1 — — Splinter Review
In calendar-multiday-view, the scroll and resize event listeners are not removed on destruction. In the alarm dialog, the snooze and dismiss listeners are also not removed. This patch should do it.

I heard that window load and unload event listeners are automatically unloaded on window close, so calendar-minimonth-busy.js should be ok.
Attachment #272390 - Flags: review?(michael.buettner)
Comment on attachment 272390 [details] [diff] [review]
Fix listeners - v1

nice catch, philipp. r=mickey.
Attachment #272390 - Flags: review?(michael.buettner) → review+
Checked in on HEAD and MOZILLA_1_8_BRANCH

-> FIXED
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: