Hande browser-lastwindow-close events

RESOLVED FIXED in Firefox 11

Status

()

defect
P1
normal
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: bnicholson, Assigned: bnicholson)

Tracking

unspecified
Firefox 12
All
Android
Points:
---

Firefox Tracking Flags

(firefox11 fixed, firefox12 fixed, fennec11+)

Details

Attachments

(1 attachment)

We should handle/send browser-lastwindow-close-requested and browser-lastwindow-close-granted as we did in XUL Fennec.
Assignee

Comment 1

8 years ago
Posted patch patchSplinter Review
Attachment #586226 - Flags: review?(mark.finkle)
Comment on attachment 586226 [details] [diff] [review]
patch


>+        lastBrowser = false;
>+    }
>+    if (lastBrowser) {
>+      // Let everyone know we are closing the last browser window

Add a space between the blocks
Attachment #586226 - Flags: review?(mark.finkle) → review+
Priority: -- → P1
tracking-fennec: --- → 11+
One of three changesets backed out of inbound due to test coalescing making it hard to identify which caused the native android test failures:
https://hg.mozilla.org/integration/mozilla-inbound/rev/1927c7905f5e
https://hg.mozilla.org/mozilla-central/rev/460ffa419de5
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 12
Comment on attachment 586226 [details] [diff] [review]
patch

[Approval Request Comment]
needed to correctly handle saving the session
Attachment #586226 - Flags: approval-mozilla-aurora?
Comment on attachment 586226 [details] [diff] [review]
patch

[Triage Comment]
Mobile only - approved for Aurora.
Attachment #586226 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
You need to log in before you can comment on or make changes to this bug.