Closed Bug 1502857 Opened 6 years ago Closed 6 years ago

Refactor loading tools in Inspector sidebar

Categories

(DevTools :: Inspector, enhancement, P3)

enhancement

Tracking

(firefox65 fixed)

RESOLVED FIXED
Firefox 65
Tracking Status
firefox65 --- fixed

People

(Reporter: rcaliman, Assigned: rcaliman)

References

Details

Attachments

(1 file)

- removes duplication of logic to create each panel.
- removes needless exposing of panel instances as properties on `inspector`:
  - `inspector.layoutview`
  - `inspector.fontinspector`
  - `inspector.animationinspector`
  - `inspector.changesview`
- updates tests to not rely on those exposed properties and instead call `inspector.getPanel(toolId)` (previously created panels are stored and a reference is returned).
- consolidates panel `destroy()` so we don't have to remember to destroy them individually.
- removes duplication of logic to create each panel.
- removes needless exposing of panel instances as properties on `inspector`:
  - `inspector.layoutview`
  - `inspector.fontinspector`
  - `inspector.animationinspector`
  - `inspector.changesview`
- updates tests to not rely on those exposed properties and instead
call `inspector.getPanel(toolId)` (previously created panels are stored
and a reference is returned).
- consolidates panel `destroy()` so we don't have to remember to
destroy them individually.

MozReview-Commit-ID: GVkP6z7FxKt
Pushed by rcaliman@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/b832377ab7a5
Refactor loading tools in Inspector sidebar; r=gl
There was an uncaught promise rejection which triggered the failure. Fixed and pushed to try to validate:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=45cc9127c594885c4eb6546e1caa24ac64384aae

Once confirmed, I'l reattempt landing.
Flags: needinfo?(rcaliman)
Pushed by rcaliman@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/48bcca6536ed
Refactor loading tools in Inspector sidebar; r=gl
https://hg.mozilla.org/mozilla-central/rev/48bcca6536ed
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 65
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: