Closed Bug 715673 Opened 9 years ago Closed 9 years ago

Hande browser-lastwindow-close events

Categories

(Firefox for Android :: General, defect, P1)

All
Android
defect

Tracking

()

RESOLVED FIXED
Firefox 12
Tracking Status
firefox11 --- fixed
firefox12 --- fixed
fennec 11+ ---

People

(Reporter: bnicholson, Assigned: bnicholson)

Details

Attachments

(1 file)

We should handle/send browser-lastwindow-close-requested and browser-lastwindow-close-granted as we did in XUL Fennec.
Attached 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: 9 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.