Bug 229766 – [Mac] imported/w3c/web-platform-tests/html/cross-origin-opener-policy/coop-csp-sandbox-navigate.https.html is a flaky failure

• Status: RESOLVED
• Resolution: FIXED
• Component: DOM

[Eric Hutchison]

imported/w3c/web-platform-tests/html/cross-origin-opener-policy/coop-csp-sandbox-navigate.https.html

is a flaky failure on Mac.

History: https://results.webkit.org/?suite=layout-tests&test=imported%2Fw3c%2Fweb-platform-tests%2Fhtml%2Fcross-origin-opener-policy%2Fcoop-csp-sandbox-navigate.https.html&platform=mac

Results: https://build.webkit.org/results/Apple-BigSur-Debug-WK2-Tests/r281848%20(3536)/results.html

Diff:
--- /Volumes/Data/worker/bigsur-debug-tests-wk2/build/layout-test-results/imported/w3c/web-platform-tests/html/cross-origin-opener-policy/coop-csp-sandbox-navigate.https-expected.txt
+++ /Volumes/Data/worker/bigsur-debug-tests-wk2/build/layout-test-results/imported/w3c/web-platform-tests/html/cross-origin-opener-policy/coop-csp-sandbox-navigate.https-actual.txt
@@ -1,4 +1,4 @@
 
-PASS CSP: sandbox allow-popups allow-scripts allow-same-origin; CSP sandbox popup navigate to Cross-Origin-Opener-Policy document should work
-PASS CSP: sandbox allow-popups allow-scripts; CSP sandbox popup navigate to Cross-Origin-Opener-Policy document should work
+FAIL CSP: sandbox allow-popups allow-scripts allow-same-origin; CSP sandbox popup navigate to Cross-Origin-Opener-Policy document should work assert_unreached: Navigation from CSP sandbox to COOP document failed Reached unreachable code
+FAIL CSP: sandbox allow-popups allow-scripts; CSP sandbox popup navigate to Cross-Origin-Opener-Policy document should work assert_unreached: Navigation from CSP sandbox to COOP document failed Reached unreachable code

Reproduced on r281856 using run-webkit-tests- --debug --iterations 1000 -exit-after-n-failures 1 imported/w3c/web-platform-tests/html/cross-origin-opener-policy/coop-csp-sandbox-navigate.https.html

[Radar WebKit Bug Importer]

<rdar://problem/82642738>

[Eric Hutchison]

Updated test expectations at https://trac.webkit.org/changeset/281869/webkit

[Chris Dumez]

I was able to reproduce with the provided steps on ToT, thanks.

[Chris Dumez]

The test has:
```
    addEventListener('load', t.step_func(() => {
      t.step_timeout(() => {
        assert_unreached('Navigation from CSP sandbox to COOP document failed')
      }, 1500);
    }));
```

Maybe the test runs slow that the timer fires before the load has completed.

[Chris Dumez]

https://github.com/web-platform-tests/wpt/pull/30318

[Chris Dumez]

Created attachment 437269 [details]
Patch

[EWS Watchlist]

This patch modifies the imported WPT tests. Please ensure that any changes on the tests (not coming from a WPT import) are exported to WPT. Please see https://trac.webkit.org/wiki/WPTExportProcess

[EWS]

Committed r281999 (241306@main): <https://commits.webkit.org/241306@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 437269 [details].