Remove unused tool gcli commands

RESOLVED FIXED in Firefox 63

Status

enhancement
P2
normal
RESOLVED FIXED
11 months ago
7 months ago

People

(Reporter: ochameau, Assigned: yulia)

Tracking

unspecified
Firefox 63
Dependency tree / graph

Firefox Tracking Flags

(firefox63 fixed)

Details

Attachments

(1 attachment)

(Reporter)

Description

11 months ago
Another good followup to bug 1461970 would be to remove all the gcli commands implemented by each tools that are not used:
https://searchfox.org/mozilla-central/search?q=commands%3A&case=false&regexp=false&path=definitions.js

It looks like only eyedropper command, defined by:
https://searchfox.org/mozilla-central/source/devtools/client/inspector/inspector-commands.js#39-115
is still used.

Updated

10 months ago
Product: Firefox → DevTools
(Assignee)

Comment 1

8 months ago
This patch removes remaining files and traces of GCLI panel commands.
(Assignee)

Updated

8 months ago
Depends on: 1447487
(Assignee)

Comment 2

8 months ago
This patch is blocked by 1447487, as the eyedropper is migrated from GCLI there. 

This try run is to see if anything unexpected is failing:  https://treeherder.mozilla.org/#/jobs?repo=try&revision=b523f81686ebff27d0b8f13f3319f6d8bba36809
(Assignee)

Updated

8 months ago
Assignee: nobody → ystartsev
(Reporter)

Comment 3

8 months ago
Comment on attachment 9001991 [details]
Bug 1462399 - remove GCLI commands; r=ochameau

Alexandre Poirot [:ochameau] has approved the revision.
Attachment #9001991 - Flags: review+

Comment 5

8 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/253b4b3b7055
Status: NEW → RESOLVED
Last Resolved: 8 months ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 63

Updated

7 months ago
Product: DevTools → DevTools Graveyard
You need to log in before you can comment on or make changes to this bug.