1864208 - [wpt-sync] Sync PR 43082 - FLEDGE: In WPT currency tests, use forDebuggingOnly.reportAdAuctionWin

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 43082 into mozilla-central (this bug is closed when the sync is complete).

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

Maks Orlovich <[e-mail]> wrote:

FLEDGE: In WPT currency tests, use forDebuggingOnly.reportAdAuctionWin

... where we previously only looked at highestScoringOtherBid, since it covers more uses of converted values.

To support this, createTrackerURL was changed to no longer form-encode the ID, since keyword substitution in reportAdAuctionWin doesn't. This also permitted dropping a bunch of encodeURIComponent usages with sendReportTo()

Change-Id: Ibda46f34f3c17fc72374b175d982b9f235681076
Reviewed-on: https://chromium-review.googlesource.com/5017316
WPT-Export-Revision: 2c809de00b67d5d8f3c49900e2481a337cdb41a3

Comment 1

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

PR 43082 applied with additional changes from upstream: f1212f52f0be8e2cbb91cbaf167eb04e4540acab, 2c5fe89e8eba261ea32e71172f0a6b62db15e45b, 3e8972dae3627c3824b8fea907cb36e78b05c91b

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=95ae919ce07f80f8707cb7e409fcce1ce1f7a5d0

Comment 3

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

Attachment: Notable wpt changes

# CI Results

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

Total 91 tests and 5 subtests

## Status Summary

### Firefox
`OK`  : 91
`FAIL`: 446

### Chrome
`OK`  : 91
`FAIL`: 446

### Safari
`OK`  : 91
`FAIL`: 446

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

## Details

(See attachment for full changes)

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/2cd69d6ec18f
[wpt PR 43082] - FLEDGE: In WPT currency tests, use forDebuggingOnly.reportAdAuctionWin, a=testonly
https://hg.mozilla.org/integration/autoland/rev/7a78c33854ce
[wpt PR 43082] - Update wpt metadata, a=testonly

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/9bfd7c551ff5
[wpt PR 43082] - FLEDGE: In WPT currency tests, use forDebuggingOnly.reportAdAuctionWin, a=testonly
https://hg.mozilla.org/integration/autoland/rev/77562e3562d3
[wpt PR 43082] - Update wpt metadata, a=testonly

Comment 6

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/9bfd7c551ff5
https://hg.mozilla.org/mozilla-central/rev/77562e3562d3