Closed Bug 1676808 Opened 2 years ago Closed 2 years ago

Expose WatcherFront via TargetList (instead of ResourceWatcher)

Categories

(DevTools :: Framework, enhancement)

enhancement

Tracking

(firefox85 fixed)

RESOLVED FIXED
85 Branch
Tracking Status
firefox85 --- fixed

People

(Reporter: ochameau, Assigned: ochameau)

References

Details

Attachments

(3 files)

Currently, ResourceWatcher exposes the WatcherFront via a watcher attribute.
https://searchfox.org/mozilla-central/rev/50215d649d4854812837f1343e8f47bd998dacb5/devtools/shared/resources/resource-watcher.js#111

But it appears that TargetList also uses the watcher front:
https://searchfox.org/mozilla-central/rev/50215d649d4854812837f1343e8f47bd998dacb5/devtools/shared/resources/target-list.js#288
and we would benefit from exposing it from TargetList as this is a sub-component of ResourceWatcher.

Pushed by apoirot@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/03abab648794
[devtools] Expose WatcherFront via the TargetList instead of ResourceWatcher. r=jdescottes
https://hg.mozilla.org/integration/autoland/rev/9e26cb0f0bdf
[devtools] Rename ResourceWatcher.hasWatcherSupport to hasResourceWatcherSupport. r=jdescottes
https://hg.mozilla.org/integration/autoland/rev/bc9b1ba331c3
[devtools] Rename all watcher to watcherFront. r=jdescottes
Pushed by apoirot@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/575f1754a9ce
[devtools] Expose WatcherFront via the TargetList instead of ResourceWatcher. r=jdescottes
https://hg.mozilla.org/integration/autoland/rev/16d35d5bf7f2
[devtools] Rename ResourceWatcher.hasWatcherSupport to hasResourceWatcherSupport. r=jdescottes
https://hg.mozilla.org/integration/autoland/rev/6dba1920e0cc
[devtools] Rename all watcher to watcherFront. r=jdescottes
Flags: needinfo?(poirot.alex)
You need to log in before you can comment on or make changes to this bug.