Closed Bug 504352 Opened 15 years ago Closed 15 years ago

Need to handle DOMWindowClose

Categories

(Firefox for Android Graveyard :: General, defect)

defect
Not set
normal

Tracking

(fennec1.0b3+)

VERIFIED FIXED
fennec1.0b3
Tracking Status
fennec 1.0b3+ ---

People

(Reporter: Gavin, Assigned: Gavin)

Details

Attachments

(2 files)

To make sure that script-opened windows close()ing themselves aren't able to shut down the entire app.
Attached file testcase
STR:
1) Click the "open popup"
2) in the resulting tab, click "close popup"

Expected: tab closes
Actual: entire window closes
Attached patch patchSplinter Review
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)
tracking-fennec: ? → 1.0b3+
Attachment #388733 - Flags: review?(mark.finkle) → review+
https://hg.mozilla.org/mobile-browser/rev/74839270d6e6
Status: ASSIGNED → RESOLVED
Closed: 15 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.

Attachment

General

Created:
Updated:
Size: