Closed
Bug 350389
Opened 18 years ago
Closed 18 years ago
session restore doesn't work if no homepage is specified
Categories
(Firefox :: Settings UI, defect)
Tracking
()
RESOLVED
FIXED
Firefox 2
People
(Reporter: maxxmozilla, Assigned: asqueella)
References
Details
(Keywords: fixed1.8.1, regression)
Attachments
(1 file)
(deleted),
patch
|
dietrich
:
review+
beltzner
:
approval1.8.1+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9a1) Gecko/20060827 Minefield/3.0a1
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9a1) Gecko/20060827 Minefield/3.0a1
Session restore doesn't work if empty homepage is specified.
Reproducible: Always
Steps to Reproduce:
1. Go to Tools -> Options -> Main and in Startup section action for "When Minefield starts:" choose "Show my windows and tabs from last time"
2. Remove homepage if you have any
3. Close Firefox (make sure you have at least one tab with content loaded before doing so)
4. Start Firefox
Actual Results:
Last session isn't restored (empty window).
Expected Results:
Last session is restored per pref.
Session restore feature shouldn't be affected by having homepage set or not.
Comment 1•18 years ago
|
||
Confirmed, I see the same.
Assignee | ||
Comment 2•18 years ago
|
||
It's not a regression from bug 340898, but rather from bug 318639, some weird usage of string bundle APIs there.
Updated•18 years ago
|
Flags: blocking-firefox2?
Assignee | ||
Comment 3•18 years ago
|
||
This fixes the bogus code in http://bonsai.mozilla.org/cvsblame.cgi?file=mozilla/browser/base/content/browser.js&rev=1.696&mark=4285-4286#4273 and as a bonus appears to fix the reported bug. I didn't read the surrounding code or test very thoroughly though.
Attachment #235668 -
Flags: review?(dietrich)
Updated•18 years ago
|
Target Milestone: --- → Firefox 2
Version: unspecified → 2.0 Branch
Updated•18 years ago
|
Flags: blocking-firefox2? → blocking-firefox2+
Comment 4•18 years ago
|
||
Comment on attachment 235668 [details] [diff] [review]
patch
This fixed the problem described. r=me
Attachment #235668 -
Flags: review?(dietrich) → review+
Assignee | ||
Updated•18 years ago
|
Assignee: nobody → asqueella
Whiteboard: [checkin needed]
Comment 5•18 years ago
|
||
mozilla/browser/base/content/browser.js 1.698
Status: NEW → RESOLVED
Closed: 18 years ago
OS: Windows 2000 → All
Hardware: PC → All
Resolution: --- → FIXED
Whiteboard: [checkin needed]
Updated•18 years ago
|
Attachment #235668 -
Flags: approval1.8.1?
Comment 6•18 years ago
|
||
Comment on attachment 235668 [details] [diff] [review]
patch
a=beltzner for the 181drivers
Attachment #235668 -
Flags: approval1.8.1? → approval1.8.1+
Updated•18 years ago
|
Whiteboard: [checkin needed (1.8 branch)]
Updated•18 years ago
|
Keywords: fixed1.8.1
Whiteboard: [checkin needed (1.8 branch)]
You need to log in
before you can comment on or make changes to this bug.
Description
•