1615236 - Parallelize target creation in the target list helper

Status: RESOLVED FIXED

(DevTools :: Framework, task, P3)

Description

[Julian Descottes [:jdescottes]]

Follow up to Bug 1605763

Today the target list is using for () { await } in several spots, and we are adding a few more. When connecting to many targets this means we will need to wait for each target to sequentially initialize, and each consumer callback to be executed.

I tested the current stack from Bug 1605763 with the MBT against a session of 40 tabs (on rather heavy pages), and measured the time to open the MBT + display the Console (with messages rendered)

• without the patches here, it takes 13 seconds
• with the patches, it takes 11 seconds
• with a patch to use Promise.all in target-list, it takes 3 seconds (although it was without waiting for the onAvailable callbacks, which was added as a requirement in Bug 1605763)

Comment 1

[Julian Descottes [:jdescottes]]

Attachment: Bug 1615236 - Parallelize target creation in the target list helper

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/68f4d9219d3f
Parallelize target creation in the target list helper r=ochameau

Comment 3

[Razvan Maries]

https://hg.mozilla.org/mozilla-central/rev/68f4d9219d3f