729948 - HTML Desktop Notifications 1.0.6 causes immortal content page zombie compartments

Status: RESOLVED FIXED

(Core :: General, defect)

Description

[Nils Maier [:nmaier]]

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.

Comment 1

[Nils Maier [:nmaier]]

Cyril fixed this version 1.0.7 which I just fully reviewed.

Comment 2

[Nils Maier [:nmaier]]

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.