Closed
Bug 1193196
Opened 9 years ago
Closed 9 years ago
Setting Firefox as default on Windows 10 can sometimes open a windows dialog instead of the defaults app.
Categories
(Firefox :: Tours, defect)
Tracking
()
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
firefox40 | + | fixed |
firefox41 | --- | unaffected |
People
(Reporter: agibson, Assigned: jaws)
References
Details
(Whiteboard: [fixed by uplifting bug 1172724 and bug 1179249])
Attachments
(2 obsolete files)
In Bug 1175293 we added the ability to set Firefox as the default browser via UITour. We then built a welcome page for Win10 users in Bug 1186485, which makes it easier for the user to set Firefox as their default browser.
When clicking the "Let's do it" CTA button on the welcome page, one of two things happens:
a.) Firefox is set as the default browser in a single click, if possible.
b.) Firefox opens the Win10 default settings app, where the user can set Firefox as the default browser themselves.
During QA we noted that a third scenario can also sometimes occur:
Clicking the "Let's do it" button brings up a Windows message "To change your default apps, go to Settings > System > Default apps [OK]", instead of going to the system file. [1]
[1] https://bugzilla.mozilla.org/show_bug.cgi?id=1186485#c26
I'm not clear on when this scenario happens or it's already a known issue, however I felt we should still open a bug for this. Given the copy on the page I think this can add to some confusion for users.
Both myself and rbillings have seen this message occur. It definitely occurs sometimes for a), not sure about b).
Reporter | ||
Updated•9 years ago
|
Version: 41 Branch → 40 Branch
Reporter | ||
Updated•9 years ago
|
Summary: Setting Firefox as default can sometimes open a windows dialog instead of the defaults app. → Setting Firefox as default on Windows 10 can sometimes open a windows dialog instead of the defaults app.
Comment 1•9 years ago
|
||
NI jaws - any thoughts?
Got another piece of input from verdi: "it happened on a new install on a VM and it happened with a new profile on my win10 machine"
Flags: needinfo?(jaws)
Assignee | ||
Comment 2•9 years ago
|
||
If Windows encounters an error when opening the Settings application we fall back to showing this dialog. We aren't tracking the error state that is returned from Windows when this happens, but we could track it with telemetry.
I've seen this once and it was because the Settings app was hung on my machine. Perhaps Windows times out when trying to open the Settings app and returns an error code causing us to show this dialog.
Flags: needinfo?(jaws)
Comment 3•9 years ago
|
||
FWIW, on both my windows 10 machine and on my VM, I tested the Firefox modal and it works as expected. I didn't change any setting and closed the settings app. Then I clicked the "let's do it" button and instantly get the windows modal and it seems to happen no matter what I do, including restarting and restoring my session.
Comment 4•9 years ago
|
||
Questions:
1. Is there a way to understand the reach of this bug? E.g. what percentage of Windows 10 users are encountering it.
2. Is there any way the web can know that the Windows dialogue was popped up instead of the settings app?
Comment 5•9 years ago
|
||
(In reply to Cory Price [:ckprice] from comment #4)
> 2. Is there any way the web can know that the Windows dialogue was popped up
> instead of the settings app?
And if so, can we trigger a GA event for this?
Reporter | ||
Comment 6•9 years ago
|
||
(In reply to Cory Price [:ckprice] from comment #4)
> 2. Is there any way the web can know that the Windows dialogue was popped up
> instead of the settings app?
Not as far as I'm aware - as soon as we handle the button click on the page we hand off to Firefox to do the rest.
Comment 7•9 years ago
|
||
NI jaws - is there anything that can be done on the product side to fix this mid release?
Also, is there a way to understand the reach of this bug? E.g. what percentage of Windows 10 users are encountering it.
Flags: needinfo?(jaws)
Comment 8•9 years ago
|
||
Seeing this on a Windows 10 VM.
It happens all of the time when clicking the button on the Welcome page. It works as expected when I use the dialog.
Comment 9•9 years ago
|
||
I see this on both my Windows 10 machines. Curiously it happens in the Firefox 40 release build, but not Nightly.
Assignee | ||
Comment 10•9 years ago
|
||
This is because bug 1172724 wasn't uplifted to Firefox 40.
To fix this we can either uplift the patch from bug 1172724 or change the UITour.jsm code to call setDefaultBrowser with aClaimAllTypes=false.
See http://screencast.com/t/QbKeMWLdKuU for a comparison between the mozilla-release source code and mozilla-central.
Assignee: nobody → jaws
Status: NEW → ASSIGNED
Flags: needinfo?(jaws)
Assignee | ||
Comment 11•9 years ago
|
||
Follow-up to comment #10, this is going to be broken for *all* users on Release and will work for people trying with Firefox 41 and later.
The quickest thing we could do is update the webpage to not use this UITour functionality but still include the step-by-step instructions.
Assignee | ||
Comment 12•9 years ago
|
||
The screenshot in comment #10 is wrong when it says that the Preferences calls setDefaultBrowser with (true, false).
In mozilla-release, it calls it with (false, false), but in mozilla-central it calls it with (true, false).
Assignee | ||
Comment 13•9 years ago
|
||
Try push of bug 1172724 and bug 1179249 for mozilla-release,
https://treeherder.mozilla.org/#/jobs?repo=try&revision=68ad76ae36c4
Comment 14•9 years ago
|
||
(In reply to Jared Wein [:jaws] (please needinfo? me) from comment #13)
> Try push of bug 1172724 and bug 1179249 for mozilla-release,
> https://treeherder.mozilla.org/#/jobs?repo=try&revision=68ad76ae36c4
Thanks Jared. :dolske - what are the next steps for getting this into release?
Flags: needinfo?(dolske)
Comment 15•9 years ago
|
||
I've spoken with dolske. We're going to land the required patches and spin 40.0.1 tonight with the intention of pushing it tomorrow. Automatic updates are currently disabled for Windows instances.
jaws - Can you please comment with steps for how SV can verify the fix?
Flags: needinfo?(dolske) → needinfo?(jaws)
Assignee | ||
Comment 16•9 years ago
|
||
Approval Request Comment
[Feature/regressing bug #]: bug 1172724 should have been uplifted to 40
[User impact if declined]: uitour default browser ability broken
[Describe test coverage new/current, TreeHerder]: manual testing
[Risks and why]: low risk, same patch has been on 41 and 42
[String/UUID change made/needed]: none
Flags: needinfo?(jaws)
Attachment #8646671 -
Flags: approval-mozilla-release?
Assignee | ||
Comment 17•9 years ago
|
||
Attachment #8646672 -
Flags: approval-mozilla-release?
Assignee | ||
Comment 18•9 years ago
|
||
Steps to verify the fix:
Using a build that has these patches (either the builds located at http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/jwein@mozilla.com-68ad76ae36c4/try-win32/ or official Release builds once this has landed):
1. Use Windows 10
2. Make sure that Firefox is not set as the default browser on the system.
3. Use one of the builds mentioned above, visit https://www.mozilla.org/en-US/firefox/windows-10/welcome/?utm_source=firefox-browser&utm_medium=firefox-browser
4. Click on the "Lets do it" button
Expected result:
Windows' Settings app should be launched
Also, please verify that the "Make Default" button within the Preferences has the same result, as well as the dialog that appears when the browser starts up if it is not the default.
Updated•9 years ago
|
Attachment #8646671 -
Attachment is obsolete: true
Attachment #8646671 -
Flags: approval-mozilla-release?
Updated•9 years ago
|
Attachment #8646672 -
Attachment is obsolete: true
Attachment #8646672 -
Flags: approval-mozilla-release?
Comment 19•9 years ago
|
||
I took care of approving and uplifting these two patches in their original bugs (bug 1179249 and bug 1172724), to keep tracking and history a little less muddled. Already discussed and approved with release-management, and we'll be spinning Firefox 40.0.1 builds tonight.
Jared tested this locally, but we should still verify the actual 40.0.1 bits before they go out the door.
Assignee: jaws → nobody
Status: ASSIGNED → NEW
Comment 20•9 years ago
|
||
Some info for SoftVision to help verify the 40.0.1 fixes (in addition to whatever details previously existed in bug 1179249 and bug 1172724).
Ultimately, we want to make sure our 3 entry points to setting the default browser are working correctly. Case #1 was broken on Windows 10. It would be good to verify that it's now working with 40.0.1, that the other cases are also still working on Windows 10, and that the 3 cases did not regress on previous versions of Windows (8.1 and Win7 would be the two most interesting to check). Other operating systems (OS X, Linux) should be unaffected.
I'm not expecting risk of breaking anything (since 40.0.1 will just bring 40 up to date with what's already been in 41+). But I'll be overly cautious in describing what's relevant to consider checking:
1) The Windows 10 welcome page
https://www.mozilla.org/en-US/firefox/windows-10/welcome/?utm_source=firefox-browser&utm_medium=firefox-browser
(The browser changes the "en-US" part to the user's actual locale, but in general the default browser stuff is locale-independent.)
On Windows 10: Load the page, click the "Let's do it >>" button. This should trigger the Windows 10 settings UI (similar to the screenshot in the page). The bug in 40.0 was that only a small "To change your default apps, go to Settings > System > Default apps [OK]" dialog was shown.
Users on previous versions of Windows shouldn't ever see this page, so no need to test it there.
Oh, and note that if you've previously set Firefox as your default with a given profile, this UI may be skipped entirely, and Firefox will set itself as the default without the OS UI being shown.
2) Prompt upon starting the browser
The usual prompt that's shown when starting the browser, when Firefox is not the default. "Firefox is not currently set as you default browser. Would you like to make it your default browser? [Use Firefox as my default browser] [Not now]". Clicking "Use Firefox as my default" should trigger a flow as above on Windows 10.
On Windows 7 and below clicking this should make Firefox the default without further UI. Windows 8 / 8.1 is a little complicated, either getting UI like http://i.imgur.com/geXwiVC.png as shown in https://support.mozilla.org/en-US/kb/make-firefox-your-default-browser#firefox:win8:fx40
3) Options -> General -> "Make Default" button (enabled when Firefox is not the default).
Basically the same as #2. I would expect, with high confidence, that if #2 works this will as well.
Updated•9 years ago
|
Comment 21•9 years ago
|
||
I confirm that under Windows 10 x86, after launching Firefox 40.0.1RC build 2 (Build ID: 20150811223153) with a clean profile and selecting to make it as default browser, the Windows 10 Settings UI is properly triggered in all the 3 cases mentioned in comment 20.
*Please note* that the Windows 10 Settings UI is opened *only* at 1st start-up with a clean profile; if the default browser is changed again (to Chrome/Edge), the Windows 10 Settings UI is no longer displayed for the same Firefox session - the change is made automatically. Justin, is this an expected behavior?
Also, under Windows 7 x64 and Windows 8.1 x64, setting Firefox as default browser works from both the prompt box as well as from about:preferences.
Flags: needinfo?(dolske)
Comment 22•9 years ago
|
||
I also verified on Windows 10 following set up from Comment 18 and verification from Comment 20.
Assignee | ||
Comment 23•9 years ago
|
||
(In reply to Alexandra Lucinet, QA Mentor [:adalucinet] from comment #21)
> *Please note* that the Windows 10 Settings UI is opened *only* at 1st
> start-up with a clean profile; if the default browser is changed again (to
> Chrome/Edge), the Windows 10 Settings UI is no longer displayed for the same
> Firefox session - the change is made automatically. Justin, is this an
> expected behavior?
I believe what you're seeing here comes from Windows 10, which shows a "default application" dialog for new apps when they are installed, but only on their first run. We don't know of a way to retrigger this dialog on subsequent runs.
Comment 24•9 years ago
|
||
(In reply to Alexandra Lucinet, QA Mentor [:adalucinet] from comment #21)
> *Please note* that the Windows 10 Settings UI is opened *only* at 1st
> start-up with a clean profile; if the default browser is changed again (to
> Chrome/Edge), the Windows 10 Settings UI is no longer displayed for the same
> Firefox session - the change is made automatically. Justin, is this an
> expected behavior?
Yes, this seems expected.
From comment 20:
> Oh, and note that if you've previously set Firefox as your default with a
> given profile, this UI may be skipped entirely, and Firefox will set itself
> as the default without the OS UI being shown.
This is the change from bug 1184508. The OS UI is still mandatory to set Firefox as the default the first time, but we cache the resulting registry entries (in your profile), and can use those to restore Firefox as the default without going through the native UI.
(In reply to Jared Wein [:jaws] (please needinfo? me) from comment #23)
> I believe what you're seeing here comes from Windows 10, which shows a
> "default application" dialog for new apps when they are installed, but only
> on their first run. We don't know of a way to retrigger this dialog on
> subsequent runs.
Aroo? I'm not familiar with this dialog.
Flags: needinfo?(dolske)
Assignee | ||
Comment 25•9 years ago
|
||
(In reply to Justin Dolske [:Dolske] from comment #24)
> Aroo? I'm not familiar with this dialog.
I don't have a screenshot of the browser-specific one at hand and I haven't figured out how to reproduce it (you gave a good idea of creating a new user account), but this is what it roughly looks like:
http://i.kinja-img.com/gawker-media/image/upload/s--mHY485Cq--/c_fit,fl_progressive,q_80,w_636/1362003383437518225.jpg
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → jaws
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Whiteboard: [fixed by uplifting bug 1172724 and bug 1179249]
You need to log in
before you can comment on or make changes to this bug.
Description
•