1959241 - [wpt-sync] Sync PR 51902 - [DIP] Fix flaky test

Status: RESOLVED FIXED

(Core :: CSS Parsing and Computation, task, P4)

Description

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

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

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

Camille Lamy <[e-mail]> wrote:

[DIP] Fix flaky test

This CL updates the DocumentIsolationPolicy shared-workers WPT test,
that was flaky. The test was doing several calls for workers in parallel
with different configurations, and sometimes one would resolve before
the other, with a different result from the baseline. It was
unnecessary to load all those workers in parallel, so now each subset
only loads one worker. In addition, the worker is created from a remote
executor context, which allows to experiment with different values of
DIP for the worker creator. The previous alternative created an iframe
and reached into its DOM directly to create the worker, but this doesn't
work if the iframe and the main frame of the test have different DIP.
Finally, we removed tests that tried to create workers from FileSystem
URLs because SharedWorkers cannot be loaded from FileSystem URLs.

Bug: 333029146
Change-Id: Iaf74a8efb1268d7b44424beda9531b099b62f179
Reviewed-on: https://chromium-review.googlesource.com/6440923
WPT-Export-Revision: 9b816a6281b2295b1f5c93957564acd53003fabc

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=ab682b63afdfbb58a7b9a58971fecfb1e26c063b

Comment 2

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

CI Results

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

Total 1 tests and 8 subtests

Status Summary

Firefox

OK : 1
PASS: 4
FAIL: 4

Chrome

OK : 1
PASS: 5
FAIL: 3

Safari

OK : 1
PASS: 4
FAIL: 4

Links

GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

• /html/document-isolation-policy/shared-workers.tentative.https.html [wpt.fyi]
  • "isolate-and-require-corp" (derived from response): FAIL (Chrome: PASS, Safari: FAIL)
  • isolate-and-require-corp (derived from blob URL creator): FAIL (Chrome: FAIL, Safari: FAIL)
  • "isolate-and-require-corp" (derived from owner set due to use of local scheme - blob URL): FAIL (Chrome: FAIL, Safari: FAIL)
  • "isolate-and-require-corp" (derived from owner set due to use of local scheme - data URL): FAIL (Chrome: FAIL, Safari: FAIL)

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/ac66545082f3
[wpt PR 51902] - [DIP] Fix flaky test, a=testonly

Comment 4

[Cosmin Sabou [:CosminS]]

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