Closed
Bug 369151
Opened 18 years ago
Closed 18 years ago
Restored blank tabs can't be reopened once they're closed
Categories
(Firefox :: Session Restore, defect)
Firefox
Session Restore
Tracking
()
VERIFIED
FIXED
People
(Reporter: zeniko, Assigned: zeniko)
References
Details
(Keywords: verified1.8.1.2)
Attachments
(1 file)
1.30 KB,
patch
|
dietrich
:
review+
dveditz
:
approval1.8.1.2+
|
Details | Diff | Splinter Review |
Steps to reproduce:
1. Set Firefox to show the windows and tabs from last time at startup
2. Open a blank tab
3. Close and restart Firefox
4. Load anything in the blank tab (e.g. www.google.com)
5. Close that formerly blank tab
Expected result:
That tab is listed under History -> Recently Closed Tabs and can be reopened.
Actual result:
It's gone, sorry, but thanks for trying...
Assignee | ||
Comment 1•18 years ago
|
||
Assignee | ||
Comment 2•18 years ago
|
||
Drivers: The patch is low-risk and would prevent a rare case of dataloss: this bug will mostly bite people having either about:blank as the homepage or having blank tabs open when restarting during an extension installation.
Flags: blocking1.8.1.2?
Comment 3•18 years ago
|
||
Comment on attachment 253806 [details] [diff] [review]
be somewhat more realistic about what a blank tab looks like
Looks good. Does this also fix bug 367052?
Attachment #253806 -
Flags: review?(dietrich) → review+
Assignee | ||
Comment 4•18 years ago
|
||
It doesn't fix the whole other bug, just one of the symptoms (see bug 367052 comment #2).
Whiteboard: [checkin needed]
Assignee | ||
Comment 5•18 years ago
|
||
Comment on attachment 253806 [details] [diff] [review]
be somewhat more realistic about what a blank tab looks like
Drivers: Low-risk one-liner (see comment #2). Please move the flag over to approval1.8.1.3 should it be too late for the 1.8.1.2 release.
Attachment #253806 -
Flags: approval1.8.1.2?
Comment 6•18 years ago
|
||
Comment on attachment 253806 [details] [diff] [review]
be somewhat more realistic about what a blank tab looks like
approved for 1.8, but this isn't a blocker and needs to land today if it's going to make it in.
a=dveditz
Attachment #253806 -
Flags: approval1.8.1.2? → approval1.8.1.2+
Updated•18 years ago
|
Flags: blocking1.8.1.2? → wanted1.8.1.x+
Assignee | ||
Comment 7•18 years ago
|
||
Dietrich or Gavin: thanks for a speedy landing.
Whiteboard: [checkin needed] → [checkin needed][checkin needed (1.8 branch)]
Comment 8•18 years ago
|
||
mozilla/browser/components/sessionstore/src/nsSessionStore.js 1.5.2.44
mozilla/browser/components/sessionstore/src/nsSessionStore.js 1.59
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Keywords: fixed1.8.1.2
Resolution: --- → FIXED
Whiteboard: [checkin needed][checkin needed (1.8 branch)]
Comment 9•18 years ago
|
||
verified using Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.8.1.2pre) Gecko/2007020604 BonEcho/2.0.0.2pre. Followed Simon's steps, and I get the expected results. Adding verified keyword.
Keywords: fixed1.8.1.2 → verified1.8.1.2
Comment 11•16 years ago
|
||
Verified with Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.2a1pre) Gecko/20090617 Minefield/3.6a1pre ID:20090617031528
Status: RESOLVED → VERIFIED
Flags: in-litmus?
Comment 12•16 years ago
|
||
Flags: in-litmus? → in-litmus+
You need to log in
before you can comment on or make changes to this bug.
Description
•