Closed Bug 1551541 Opened 3 months ago Closed 3 months ago

HTML about:addons should show permissions prompt when enabling sideloaded extensions

Categories

(Toolkit :: Add-ons Manager, defect, P1)

68 Branch
defect

Tracking

()

RESOLVED FIXED
mozilla69
Tracking Status
firefox68 --- fixed
firefox69 --- fixed

People

(Reporter: rpl, Assigned: rpl)

References

Details

Attachments

(1 file)

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: nobody → lgreco
Blocks: 1551490
Status: NEW → ASSIGNED
Priority: -- → P1
Summary: HTML about:addons should show permission prompt when enabling sideloaded extensions → HTML about:addons should show permissions prompt when enabling sideloaded extensions
Pushed by luca.greco@alcacoop.it:
https://hg.mozilla.org/integration/autoland/rev/1978cee5e963
HTML about:addons should show permissions prompt when enabling sideloaded extensions. r=mstriemer
Status: ASSIGNED → RESOLVED
Closed: 3 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla69

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
Attachment #9064759 - Flags: approval-mozilla-beta?

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

Attachment #9064759 - Flags: approval-mozilla-beta? → approval-mozilla-beta+

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-"?

Flags: needinfo?(lgreco)
Flags: needinfo?(lgreco) → qe-verify-
You need to log in before you can comment on or make changes to this bug.