Closed Bug 1380984 Opened 7 years ago Closed 7 years ago

UAC prompt shown when setting as default browser on Windows 7

Categories

(Firefox :: Installer, defect, P1)

54 Branch
All
Windows
defect

Tracking

()

VERIFIED FIXED
Firefox 57
Tracking Status
firefox-esr52 --- unaffected
firefox55 --- wontfix
firefox56 --- wontfix
firefox57 --- verified

People

(Reporter: andrej8anubis, Assigned: molly)

References

Details

(Keywords: regression)

Attachments

(1 file)

User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:53.0) Gecko/20100101 Firefox/53.0
Build ID: 20170504105526

Steps to reproduce:

I did a clean install of firefox 54.0.1 and 55 beta 8, on windows 7



Actual results:

FireFox ask me if I want to make fireFox default browser after both installations, I clicked yes and I got "User account control"!


Expected results:

The browser should become default browser without "user account control". I works fine in version 53, where no "user account control" is necessary to make fireFox a default browser for a non administrative user.
In both cases I was installing 64 bit versions
Component: Untriaged → Shell Integration
Please describe what is "User account control" if you "got" that.
Flags: needinfo?(andrej8anubis)
"user account control" is something like picture on the link:
http://contentinjection.com/wp-content/uploads/2014/02/Learn-More-with-Windows-User-Account-Control.png
It is request to "raise privileges", but if a user is not an administrator, it can not raise privileges!
Flags: needinfo?(andrej8anubis)
The request for elevation when setting as default comes from helper.exe which is part of the installer so moving component.

Matt, can you take a look at this?
Component: Shell Integration → Installer
Flags: needinfo?(mhowell)
I've reproduced this and tracked it down to a regression introduced by bug 1340568. That means everything back to 54 is affected. The fix is simple enough so I'll go ahead and patch it, but I don't think the severity of the issue is enough to merit any uplifts.
Assignee: nobody → mhowell
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Flags: needinfo?(mhowell)
Keywords: regression
OS: Unspecified → Windows
Priority: -- → P1
Hardware: Unspecified → All
Version: 55 Branch → 54 Branch
Comment on attachment 8905578 [details]
Bug 1380984 - Fix a failure to detect that an install is registered as a possible default browser on Windows 7.

https://reviewboard.mozilla.org/r/177384/#review182556
Attachment #8905578 - Flags: review?(agashlin) → review+
Pushed by mhowell@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/fc717f341681
Fix a failure to detect that an install is registered as a possible default browser on Windows 7. r=agashlin
Summary: user account control → UAC prompt shown when setting as default browser on Windows 7
https://hg.mozilla.org/mozilla-central/rev/fc717f341681
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 57
See Also: → 1403462
Flags: qe-verify+
I've tested on 2 machines with Windows 7 x64, standard user (without admin privileges) and admin user and I wasn't able to reproduced the initial issue. I've tried using Firefox 53, Firefox 54.0.1 and Firefox 55 Beta 3. 

Could you please verify that the bug is fixed on Firefox 57 Beta 14 (https://archive.mozilla.org/pub/firefox/candidates/57.0b14-candidates/build1/win64/)? 

Thanks!
Flags: needinfo?(andrej8anubis)
I have updated to "Firefox 57 Beta 14" and when I click to make a firefox a default browser in a start up dialog or in options button, eater way, nothing happens. There is no UAC prompt, but also firefox is not a default browser. If I would uninstall and install I guess it would work, but that is not a good option.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
This bug is about UAC prompts. If no UAC prompt appeared, then that is not this bug. Please file a new bug instead of reopening an unrelated one.
Status: REOPENED → RESOLVED
Closed: 7 years ago7 years ago
Resolution: --- → FIXED
hombre83, can you then confirm that this bug is fixed, isn't reproduce anymore so we can close it? And will you log a new bug for the issue mentioned in comment 14?
(In reply to Camelia Badau, QA [:cbadau] from comment #16)
> hombre83, can you then confirm that this bug is fixed, isn't reproduce
> anymore so we can close it? And will you log a new bug for the issue
> mentioned in comment 14?

It is no UAC any more yes. Thank you
Flags: needinfo?(andrej8anubis)
Thank you, hombre83. 

Marking this issue as VERIFIED FIXED based on previous comments.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.