Closed Bug 420938 Opened 17 years ago Closed 13 years ago

ProfileMigrator's GetDefaultBrowser does not work any more on Vista

Categories

(Firefox :: Migration, defect)

2.0 Branch
x86
Windows Vista
defect
Not set
normal

Tracking

()

VERIFIED FIXED
Firefox 15
Tracking Status
firefox14 --- verified

People

(Reporter: bruno, Assigned: asaf)

References

Details

(Whiteboard: [qa+:paul.silaghi])

Attachments

(1 obsolete file)

User-Agent: Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.8.1.12) Gecko/20080227 Firefox/2.0.0.12 Flock/1.1e Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.8.1.12) Gecko/20080227 Firefox/2.0.0.12 Flock/1.1e As of January 2008, Vista does not (cannot) use HTTP\\shell\\open\\command any more and the above function does not determine correctly the default browser. I can only think that this is caused by a Windows Update. Reproducible: Always Steps to Reproduce: 1. 2. 3. Actual Results: The consequence is that Microsoft Internet Explorer is always selected in the profile migrator, even if your default browser was set to something else (e.g Opera)
Attached patch fix (obsolete) — Splinter Review
Bruno, do you want to get this patch checked in?
Yes, we have it checked in in our Flock code.
Version: unspecified → 2.0 Branch
I'm working on this code, the patch was never checked in. But now this works with Windows 7.
Assignee: nobody → mano
Status: UNCONFIRMED → ASSIGNED
Depends on: 715099
Ever confirmed: true
Depends on: 748569
Summary: nsProfileMigrator::GetDefaultBrowserMigratorKey does not work any more on Vista → ProfileMigrator's GetDefaultBrowser does not work any more on Vista
Fixed by bug 748569.
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 15
Attachment #307297 - Attachment is obsolete: true
Isn't this a dupe of bug 262326 ?
No...
The default browser is correctly selected on initial Migration Wizard on Mozilla/5.0 (Windows NT 6.0; rv:14.0) Gecko/20100101 Firefox/14.0b8. Verified fixed.
Status: RESOLVED → VERIFIED
Whiteboard: [qa+:paul.silaghi]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: