Closed Bug 369451 Opened 18 years ago Closed 18 years ago

When setting default browser first clean up HKCU keys

Categories

(Firefox :: Shell Integration, defect)

x86
Windows XP
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 2

People

(Reporter: robert.strong.bugs, Assigned: robert.strong.bugs)

References

Details

(Keywords: verified1.8.1.2)

Attachments

(1 file)

We need this to properly set the default app when it was set under HKCU previously and the user is given admin rights.
this fixed on the branch, but not trunk yet. robert is forward porting the fix to the trunk
Keywords: fixed1.8.1.2
Target Milestone: --- → Firefox 2
Attached patch patch (deleted) — Splinter Review
Attachment #254133 - Flags: review?(sspitzer)
Comment on attachment 254133 [details] [diff] [review] patch r=sspitzer
Attachment #254133 - Flags: review?(sspitzer) → review+
I'll land this on the trunk after it re-opens
Blocks: 369465
No longer blocks: 352420
Checked in to trunk
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
rstrong/seth: I was trying to verify this bug, but could not get anything to create keys in HKCU. What would normally create the keys there that we want to clean up?
For the purposes of verification you could set them manually. Alternatively, on WinXP as admin you could set IE as the default browser, as a regular user set Firefox as the default browser (this should create the keys under HKCU), grant the regular user admin rights, change the default value under HKCU Software\Classes\http\shell\open\command, and then set Firefox as the default browser.
btw: this will only remove the values which is the minimum necessary. We will add the ability to remove empty keys as part of bug 369770.
(In reply to comment #8) > btw: this will only remove the values which is the minimum necessary. We will > add the ability to remove empty keys as part of bug 369770. > In that case, I already verified this by adding my own keys and values as you suggested, where the values were removed when setting Firefox to default. I just got confused when the keys remained (and were even taken over by IE if I switched the default back). v.fixed with Firefox 2.0.0.2 rc2.
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: