Hande browser-lastwindow-close events

RESOLVED FIXED in Firefox 11

Status

()

P1
normal
RESOLVED FIXED
7 years ago
7 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)

(Assignee)

Description

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

Comment 1

7 years ago
Created attachment 586226 [details] [diff] [review]
patch
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
Last Resolved: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 12
status-firefox11: --- → affected
status-firefox12: --- → fixed
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.