1856126 - [wpt-sync] Sync PR 42137 - LoAF: ensure scripts are added after microtask checkpoint

Status: RESOLVED FIXED

(Testing :: web-platform-tests, task, P4)

Description

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Sync web-platform-tests PR 42137 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/42137
Details from upstream follow.

Noam Rosenthal <[e-mail]> wrote:

LoAF: ensure scripts are added after microtask checkpoint

In most cases this already works, but there are some edge cases
where the microtask queue can be cleared after the script probe
is executed, for example in TrustedTypes.

To rectify, when adding a script, we first wait unti microtasks are
flushed if they aren't already.

Added a test that reproduces it via trusted-types.

Bug: 1485516
Change-Id: I5bb017ffc441471de65aef7c34ad1dcef159157c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4887255
Reviewed-by: Yoav Weiss \<[e-mail]>
Commit-Queue: Noam Rosenthal \<[e-mail]>
Cr-Commit-Position: refs/heads/main@{#1202659}

Comment 1

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=0a9ca7317584d1bb85da6ba8489de047646f96aa

Comment 2

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

CI Results

Ran 9 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI

Total 1 tests and 1 subtests

Status Summary

Firefox

ERROR : 1

Chrome

TIMEOUT: 2

Safari

ERROR : 1

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

• /long-animation-frame/tentative/loaf-trusted-types.html [wpt.fyi]: ERROR (Chrome: TIMEOUT, Safari: ERROR)

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/375329b97d12
[wpt PR 42137] - LoAF: ensure scripts are added after microtask checkpoint, a=testonly
https://hg.mozilla.org/integration/autoland/rev/fa409e227b5a
[wpt PR 42137] - Update wpt metadata, a=testonly

Comment 4

[Cristina Horotan [:chorotan]]

https://hg.mozilla.org/mozilla-central/rev/375329b97d12
https://hg.mozilla.org/mozilla-central/rev/fa409e227b5a