Open Bug 1687459 Opened 3 years ago Updated 1 year ago

[meta] Enable all resource watchers on the server, for all target types

Categories

(DevTools :: Framework, task)

task

Tracking

(Not tracked)

People

(Reporter: ochameau, Unassigned)

References

(Depends on 4 open bugs, Blocks 1 open bug)

Details

(Keywords: meta)

The Watcher actor contains a trait, which reports which resource is supported by the current watcher. The current watcher is bound to a given descriptor. So it can be bound to one tab, one webextension or everything (for the browser toolbox).

These traits are defined here:
https://searchfox.org/mozilla-central/rev/c54c71ecbd9e64cafc0df3b596e206ac4072cb91/devtools/server/actors/watcher.js#119-155
Today, most of them are set only for tab debugging. So only for regular toolbox debugging a tab. And typically not for the browser toolbox, or worker toolboxes.

We should enable the resource watchers on the server for all toolboxes,
so that we can get rid of:

  • legacy listeners on the client side, which is still used for watcher retporting false traits,
  • legacy server code, like WebConsoleActor.startListeners, ThreadActor.sources, ... which have been replaced by ResourceWatcher.watchResources API.
Keywords: meta
Summary: Enable all resource watchers on the server, for all target types → [meta] Enable all resource watchers on the server, for all target types
Depends on: 1657101
Depends on: 1675763
Depends on: 1687461
Depends on: 1689459

The Bugbug bot thinks this bug should belong to the 'DevTools::Framework' component, and is moving the bug to that component. Please revert this change in case you think the bot is wrong.

Component: General → Framework
Depends on: 1772816
Depends on: 1772817
Depends on: 1772820
Depends on: 1772822
Depends on: 1772823
Depends on: 1721592
No longer depends on: 1721592
Depends on: 1807804
Depends on: 1666534
You need to log in before you can comment on or make changes to this bug.