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)

2.0 Branch
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 2

People

(Reporter: maxxmozilla, Assigned: asqueella)

References

Details

(Keywords: fixed1.8.1, regression)

Attachments

(1 file)

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.
Confirmed, I see the same.
Blocks: 340898
Status: UNCONFIRMED → NEW
Ever confirmed: true
It's not a regression from bug 340898, but rather from bug 318639, some weird usage of string bundle APIs there.
Blocks: 318639
No longer blocks: 340898
Severity: minor → normal
Keywords: regression
Flags: blocking-firefox2?
Attached patch patch (deleted) — Splinter Review
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)
Target Milestone: --- → Firefox 2
Version: unspecified → 2.0 Branch
Flags: blocking-firefox2? → blocking-firefox2+
Comment on attachment 235668 [details] [diff] [review] patch This fixed the problem described. r=me
Attachment #235668 - Flags: review?(dietrich) → review+
Assignee: nobody → asqueella
Whiteboard: [checkin needed]
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]
Comment on attachment 235668 [details] [diff] [review] patch a=beltzner for the 181drivers
Attachment #235668 - Flags: approval1.8.1? → approval1.8.1+
Whiteboard: [checkin needed (1.8 branch)]
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.

Attachment

General

Created:
Updated:
Size: