Closed Bug 1847230 Opened 1 year ago Closed 1 year ago

[wpt-sync] Sync PR 41339 - Fenced Frames: Parameterize automatic beacons in WPTs

Categories

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

task

Tracking

(firefox118 fixed)

RESOLVED FIXED
118 Branch
Tracking Status
firefox118 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

(Depends on 1 open bug, Blocks 1 open bug, )

Details

(Whiteboard: [wptsync downstream])

Attachments

(1 file)

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

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

Andrew Verge <averge@chromium.org> wrote:

Fenced Frames: Parameterize automatic beacons in WPTs

Instead of hardcoding whether an automatic beacon should be registered
as part of reportWin() in FLEDGE bidding logic, tests can now accept
a parameter to indicate whether they should register the beacon or not.
Existing automatic beacon tests have been modified to use this approach.

Change-Id: Id33d956f8c6d5af863ef4a5ae0994e5141bdee18
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4740218
Reviewed-by: Liam Brady \<lbrady@google.com>
Commit-Queue: Andrew Verge \<averge@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1179156}

PR 41339 applied with additional changes from upstream: 535a9abff0616c0d7f2b433f5aff0fd725ebc22b
Attached file Notable wpt changes
# CI Results Ran 9 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI Total 181 tests and 1 subtests ## Status Summary ### Firefox `OK` : 164[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, Gecko-windows11-64-2009-qr-opt] 172[GitHub] `PASS` : 16 `CRASH` : 1[Gecko-linux1804-64-qr-debug, Gecko-windows11-64-2009-qr-debug] 2[Gecko-windows11-32-2009-qr-debug] `FAIL` : 408[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, Gecko-windows11-64-2009-qr-opt] 480[GitHub] `TIMEOUT`: 12[Gecko-windows11-32-2009-qr-debug] 13[Gecko-linux1804-64-qr-debug, Gecko-windows11-64-2009-qr-debug] 14[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-opt, Gecko-windows11-32-2009-qr-opt, Gecko-windows11-64-2009-qr-opt, GitHub] `ERROR` : 2[Gecko-windows11-32-2009-qr-debug] 3[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-opt, Gecko-windows11-64-2009-qr-debug, Gecko-windows11-64-2009-qr-opt, GitHub] `NOTRUN` : 11[Gecko-windows11-32-2009-qr-debug] 14[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-opt, Gecko-windows11-64-2009-qr-debug, Gecko-windows11-64-2009-qr-opt, GitHub] ### Chrome `OK` : 172 `PASS` : 17 `FAIL` : 483 `TIMEOUT`: 14 `ERROR` : 3 `NOTRUN` : 10 ### Safari `OK` : 171 `PASS` : 15 `FAIL` : 479 `TIMEOUT`: 14 `ERROR` : 4 `NOTRUN` : 14 ## Links [Gecko CI (Treeherder)](https://treeherder.mozilla.org/#/jobs?repo=try&revision=9857b69153f64e3c087fca515119cdd745108be0) [GitHub PR Head](https://wpt.fyi/results/?sha=673f6d7014c151e5d1bd8790a7410a644942be22&label=pr_head) [GitHub PR Base](https://wpt.fyi/results/?sha=673f6d7014c151e5d1bd8790a7410a644942be22&label=pr_base) ## Details ### Crashes * [/fenced-frame/embedder-require-corp.https.html](https://wpt.live/fenced-frame/embedder-require-corp.https.html) [[wpt.fyi](https://wpt.fyi/results/fenced-frame/embedder-require-corp.https.html)]: `CRASH` [`Gecko-windows11-32-2009-qr-debug`], `ERROR` [`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-opt`, `Gecko-windows11-64-2009-qr-debug`, `Gecko-windows11-64-2009-qr-opt`, `GitHub`] linked bug:Bug 1833103 * [/fenced-frame/sandbox-mandatory-flags.https.html](https://wpt.live/fenced-frame/sandbox-mandatory-flags.https.html) [[wpt.fyi](https://wpt.fyi/results/fenced-frame/sandbox-mandatory-flags.https.html)]: `CRASH` [`Gecko-linux1804-64-qr-debug`, `Gecko-windows11-32-2009-qr-debug`, `Gecko-windows11-64-2009-qr-debug`], `TIMEOUT` [`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-opt`, `Gecko-windows11-32-2009-qr-opt`, `Gecko-windows11-64-2009-qr-opt`, `GitHub`] (See attachment for full changes)
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/9a3636908357 [wpt PR 41339] - Fenced Frames: Parameterize automatic beacons in WPTs, a=testonly
Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 118 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: