If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Need to handle DOMWindowClose

VERIFIED FIXED in fennec1.0b3

Status

Fennec Graveyard
General
VERIFIED FIXED
8 years ago
8 years ago

People

(Reporter: Gavin, Assigned: Gavin)

Tracking

Trunk
fennec1.0b3

Details

Attachments

(2 attachments)

To make sure that script-opened windows close()ing themselves aren't able to shut down the entire app.
Created attachment 388730 [details]
testcase

STR:
1) Click the "open popup"
2) in the resulting tab, click "close popup"

Expected: tab closes
Actual: entire window closes
Created attachment 388733 [details] [diff] [review]
patch

Firefox's handler (in tabbrowser.xml) includes a workaround for bug 265790 that was landed in bug 263844. It's no longer necessary now that bug 265790 was fixed otherwise, AFAICT (tested with the testcase in bug 265790). I'll file a Firefox bug on removing it.
Assignee: nobody → gavin.sharp
Status: NEW → ASSIGNED
Attachment #388733 - Flags: review?(mark.finkle)

Updated

8 years ago
tracking-fennec: ? → 1.0b3+
Attachment #388733 - Flags: review?(mark.finkle) → review+
https://hg.mozilla.org/mobile-browser/rev/74839270d6e6
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → B3
Verified fixed in the Maemo build form 8/11/2009.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.