Stop passing installed add-ons data to discovery pane

RESOLVED WONTFIX

Status

()

RESOLVED WONTFIX
2 years ago
2 years ago

People

(Reporter: mossop, Assigned: mossop)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: triaged)

MozReview Requests

()

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

Attachments

(1 attachment)

(Assignee)

Description

2 years ago
We currently pass a list of all the installed add-ons to the discovery pane. It doesn't use it anymore instead using the mozAddonManager API to get what it needs so we should stop doing this.
(Assignee)

Updated

2 years ago
Assignee: nobody → dtownsend
I filed https://github.com/mozilla/addons-frontend/issues/1840 to track this change for the disco pane.

Could we get whether the user has any themes or extensions installed instead? Eg:

{"hasExtensions": false, "hasThemes": true}
Comment hidden (mozreview-request)
(Assignee)

Updated

2 years ago
Attachment #8838281 - Flags: review?(aswan)

Comment 3

2 years ago
mozreview-review
Comment on attachment 8838281 [details]
Bug 1340312: Stop passing installed add-ons data to discovery pane.

https://reviewboard.mozilla.org/r/113230/#review114696

::: toolkit/mozapps/extensions/content/extensions.js
(Diff revision 1)
> -        this._loadURL(this.homepageURL.spec, false, notifyInitialized);
> +      this._loadURL(this.homepageURL.spec, false, notifyInitialized);
> -      else
> +    else
> -        notifyInitialized();
> +      notifyInitialized();
> -    }
> -
> -    if (Services.prefs.getBoolPref(PREF_GETADDONS_CACHE_ENABLED) == false) {

This looks like the only use of PREF_GETADDONS_CACHE_ENABLED in this file so that declaration can go too.  Same with PREF_GETADDONS_CACHE_ID_ENABLED.  I'm not familiar with those prefs, you probably know better than I do if there's further removal that can happen since these references are gone.
Attachment #8838281 - Flags: review+
(Assignee)

Comment 4

2 years ago
(In reply to Mark Striemer [:mstriemer] from comment #1)
> I filed https://github.com/mozilla/addons-frontend/issues/1840 to track this
> change for the disco pane.
> 
> Could we get whether the user has any themes or extensions installed
> instead? Eg:
> 
> {"hasExtensions": false, "hasThemes": true}

I can do this, but basically everyone will have hasThemes=true because of the default themes we ship. If you want me to somehow try to filter those out let me know.
Flags: needinfo?(mstriemer)
We manually filter out bundled add-ons with a list maintained in the disco pane code: https://github.com/mozilla/addons-frontend/commit/6331f6b8f161698f0e1e5f1e04c61b43b6467a1c#diff-8423fedb8be9cc2453639bbbe1fe40e3L13 but it's something that needs to be continually updated.
Filtering them out would be great if there's some way to be able to tell if it's a default theme or system extension in Firefox. Otherwise I guess we'll still need the GUIDs so we can use our existing filtering.
Flags: needinfo?(mstriemer)
(In reply to Mark Striemer [:mstriemer] from comment #6)
> Filtering them out would be great if there's some way to be able to tell if
> it's a default theme or system extension in Firefox. Otherwise I guess we'll
> still need the GUIDs so we can use our existing filtering.

In "about:telemetry > Environment Data > addons" extensions have an entry called isSystem: true to indicate that they are system addons. For themes I haven't seen that unfortunately, so we might still need the GUIDs for those.

Updated

2 years ago
Whiteboard: triaged
(Assignee)

Comment 8

2 years ago
This has gotten more complicated than I had originally expected. Given that we still need to do filtering somewhere I propose to leave things as they are with AMO doing the filtering server side where it can be updated without needing a Firefox update making this bug a wontfix.
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.