1192926 - Hide system add-ons from the add-ons manager

Status: RESOLVED FIXED

(Toolkit :: Add-ons Manager, defect)

Description

[Dave Townsend [:mossop]]

We don't want these to show in the main UI.

Comment 1

[Dave Townsend [:mossop]]

Attachment: MozReview Request: Bug 1192926: Hide system add-ons from the add-ons manager. r?rhelmer

Bug 1192926: Hide system add-ons from the add-ons manager. r?rhelmer

We've wanted a hidden property for add-ons for a while so we can do things like
hide "uninstalled" sideloaded add-ons so this adds the basic version of it to
hide system add-ons.

Comment 2

[Robert Helmer [:rhelmer]]

Comment on attachment 8665570 [details]
MozReview Request: Bug 1192926: Hide system add-ons from the add-ons manager. r?rhelmer

https://reviewboard.mozilla.org/r/20259/#review18343

::: toolkit/mozapps/extensions/internal/XPIProvider.jsm:6810
(Diff revision 1)
> +  this.__defineGetter__("hidden", function AddonWrapper_hidden() {

I hadn't seen this way of definine a getter so looked into it, https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object/__defineGetter__ says it is deprecated in favor of `defineProperty`.

I see `__defineGetter__` is already used in this file and you're probably already aware of this, just noticed :)

Comment 3

[Dave Townsend [:mossop]]

(In reply to Robert Helmer [:rhelmer] from comment #2)
> Comment on attachment 8665570 [details]
> MozReview Request: Bug 1192926: Hide system add-ons from the add-ons
> manager. r?rhelmer
> 
> https://reviewboard.mozilla.org/r/20259/#review18343
> 
> ::: toolkit/mozapps/extensions/internal/XPIProvider.jsm:6810
> (Diff revision 1)
> > +  this.__defineGetter__("hidden", function AddonWrapper_hidden() {
> 
> I hadn't seen this way of definine a getter so looked into it,
> https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/
> Global_Objects/Object/__defineGetter__ says it is deprecated in favor of
> `defineProperty`.
> 
> I see `__defineGetter__` is already used in this file and you're probably
> already aware of this, just noticed :)

Yeah as I was looking over this I was thinking it would be wonderful to replace all that gunk with the nicer new JS features we have now. I just filed bug 1208631 on it.

Comment 4

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/cc22ed2cafd3

Comment 5

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/4093d5840098

Comment 6

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/cc22ed2cafd3
https://hg.mozilla.org/mozilla-central/rev/4093d5840098