Closed
Bug 1344155
Opened 8 years ago
Closed 8 years ago
Remove toolbox in netmonitor
Categories
(DevTools :: Netmonitor, enhancement, P1)
DevTools
Netmonitor
Tracking
(firefox54 fixed)
| 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
|
Assignee | |
Comment 1•8 years ago
|
||
Move small refactoring patch from bug 1308441 and remove toolbox as well.
| Comment hidden (mozreview-request) |
| Comment hidden (mozreview-request) |
|
||
Updated•8 years ago
|
Iteration: --- → 54.3 - Mar 6
Flags: qe-verify?
Priority: -- → P1
Whiteboard: [netmonitor][triage] → [netmonitor-reserve]
|
||
Comment 4•8 years ago
|
||
| mozreview-review | ||
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
|
||
Comment 6•8 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox54:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 54
|
|
||
Updated•8 years ago
|
Flags: qe-verify? → qe-verify-
|
||
Updated•7 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•