[Interventions] The extension needs a way to install an update and restart for users who have opted out of automatic updates
Categories
(Firefox :: Address Bar, task, P1)
Tracking
()
People
(Reporter: adw, Assigned: adw)
References
Details
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
Users can opt out of automatic updates. In that case, when an update is available, we need to let the user download and install it and restart the browser. This case isn't covered in the interventions spec, but it's consistent with the intention to give the user a one-click way to continue on their upgrade path, so I think we should handle it.
Assignee | ||
Comment 1•5 years ago
|
||
Users can opt out of automatic updates. In that case, when an update is available, we need to let the user download and install it and restart the browser. This case isn't covered in the interventions spec, but it's consistent with the intention to give the user a one-click way to continue on their upgrade path, so I think we should handle it.
This patch adds a browser.experiments.urlbar.installBrowserUpdateAndRestart
function. At first I made it only install the update, and then the extension called our existing restartBrowser
function once that finished. But it didn't work because restartBrowser
must be called from a user-input handler, and apparently by awaiting the install function, any code running after the await
isn't considered to be in the user-input handler.
Assignee | ||
Comment 2•5 years ago
|
||
Assignee | ||
Comment 3•5 years ago
|
||
I talked with Verdi about this, and he likes it.
Assignee | ||
Comment 4•5 years ago
|
||
Comment 6•5 years ago
|
||
bugherder |
Assignee | ||
Comment 7•5 years ago
|
||
Comment on attachment 9113122 [details]
Bug 1600860 - Add browser.experiments.urlbar.installBrowserUpdateAndRestart.
Beta/Release Uplift Approval Request
- User impact if declined: We need this in order to run the urlbar interventions experiment (bug 1564506) on 72 as planned.
- Is this code covered by automated tests?: Yes
- Has the fix been verified in Nightly?: No
- Needs manual test from QE?: No
- If yes, steps to reproduce:
- List of other uplifts needed: Bug 1601369 needed to fix intermittent created by this patch
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): This is a test-only change. There's no change to shipping code. This patch causes an intermittent failure that's fixed by bug 1601369, so bug 1601369 should also be uplifted.
- String changes made/needed:
Comment 8•5 years ago
|
||
Comment on attachment 9113122 [details]
Bug 1600860 - Add browser.experiments.urlbar.installBrowserUpdateAndRestart.
this is test-only
Updated•5 years ago
|
Comment 9•5 years ago
|
||
bugherder uplift |
Description
•