HTML about:addons should show permissions prompt when enabling sideloaded extensions
Categories
(Toolkit :: Add-ons Manager, defect, P1)
Tracking
()
People
(Reporter: rpl, Assigned: rpl)
References
Details
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
jcristau
:
approval-mozilla-beta+
|
Details |
When a sideloaded extension is enabled from about:addons, it should show a permission prompt (if the extension does have permissions), and the extension should be actually enabled when (and if) the user has accepted the permission prompt.
In the XUL about:addons page this behavior is implemented in the cmd_enableItem
's doCommand
method, and so we should refactor and move that logic from extensions.js into aboutaddonsCommon.js (as we did for other logic we need to share between the XUL and HMTL about:addons pages) and then ensure that aboutaddons.js also use the shared logic to show the permissions prompt in the above scenario.
This behavior is tested by browser_extension_sideloading.js
, which is one of the tests that fails when the HTML about:addons page is enabled by default (Bug 1551490),
and so we should be sure that it completes successfully with the HTML about:addons page enabled by default as part of this issue.
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Comment 1•5 years ago
|
||
Comment 3•5 years ago
|
||
bugherder |
Comment 4•5 years ago
|
||
Comment on attachment 9064759 [details]
Bug 1551541 - HTML about:addons should show permissions prompt when enabling sideloaded extensions. r?mstriemer!
Beta/Release Uplift Approval Request
- User impact if declined: Users are not prompted for extension permissions when they enable a sideloaded add-on via the HTML
about:addons
page. Consequently they may enable a sideloaded add-on without being informed about its (powerful) capabilities. - 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: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): Patch consists of UI logic (reusing existing logic) and is well-covered by unit tests. The patch has baked for a week without any reported regressions or intermittent failures.
- String changes made/needed: none
Comment 5•5 years ago
|
||
Comment on attachment 9064759 [details]
Bug 1551541 - HTML about:addons should show permissions prompt when enabling sideloaded extensions. r?mstriemer!
fix for html about:addons, approved for 68.0b8
Comment 6•5 years ago
|
||
bugherder uplift |
Comment 7•5 years ago
|
||
In the uplift request there is stated that there is no need of manual qa. If that is true can you please mark this as "qe-verify-"?
Assignee | ||
Updated•5 years ago
|
Description
•