1860852 - [wpt-sync] Sync PR 42721 - [FLEDGE] WPT tests - clean up request tracker a little.

Status: RESOLVED FIXED

(Testing :: web-platform-tests, task, P4)

Description

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

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

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

Matt Menke <[e-mail]> wrote:

[FLEDGE] WPT tests - clean up request tracker a little.

Rename tracked_requests to tracked_data, in preparation for adding more
tracked types of data, which are all retrieved by the same function.

Add helper to fetch tracked_data and check it for errors.

Clean up waitForObservedRequests() to have a little less redundant code.

Also update the TODO file to reflect previously done work.

Bug: 1425952
Change-Id: I0903e976192e749afaa18f2005a1279ed02c9588

Reviewed-on: https://chromium-review.googlesource.com/4972236
WPT-Export-Revision: 02389b7e7527cde0584b5f4dbd1f059bc59969a6

Comment 1

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

PR 42721 applied with additional changes from upstream: 9df6bbbc8c02189b54f6774e5e087ab990374a0a, 8b581dc221cfd57b5d615b5707744d3eb75140b6, c46b06d669b9af8b5a97ee9533e3f2f6abadeafa, 7d7ae45c502ee8087c9cb730e4446e23f4fa12a7

Comment 2

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

Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=1bfb5a3a4f2737d43e84348cc6fdfe751eb563ea

Comment 3

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

Attachment: Notable wpt changes

# CI Results

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

Total 72 tests and 5 subtests

## Status Summary

### Firefox
`OK`  : 9[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, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt, Gecko-windows11-32-2009-qr-debug, Gecko-windows11-32-2009-qr-opt, Gecko-windows11-64-2009-qr-debug] 72[GitHub]
`PASS`: 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, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt, Gecko-windows11-32-2009-qr-debug, Gecko-windows11-32-2009-qr-opt, Gecko-windows11-64-2009-qr-debug] 2[GitHub]
`FAIL`: 24[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, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt, Gecko-windows11-32-2009-qr-debug, Gecko-windows11-32-2009-qr-opt, Gecko-windows11-64-2009-qr-debug] 365[GitHub]

### Chrome
`OK`  : 72
`PASS`: 2
`FAIL`: 365

### Safari
`OK`  : 72
`PASS`: 2
`FAIL`: 365

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

## Details

(See attachment for full changes)

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/5b203c8b6db8
[wpt PR 42721] - [FLEDGE] WPT tests - clean up request tracker a little., a=testonly
https://hg.mozilla.org/integration/autoland/rev/ba09e2e9e4a2
[wpt PR 42721] - Update wpt metadata, a=testonly

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/1602c37f4f27
[wpt PR 42721] - [FLEDGE] WPT tests - clean up request tracker a little., a=testonly
https://hg.mozilla.org/integration/autoland/rev/ec0d6858c4de
[wpt PR 42721] - Update wpt metadata, a=testonly

Comment 6

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/1602c37f4f27
https://hg.mozilla.org/mozilla-central/rev/ec0d6858c4de