Closed Bug 729948 Opened 13 years ago Closed 13 years ago

HTML Desktop Notifications 1.0.6 causes immortal content page zombie compartments

Categories

(Core :: General, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: nmaier, Unassigned)

References

()

Details

(Whiteboard: [MemShrink])

Regular restart-required addon, implementing the Web Notifications API STR: - Open some web notification website, e.g. http://www.thecssninja.com/demo/web_notifications/ or http://code.google.com/p/ff-html5notifications/ - Click the corresponding button/link to make the notification approval bar appear and allow the notifications - Click the corresponding button/link to make the notification appear, if this is a distinct step - Close the corresponding tab - about:memory?verbose + Minimize memory usage -> One content page compartment per notification site stays behind Author CC'ed.
Cyril fixed this version 1.0.7 which I just fully reviewed.
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
The add-on kept some references to the window and similar for each notification around. The fix was to add an unload event listener to clear that stuff again.
You need to log in before you can comment on or make changes to this bug.