1808249 - [wpt-sync] Sync PR 37726 - Add a time limit to avoid loading timeouts in Resource-Timing tests

Status: RESOLVED FIXED

(Core :: DOM: Performance APIs, task, P4)

Description

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

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

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

Yoav Weiss <[e-mail]> wrote:

Add a time limit to avoid loading timeouts in Resource-Timing tests

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

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=619eef3877acc1148856293e9a03385ce331b695

Comment 2

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

Attachment: Notable wpt changes

# CI Results

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

Total 51 tests and 6 subtests

## Status Summary

### Firefox
`OK`     : 43[Gecko-android-em-7.0-x86_64-qr-debug-isolated-process-geckoview] 46[Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt, Gecko-windows10-32-2004-qr-debug, Gecko-windows10-32-2004-qr-opt, Gecko-windows10-64-2004-qr-debug, Gecko-windows10-64-2004-qr-opt, GitHub] 48[Gecko-android-em-7.0-x86_64-lite-qr-opt-geckoview, Gecko-android-em-7.0-x86_64-qr-debug-geckoview, Gecko-android-em-7.0-x86_64-qr-opt-geckoview]
`PASS`   : 142[Gecko-android-em-7.0-x86_64-qr-debug-isolated-process-geckoview] 148[GitHub] 149[Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt, Gecko-windows10-32-2004-qr-debug, Gecko-windows10-32-2004-qr-opt, Gecko-windows10-64-2004-qr-debug, Gecko-windows10-64-2004-qr-opt] 150[Gecko-android-em-7.0-x86_64-lite-qr-opt-geckoview, Gecko-android-em-7.0-x86_64-qr-debug-geckoview, Gecko-android-em-7.0-x86_64-qr-opt-geckoview]
`FAIL`   : 120[Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt, Gecko-windows10-32-2004-qr-debug, Gecko-windows10-32-2004-qr-opt, Gecko-windows10-64-2004-qr-debug, Gecko-windows10-64-2004-qr-opt] 202[Gecko-android-em-7.0-x86_64-qr-debug-isolated-process-geckoview] 210[Gecko-android-em-7.0-x86_64-lite-qr-opt-geckoview, Gecko-android-em-7.0-x86_64-qr-debug-geckoview, Gecko-android-em-7.0-x86_64-qr-opt-geckoview] 307[GitHub]
`TIMEOUT`: 1[Gecko-android-em-7.0-x86_64-lite-qr-opt-geckoview, Gecko-android-em-7.0-x86_64-qr-debug-geckoview, Gecko-android-em-7.0-x86_64-qr-opt-geckoview] 5[Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt, Gecko-windows10-32-2004-qr-debug, Gecko-windows10-32-2004-qr-opt, Gecko-windows10-64-2004-qr-debug, Gecko-windows10-64-2004-qr-opt] 6[Gecko-android-em-7.0-x86_64-qr-debug-isolated-process-geckoview] 7[GitHub]
`ERROR`  : 2
`NOTRUN` : 7[Gecko-android-em-7.0-x86_64-lite-qr-opt-geckoview, Gecko-android-em-7.0-x86_64-qr-debug-geckoview, Gecko-android-em-7.0-x86_64-qr-debug-isolated-process-geckoview, Gecko-android-em-7.0-x86_64-qr-opt-geckoview] 58[GitHub] 96[Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt, Gecko-windows10-32-2004-qr-debug, Gecko-windows10-32-2004-qr-opt, Gecko-windows10-64-2004-qr-debug, Gecko-windows10-64-2004-qr-opt]

### Chrome
`OK`     : 50
`PASS`   : 500
`FAIL`   : 17
`ERROR`  : 1

### Safari
`OK`     : 44
`PASS`   : 77
`FAIL`   : 68
`TIMEOUT`: 11
`ERROR`  : 2
`NOTRUN` : 365

