Syntax error breaks removing closed tabs

RESOLVED FIXED in Firefox 13

Status

()

Firefox for Android
General
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: mfinkle, Assigned: mfinkle)

Tracking

unspecified
Firefox 14
x86
Linux
Points:
---

Firefox Tracking Flags

(firefox13 fixed, firefox14 fixed)

Details

Attachments

(1 attachment)

Created attachment 608562 [details] [diff] [review]
patch

The code throws an exception and the tab is not completely destroyed or removed from the BrowserApp.tabs array. This patch fixes it.
Attachment #608562 - Flags: review?(wjohnston)
(Assignee)

Updated

6 years ago
Blocks: 697309
Duplicate of this bug: 738768
Attachment #608562 - Flags: review?(wjohnston) → review+
*rages against the insanity of using javascript for critical components*
https://hg.mozilla.org/integration/mozilla-inbound/rev/563021526711

Comment 4

6 years ago
Please can you set the target milestone + assignee when landing on inbound, along the lines of http://blog.bonardo.net/2012/03/23/how-you-can-help-mozilla-inbound-sheriffs-when-pushing :-)

https://hg.mozilla.org/mozilla-central/rev/563021526711
Assignee: nobody → mark.finkle
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 14
https://hg.mozilla.org/releases/mozilla-aurora/rev/c71d1eb305bc
status-firefox13: --- → fixed
status-firefox14: --- → fixed
You need to log in before you can comment on or make changes to this bug.