1477598 - Show/update/debug extensions

Status: RESOLVED FIXED

(DevTools :: about:debugging, enhancement, P1)

Description

[Daisuke Akatsuka (:daisuke)]

This is the splitted bug of bug 1471795.
In this, implement to show/update/debug extensions.

Comment 1

[:Harald Kirschner :digitarald]

Moving major about:debugging ng work into milestone 1, leaving m0 for prior bugfix work.

Comment 2

[Daisuke Akatsuka (:daisuke)]

Attachment: Bug 1477598 - Part 1: Implement a mechanism to dispatch extensions. r=jdescottes

Comment 3

[Daisuke Akatsuka (:daisuke)]

Attachment: Bug 1477598 - Part 2: Implement UI for extensions. r=jdescottes

Depends on D3083

Comment 4

[Daisuke Akatsuka (:daisuke)]

Attachment: Bug 1477598 - Part 3: Implement inspection action. r=jdescottes

Depends on D3084

Comment 5

[Daisuke Akatsuka (:daisuke)]

Attachment: Bug 1477598 - Part 4: Add a listener to detect updating the extensions. r=jdescottes

Depends on D3087

Comment 6

[Julian Descottes [:jdescottes]]

Comment on attachment 8999109 [details]
Bug 1477598 - Part 2: Implement UI for extensions. r=jdescottes

Julian Descottes [:jdescottes][:julian] has approved the revision.

Comment 7

[Julian Descottes [:jdescottes]]

Looks pretty good IMO! 

Some overall comments for follow up bugs:
- aboutdebugging now hides system addons by default: see https://searchfox.org/mozilla-central/search?q=devtools.aboutdebugging.showSystemAddons&case=false&regexp=false&path=
  It would be nice to have a follow up bug to do the same
- not just related to addons, but without a separator between the items, it is difficult to know which Inspect button is connected to which debug target. In the application panel we added a separator to address that. We don't have to do exactly that, but let's open a bug to discuss it
- what should happen to empty categories? open a bug to address this question.

Comment 8

[Julian Descottes [:jdescottes]]

Comment on attachment 8999108 [details]
Bug 1477598 - Part 1: Implement a mechanism to dispatch extensions. r=jdescottes

Julian Descottes [:jdescottes][:julian] has approved the revision.

Comment 9

[Julian Descottes [:jdescottes]]

Comment on attachment 8999112 [details]
Bug 1477598 - Part 3: Implement inspection action. r=jdescottes

Julian Descottes [:jdescottes][:julian] has approved the revision.

Comment 10

[Julian Descottes [:jdescottes]]

Comment on attachment 8999115 [details]
Bug 1477598 - Part 4: Add a listener to detect updating the extensions. r=jdescottes

Julian Descottes [:jdescottes][:julian] has approved the revision.

Comment 11

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/a6ccfdee5d73
Part 1: Implement a mechanism to dispatch extensions. r=jdescottes
https://hg.mozilla.org/integration/mozilla-inbound/rev/72e1e5b24aaa
Part 2: Implement UI for extensions. r=jdescottes
https://hg.mozilla.org/integration/mozilla-inbound/rev/7e20f49730d2
Part 3: Implement inspection action. r=jdescottes
https://hg.mozilla.org/integration/mozilla-inbound/rev/f4bbda249bee
Part 4: Add a listener to detect updating the extensions. r=jdescottes

Comment 12

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/a5f1f7df5521
Part 5: Remove extra css loading. r=me

Comment 13

[Bogdan Tara[:bogdan_tara | bogdant]]

https://hg.mozilla.org/mozilla-central/rev/a6ccfdee5d73
https://hg.mozilla.org/mozilla-central/rev/72e1e5b24aaa
https://hg.mozilla.org/mozilla-central/rev/7e20f49730d2
https://hg.mozilla.org/mozilla-central/rev/f4bbda249bee
https://hg.mozilla.org/mozilla-central/rev/a5f1f7df5521