commit 45f1864bc756f18e1e34b64b0703717536adbf03
parent b7e389e5b1dad67a52aae98681298d3ed4084ede
Author: luci-bisection@appspot.gserviceaccount.com <luci-bisection@appspot.gserviceaccount.com>
Date: Thu, 9 Oct 2025 20:34:16 +0000
Bug 1992186 [wpt PR 55185] - Revert "Extend timeout duration in reporting-common.js", a=testonly
Automatic update from web-platform-tests
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}
--
wpt-commits: 2b7d952d8da1e5588612c85d98ee05af60e11b4b
wpt-pr: 55185
Diffstat:
1 file changed, 1 insertion(+), 1 deletion(-)
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, 3000, "timeout");
+ step_timeout(resolve, 1000, "timeout");
})
]);
if (reports == "timeout")
