Webextension pop-up style displayed while installing legacy add-ons from about:addons search list

VERIFIED FIXED in Firefox 55

Status

()

Toolkit
Add-ons Manager
VERIFIED FIXED
10 months ago
9 months ago

People

(Reporter: Vasilica Mihasca, QA [away for an extended period of time - please needinfo addonsqa@softvision.ro], Assigned: aswan)

Tracking

({regression})

Trunk
mozilla55
regression
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(firefox52 unaffected, firefox-esr52 unaffected, firefox53 unaffected, firefox54 unaffected, firefox55+ verified)

Details

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(2 attachments)

Created attachment 8849125 [details]
2017-03-20_1701.png

[Affected versions]:
Firefox 55.0a1 (2017-03-20)
Firefox 54.0a2 (2017-03-20)


[Affected platforms]:
Windows 10 64-bit
Ubuntu 16.04 32-bit

[Steps to reproduce]:
1.Launch Firefox with clean profile.
2.Create extensions.webextPermissionPrompts and set it to true.
3.Navigate to about:addons -> Extensions tab.
4.Search for a legacy add-on using the search bar displayed in the top right corner of the page.
5.Choose one item from the list and click “Install” button.


[Expected Results]:
As was discussed in https://bugzilla.mozilla.org/show_bug.cgi?id=1335697#c19, no pop-up should be displayed for legacy add-ons while installing from about:add-ons search list.


[Actual Results]:
- Webextension pop-up, which correspond for a webextension with no permissions, appears while installing a legacy add-on from about:addons search list.
- See attached screenshot

[Regression Range]:
Last good revision: 69d2cf007cdc13cf2bd33370b1732d4f969fd9af
First bad revision: c7935d540027158ef1bf8e1c0e69af6b2eb9df93
Pushlog:
https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=69d2cf007cdc13cf2bd33370b1732d4f969fd9af&tochange=c7935d540027158ef1bf8e1c0e69af6b2eb9df93
I’m not sure which bug regressed this issue, Andrew do you have any idea?
Flags: needinfo?(aswan)
(Assignee)

Comment 2

10 months ago
The comment linked above was about the post-install notifications.  I don't think we ever talked about whether there should be a confirmation during the isntall process for legacy add-ons.  Markus, what say you?
Flags: needinfo?(aswan) → needinfo?(mjaritz)
It is strange that we decided to not do the post-install notification for legacy but no do the pre-install confirmation...
I'd rather not have that pop-up show for legacy extensions to be consistent with our previous decision.
Flags: needinfo?(mjaritz)

Comment 4

10 months ago
Is this something we need to be tracking for release?
Flags: needinfo?(amckay)

Comment 5

10 months ago
No need to track, this is a new feature.
Flags: needinfo?(amckay)
(Assignee)

Comment 6

10 months ago
This is new but we've changed the behavior by flipping the pref on by default.  I'm marking 54 as unaffected since it doesn't occur there with the default settings.

Its easy to fix though, I'll get the patch together today.  I don't think its a big deal but it probably should be tracking.

[Tracking Requested - why for this release]:
With bug 1342142 we have unintentionally hanged the flow for installs from the search page, this bug is to fix it.
Assignee: nobody → aswan
status-firefox54: affected → unaffected
tracking-firefox55: --- → ?
Tracking 55+ for the reasons in Comment 6.
tracking-firefox55: ? → +
Comment hidden (mozreview-request)
(Assignee)

Comment 9

10 months ago
The patch is simple, a test wouldn't be too much work but I'm at a work week at the moment and won't have a chance to write one for a bit.  Given the short lifetime remaining for legacy add-ons maybe we should just land this as-is?
(Assignee)

Updated

10 months ago
Attachment #8852582 - Flags: review?(rhelmer)

Comment 10

10 months ago
mozreview-review
Comment on attachment 8852582 [details]
Bug 1348854 Don't show permissions prompt for legacy addons installed from search

https://reviewboard.mozilla.org/r/124774/#review127372

I agree, we don't need to worry about testing since it's just a few releases we need to care about it.

Comment 11

10 months ago
mozreview-review
Comment on attachment 8852582 [details]
Bug 1348854 Don't show permissions prompt for legacy addons installed from search

https://reviewboard.mozilla.org/r/124774/#review127374

Comment 12

10 months ago
mozreview-review
Comment on attachment 8852582 [details]
Bug 1348854 Don't show permissions prompt for legacy addons installed from search

https://reviewboard.mozilla.org/r/124776/#review127376
Attachment #8852582 - Flags: review?(rhelmer) → review+
(Assignee)

Updated

10 months ago
Keywords: leave-open
Comment hidden (mozreview-request)
(Assignee)

Updated

10 months ago
Keywords: leave-open
Comment hidden (mozreview-request)

Comment 15

10 months ago
Pushed by aswan@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/43c03ad7e9c1
Don't show permissions prompt for legacy addons installed from search r=rhelmer

Comment 16

10 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/43c03ad7e9c1
Status: NEW → RESOLVED
Last Resolved: 10 months ago
status-firefox55: affected → fixed
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
status-firefox-esr52: --- → unaffected
Confirm that this issue is fixed on Firefox 55.0a1 (2017-04-19) under Windows 10 64-bit and Ubuntu 16.04 32-bit. The pop-up is no longer displayed while installing legacy add-ons from Add-ons Manager search list.
Status: RESOLVED → VERIFIED
status-firefox55: fixed → verified
You need to log in before you can comment on or make changes to this bug.