Closed Bug 1344155 Opened 3 years ago Closed 3 years ago

Remove toolbox in netmonitor

Categories

(DevTools :: Netmonitor, enhancement, P1)

enhancement

Tracking

(firefox54 fixed)

RESOLVED FIXED
Firefox 54
Iteration:
54.3 - Mar 6
Tracking Status
firefox54 --- fixed

People

(Reporter: rickychien, Assigned: rickychien)

References

(Blocks 1 open bug)

Details

(Whiteboard: [netmonitor-reserve])

Attachments

(1 file)

Remove toolbox in netmonitor, so that our codebase don't have to depend on it anymore.

* Remove toolbox in tabbar.js
* Remove all toolbox access in mochitest
Move small refactoring patch from bug 1308441 and remove toolbox as well.
Iteration: --- → 54.3 - Mar 6
Flags: qe-verify?
Priority: -- → P1
Whiteboard: [netmonitor][triage] → [netmonitor-reserve]
Comment on attachment 8843255 [details]
Bug 1344155 - Remove toolbox in netmonitor

https://reviewboard.mozilla.org/r/117050/#review118654

Nice work here!

R+ assuming try is green.

Honza
Attachment #8843255 - Flags: review?(odvarko) → review+
Pushed by rchien@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/0fc0b762b40a
Remove toolbox in netmonitor r=Honza
https://hg.mozilla.org/mozilla-central/rev/0fc0b762b40a
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 54
Flags: qe-verify? → qe-verify-
Depends on: 1392873
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.