Open
Bug 1861778
Opened 1 year ago
Remove waitForMultiple test helper in dom/workers/test/dom_worker_helper.js in favor of Promise.all or sequential awaits
Categories
(Core :: DOM: Workers, task, P3)
Core
DOM: Workers
Tracking
()
NEW
People
(Reporter: asuth, Unassigned)
Details
Assuming my test changes in https://phabricator.services.mozilla.com/D191822 for bug 1828084 land and stick, we should remove the waitForMultiple helper in https://searchfox.org/mozilla-central/source/dom/workers/test/dom_worker_helper.js entirely in favor of some combination of
- using Promise.all() for multiple promises where we don't really care about their ordering
- using a more explicit idiom for waiting in sequence that's explicit with actual comments explaining why an ordering invariant is guaranteed/required.
See the PR for more context.
You need to log in
before you can comment on or make changes to this bug.
Description
•