HTML Desktop Notifications 1.0.6 causes immortal content page zombie compartments

RESOLVED FIXED

Status

()

Core
General
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: nmaier, Unassigned)

Tracking

(Blocks: 1 bug)

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [MemShrink], URL)

(Reporter)

Description

6 years ago
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.
(Reporter)

Comment 1

6 years ago
Cyril fixed this version 1.0.7 which I just fully reviewed.
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
(Reporter)

Comment 2

6 years ago
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.