1694090 - Dont await for startListening on each loop iteration in ResourceWatcher#_onTargetAvailable

Status: RESOLVED FIXED

(DevTools :: Framework, task)

Description

[Nicolas Chevobbe [:nchevobbe]]

When doing a target switch, on the resource watcher we want to re-watch the resources we were listening for, by calling _startListening: https://searchfox.org/mozilla-central/rev/c8ce16e4299a3afd560320d8d094556f2b5504cd/devtools/shared/resources/resource-watcher.js#329-331

Here we're awaiting on each loop iteration, which can cause some delay and isn't necessary.

Comment 1

[Nicolas Chevobbe [:nchevobbe]]

Attachment: Bug 1694090 - [devtools] Don't await for startListening on each loop iteration in ResourceWatcher#_onTargetAvailable. r=ladybenko,jdescottes.

This wasn't needed and could introduce unnecessary delays, so we do all the calls
at the same time and then await for all the calls to be settled.

Depends on D105943

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/dc7d09777e85
[devtools] Don't await for startListening on each loop iteration in ResourceWatcher#_onTargetAvailable. r=jdescottes.

Comment 3

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/dc7d09777e85