Closed
Bug 509625
Opened 15 years ago
Closed 15 years ago
nsSessionStore.js should deal with browsers rather than tabpanels
Categories
(Firefox :: Session Restore, defect)
Firefox
Session Restore
Tracking
()
RESOLVED
FIXED
Firefox 3.7a1
| Tracking | Status | |
|---|---|---|
| status1.9.2 | --- | beta1-fixed |
People
(Reporter: dao, Assigned: dao)
References
Details
(Keywords: perf, Whiteboard: [in-litmus-bug-week])
Attachments
(1 file)
|
15.29 KB,
patch
|
zeniko
:
review+
beltzner
:
approval1.9.2+
|
Details | Diff | Splinter Review |
Thinking about bug 508819, there doesn't seem to be a reason to use tabpanels at all.
Attachment #393680 -
Flags: review?(dietrich)
|
Assignee | |
Updated•15 years ago
|
Attachment #393680 -
Flags: review?(dietrich) → review?(zeniko)
|
||
Comment 1•15 years ago
|
||
Comment on attachment 393680 [details] [diff] [review]
patch
IIRC, this was needed for the original Crash Recovery component to work under Seamonkey 1.0 back in 2005. Had I bothered to actually comment my code, we might find even a few more of these unneeded hacks... Anyway, thanks for the clean-up!
Attachment #393680 -
Flags: review?(zeniko) → review+
|
|
Assignee | |
Comment 2•15 years ago
|
||
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3.7
|
|
Assignee | |
Updated•15 years ago
|
Attachment #393680 -
Flags: approval1.9.2?
|
|
Assignee | |
Updated•15 years ago
|
Target Milestone: Firefox 3.7 → Firefox 3.7a1
|
||
Comment 3•15 years ago
|
||
Comment on attachment 393680 [details] [diff] [review]
patch
a192=beltzner
Attachment #393680 -
Flags: approval1.9.2? → approval1.9.2+
|
|
Assignee | |
Updated•15 years ago
|
Keywords: checkin-needed
|
|
Assignee | |
Comment 4•15 years ago
|
||
status1.9.2:
--- → beta1-fixed
Keywords: checkin-needed
|
||
Comment 6•14 years ago
|
||
Not clear how to accurately reproduce this issue. Can someone please provide
steps to reproduce for a manual testcase?
|
|
Assignee | |
Comment 7•14 years ago
|
||
Various automated tests cover this code.
Flags: in-testsuite+
Flags: in-litmus-
|
|
||
Updated•14 years ago
|
Whiteboard: [in-litmus-bug-week]
You need to log in
before you can comment on or make changes to this bug.
Description
•