Closed
Bug 898781
Opened 11 years ago
Closed 11 years ago
Opening new tab sometimes causes exceptions in WindowsPreviewPerTab.jsm
Categories
(SeaMonkey :: Tabbed Browser, defect)
SeaMonkey
Tabbed Browser
Tracking
(seamonkey2.21 fixed, seamonkey2.22 fixed, seamonkey2.23 fixed)
RESOLVED
FIXED
seamonkey2.23
People
(Reporter: neil, Assigned: neil)
References
Details
Attachments
(1 file)
2.91 KB,
patch
|
philip.chee
:
review+
iannbugzilla
:
approval-comm-aurora+
Callek
:
approval-comm-beta+
|
Details | Diff | Splinter Review |
When (re)opening a tab, if it is not the rightmost tab, or if the new tab is to be selected (which is always true for undo close tab), then the tab events are dispatched in the wrong order and WindowsPreviewPerTab.jsm tries to move or select tabs that it doesn't know about yet. Bug 839891 copied bug 648101's original error in sending TabOpen at the wrong time (Firefox also sends it before the tab is moved into place).
Assignee | ||
Comment 1•11 years ago
|
||
[The wording I was looking for was foreground tab.]
Comment 2•11 years ago
|
||
Comment on attachment 782176 [details] [diff] [review] Proposed patch r=me
Attachment #782176 -
Flags: review?(philip.chee) → review+
Assignee | ||
Comment 3•11 years ago
|
||
Pushed comm-central changeset a716d7cd9d9a.
Assignee | ||
Comment 4•11 years ago
|
||
Comment on attachment 782176 [details] [diff] [review] Proposed patch [Approval Request Comment] Regression caused by (bug #): 648101/839891 User impact if declined: Probably not noticably user-visible Testing completed (on m-c, etc.): Landed on c-c Risk to taking this patch (and alternatives if risky): Low String changes made by this patch: None
Attachment #782176 -
Flags: approval-comm-beta?
Attachment #782176 -
Flags: approval-comm-aurora?
Attachment #782176 -
Flags: approval-comm-aurora? → approval-comm-aurora+
Assignee | ||
Updated•11 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
status-seamonkey2.23:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.23
Assignee | ||
Comment 5•11 years ago
|
||
http://hg.mozilla.org/releases/comm-aurora/rev/9a3e6cc59ab9
Updated•11 years ago
|
Attachment #782176 -
Flags: approval-comm-beta? → approval-comm-beta+
Assignee | ||
Comment 6•11 years ago
|
||
http://hg.mozilla.org/releases/comm-beta/rev/0045d14fc33b
You need to log in
before you can comment on or make changes to this bug.
Description
•