Closed Bug 1289463 Opened 8 years ago Closed 6 years ago

Split webconsole network-monitor.js file into client and server parts

Categories

(DevTools :: Console, enhancement, P3)

46 Branch
enhancement

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: bgrins, Unassigned)

References

Details

(Whiteboard: [reserve-html])

The webconsole network-monitor.js file is in shared/ but mostly it's used on both the client and server. Let's split it into two files and put them in their respective places.
Whiteboard: [devtools-html]
Flags: qe-verify?
Priority: -- → P2
Flags: qe-verify? → qe-verify-
I see this file being used on the server only, in devtools\server\actors\webconsole.js: loader.lazyRequireGetter(this, "NetworkMonitor", "devtools/shared/webconsole/network-monitor", true); loader.lazyRequireGetter(this, "NetworkMonitorChild", "devtools/shared/webconsole/network-monitor", true); loader.lazyRequireGetter(this, "ConsoleProgressListener", "devtools/shared/webconsole/network-monitor", true); loader.lazyRequireGetter(this, "StackTraceCollector", "devtools/shared/webconsole/network-monitor", true); But I couldn't find any usages in devtools\client\. Am I missing something?
Priority: P2 → P3
Whiteboard: [devtools-html] → [reserve-html]
(In reply to Patrick Brosset <:pbro> from comment #1) > But I couldn't find any usages in devtools\client\. > Am I missing something? That sounds correct to me as well. Maybe this bug can be repurposed to move network-monitor (and throttler and maybe other things) to devtools/server.
Product: Firefox → DevTools
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.