1807650 - Remove old "network monitor" server APIs

Status: RESOLVED DUPLICATE of bug 1721592

(DevTools :: Netmonitor, task)

Description

[Alexandre Poirot [:ochameau]]

With bug 1625937, we removed the last usage of NetworkActivity webconsole listener.
We can now remove the old NetworkMonitor and NetworkEvent actors and all its dependencies.

Comment 1

[Alexandre Poirot [:ochameau]]

Attachment: Bug 1807650 - [devtools] Remove old server codebase to listen to network events.

With the removal of the old non-fission browser toolbox,
we removed the last usage of this old codebase.
We used to be able to listen to network events via WebConsole's actor's startListeners method.
Nowadays we should rather use the Watcher actor's watchResources method and listen to
NETWORK_EVENT resource.
(or migrate to WebDriver Bidi which will soon allow to listen to network requests see bug 1790369)

This patch also removed:

• all now unused block* requests from webconsole actor,
• spawnActorInParent which was only used by this code,
• code in the frontend assuming that we still had to go through the webconsole/emulation actors to block and throttle requests,
• remove emulation actor method to handle throttling.

Comment 2

[Alexandre Poirot [:ochameau]]

Attachment: Bug 1807650 - [devtools] Remove network legacy listeners.

This required a small workaround in ResourceCommand in order to avoid
trying to spawn network event legacy listeners for workers.
The old legacy listener were bailing out for workers...

Comment 4

[Phabricator Automation]

Comment on attachment 9309980 [details]
Bug 1807650 - [devtools] Remove old server codebase to listen to network events.

Revision D165603 was moved to bug 1721592. Setting attachment 9309980 [details] to obsolete.

Comment 5

[Phabricator Automation]

Comment on attachment 9309981 [details]
Bug 1807650 - [devtools] Remove network legacy listeners.

Revision D165604 was moved to bug 1721592. Setting attachment 9309981 [details] to obsolete.