The default bug view has changed. See this FAQ.

Hande browser-lastwindow-close events

RESOLVED FIXED in Firefox 11

Status

()

Firefox for Android
General
P1
normal
RESOLVED FIXED
5 years ago
5 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

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

Comment 1

5 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+
(Assignee)

Comment 3

5 years ago
Landed on inbound:
http://hg.mozilla.org/integration/mozilla-inbound/rev/a9f4c0795560

Updated

5 years ago
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
(Assignee)

Comment 5

5 years ago
Passed try tests:
https://tbpl.mozilla.org/?tree=Try&rev=84da3ebd9c39

Re-landed on mozilla-inbound:
http://hg.mozilla.org/integration/mozilla-inbound/rev/460ffa419de5
https://hg.mozilla.org/mozilla-central/rev/460ffa419de5
Status: NEW → RESOLVED
Last Resolved: 5 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 8

5 years ago
Comment on attachment 586226 [details] [diff] [review]
patch

[Triage Comment]
Mobile only - approved for Aurora.
Attachment #586226 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
https://hg.mozilla.org/releases/mozilla-aurora/rev/5cbc92157e7d
status-firefox11: affected → fixed
You need to log in before you can comment on or make changes to this bug.