## Links
[Gecko CI (Treeherder)](https://treeherder.mozilla.org/#/jobs?repo=try&revision=619eef3877acc1148856293e9a03385ce331b695)
[GitHub PR Head](https://wpt.fyi/results/?sha=ea1be80f6ef6e02c1e688846f5b6b53d4ec0ef0e&label=pr_head)
[GitHub PR Base](https://wpt.fyi/results/?sha=ea1be80f6ef6e02c1e688846f5b6b53d4ec0ef0e&label=pr_base)

## Details

### Firefox-only Failures
* [/resource-timing/buffer-full-add-after-full-event.html](https://wpt.live/resource-timing/buffer-full-add-after-full-event.html) [[wpt.fyi](https://wpt.fyi/results/resource-timing/buffer-full-add-after-full-event.html)]: `SKIP` [`Gecko-android-em-7.0-x86_64-lite-qr-opt-geckoview`, `Gecko-android-em-7.0-x86_64-qr-debug-geckoview`, `Gecko-android-em-7.0-x86_64-qr-debug-isolated-process-geckoview`, `Gecko-android-em-7.0-x86_64-qr-opt-geckoview`, `Gecko-linux1804-64-qr-debug`, `Gecko-linux1804-64-qr-opt`, `Gecko-windows10-32-2004-qr-debug`, `Gecko-windows10-32-2004-qr-opt`, `Gecko-windows10-64-2004-qr-debug`, `Gecko-windows10-64-2004-qr-opt`], `TIMEOUT` [`GitHub`]
  * Test that entry was added to the buffer after a buffer full event: `TIMEOUT`
* [/resource-timing/buffer-full-add-then-clear.html](https://wpt.live/resource-timing/buffer-full-add-then-clear.html) [[wpt.fyi](https://wpt.fyi/results/resource-timing/buffer-full-add-then-clear.html)]
  * Test that if the buffer is cleared after entries were added to the secondary buffer, those entries make it into the primary one: `FAIL`
* [/resource-timing/buffer-full-then-increased.html](https://wpt.live/resource-timing/buffer-full-then-increased.html) [[wpt.fyi](https://wpt.fyi/results/resource-timing/buffer-full-then-increased.html)]
  * Test that overflowing the buffer and immediately increasing its limit does not trigger the resourcetimingbufferfull event: `FAIL`
* [/resource-timing/buffer-full-when-populate-entries.html](https://wpt.live/resource-timing/buffer-full-when-populate-entries.html) [[wpt.fyi](https://wpt.fyi/results/resource-timing/buffer-full-when-populate-entries.html)]: `ERROR`
  * Test that a buffer full event does not bubble and that resourcetimingbufferfull is called only once per overflow: `FAIL`
* [/resource-timing/iframe-redirect-without-location.html](https://wpt.live/resource-timing/iframe-redirect-without-location.html) [[wpt.fyi](https://wpt.fyi/results/resource-timing/iframe-redirect-without-location.html)]: `OK` [`Gecko-android-em-7.0-x86_64-lite-qr-opt-geckoview`, `Gecko-android-em-7.0-x86_64-qr-debug-geckoview`, `Gecko-android-em-7.0-x86_64-qr-debug-isolated-process-geckoview`, `Gecko-android-em-7.0-x86_64-qr-opt-geckoview`], `TIMEOUT` [`Gecko-linux1804-64-qr-debug`, `Gecko-linux1804-64-qr-opt`, `Gecko-windows10-32-2004-qr-debug`, `Gecko-windows10-32-2004-qr-opt`, `Gecko-windows10-64-2004-qr-debug`, `Gecko-windows10-64-2004-qr-opt`, `GitHub`]
* [/resource-timing/same-origin-from-cross-origin-redirect.html](https://wpt.live/resource-timing/same-origin-from-cross-origin-redirect.html) [[wpt.fyi](https://wpt.fyi/results/resource-timing/same-origin-from-cross-origin-redirect.html)]
  * Verify that cross origin resources' timings are exposed when the final resource at the end of an HTTP redirect chain is same-origin.: `FAIL`
* [/resource-timing/sizes-redirect-img.html](https://wpt.live/resource-timing/sizes-redirect-img.html) [[wpt.fyi](https://wpt.fyi/results/resource-timing/sizes-redirect-img.html)]
  * PerformanceResourceTiming sizes redirect image - direct URL: `FAIL`
  * PerformanceResourceTiming sizes redirect image - same origin redirect: `FAIL`

(See attachment for full changes)

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/da07bf9c2e90
[wpt PR 37726] - Add a time limit to avoid loading timeouts in Resource-Timing tests, a=testonly
https://hg.mozilla.org/integration/autoland/rev/5819d3d0e950
[wpt PR 37726] - Update wpt metadata, a=testonly

Comment 4

[Noemi Erli[:noemi_erli]]

https://hg.mozilla.org/mozilla-central/rev/da07bf9c2e90
https://hg.mozilla.org/mozilla-central/rev/5819d3d0e950