commit 14960bc360198cbb43203d5d5fb27b238a2d6d3c parent 3630c5d662109396c35c8163f6c174be801a7404 Author: Blink WPT Bot <blink-w3c-test-autoroller@chromium.org> Date: Tue, 21 Oct 2025 10:31:06 +0000 Bug 1992463 [wpt PR 55232] - Reland "Extend timeout duration in reporting-common.js", a=testonly Automatic update from web-platform-tests Reland "Extend timeout duration in reporting-common.js" (#55232) This reverts commit 47bb2d3c1ee73bdb803ee2c54526a457369c4ebb. Reason for revert: This changes makes external/wpt/html/cross-origin-opener-policy/reporting/access-reporting/property-self.https.html timeout. This reland also includes an extension of the timeout for property-self.https.html to avoid the problem created the first time around. Original change's description: > Revert "Extend timeout duration in reporting-common.js" > > This reverts commit 5ca6b0751e00bb57fd8f79abc59d1a92eaeb5c85. > > Reason for revert: > LUCI Bisection has identified this change as the cause of a test failure. See the analysis: https://ci.chromium.org/ui/p/chromium/bisection/test-analysis/b/4805896057126912 > > Sample build with failed test: https://ci.chromium.org/b/8702186065557938769 > Affected test(s): > [ninja://:blink_wpt_tests/external/wpt/html/cross-origin-opener-policy/reporting/access-reporting/property-opener-set.https.html](https://ci.chromium.org/ui/test/chromium/ninja:%2F%2F:blink_wpt_tests%2Fexternal%2Fwpt%2Fhtml%2Fcross-origin-opener-policy%2Freporting%2Faccess-reporting%2Fproperty-opener-set.https.html?q=VHash%3Aa7f138a2c78a0010) > > If this is a false positive, please report it at http://b.corp.google.com/createIssue?component=1199205&description=Analysis%3A+https%3A%2F%2Fci.chromium.org%2Fui%2Fp%2Fchromium%2Fbisection%2Ftest-analysis%2Fb%2F4805896057126912&format=PLAIN&priority=P3&title=Wrongly+blamed+https%3A%2F%2Fchromium-review.googlesource.com%2Fc%2Fchromium%2Fsrc%2F%2B%2F7001360&type=BUG > > Original change's description: > > Extend timeout duration in reporting-common.js > > > > Local testing shows that answering code cache requests slowly makes > > certain tests like access-from-coop-page-to-opener_coop-ro.https.html > > flaky. > > > > This test already appears to be flaky on macOS: https://ci.chromium.org/ui/test/chromium/ninja%3A%2F%2F%3Aheadless_shell_wpt%2Fexternal%2Fwpt%2Fhtml%2Fcross-origin-opener-policy%2Freporting%2Faccess-reporting%2Faccess-from-coop-page-to-opener_coop-ro.https.html?q=V%3Aos%3DMac-15+ > > > > Extending the timeout fixes the problem locally and can hopefully > > reduce flakiness. > > > > > > Bug: 446941148 > > Change-Id: I1c69503ffe846d08d2fd7174051674f46c69d15b > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7001360 > > Reviewed-by: Francois Pierre Doray <fdoray@chromium.org> > > Commit-Queue: Olivier Li <olivierli@chromium.org> > > Cr-Commit-Position: refs/heads/main@{#1523753} > > > > Bug: 446941148 > No-Presubmit: true > No-Tree-Checks: true > No-Try: true > Change-Id: Id71410888c06f5c2ee200c03767a5653624e61c4 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7003769 > Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> > Commit-Queue: Nick Birnie <birnie@google.com> > Cr-Commit-Position: refs/heads/main@{#1523826} Bug: 446941148 Change-Id: I7434aac675db6f09fa7331604db1003876ed261a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7003376 Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Reviewed-by: Nick Birnie <birnie@google.com> Reviewed-by: Andrew Williams <awillia@chromium.org> Commit-Queue: Olivier Li <olivierli@chromium.org> Cr-Commit-Position: refs/heads/main@{#1524877} Co-authored-by: Olivier Li <olivierli@chromium.org> -- wpt-commits: d112cc68f97f0e73de2e57bcc00ae76ba273bfc4 wpt-pr: 55232 Diffstat:
2 files changed, 2 insertions(+), 1 deletion(-)
diff --git a/testing/web-platform/tests/html/cross-origin-opener-policy/reporting/access-reporting/property-opener-set.https.html b/testing/web-platform/tests/html/cross-origin-opener-policy/reporting/access-reporting/property-opener-set.https.html @@ -1,3 +1,4 @@ +<meta name="timeout" content="long"> <title> Check openee.opener access is checked</title> <script src=/resources/testharness.js></script> <script src=/resources/testharnessreport.js></script> diff --git a/testing/web-platform/tests/html/cross-origin-opener-policy/reporting/resources/reporting-common.js b/testing/web-platform/tests/html/cross-origin-opener-policy/reporting/resources/reporting-common.js @@ -340,7 +340,7 @@ const receiveReport = async function(uuid, type) { let reports = await Promise.race([ receive(uuid), new Promise(resolve => { - step_timeout(resolve, 1000, "timeout"); + step_timeout(resolve, 3000, "timeout"); }) ]); if (reports == "timeout")