1847581 - [wpt-sync] Sync PR 41365 - Fenced Frames: Add WPT checks for requestedSize API.

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

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

Andrew Verge <[e-mail]> wrote:

Fenced Frames: Add WPT checks for requestedSize API.

For all existing tests that pass requestedSize to the auctionConfig:

• A URL param with the same width and height from requestedSize is
  passed in the bidding and decision logic URLs.
• The serving code for both bidding and decision inserts requestedSize
  checks in the returned Javascript.
• Those checks confirm that the requestedSize in the browserSignals
  (for bidding) or auctionConfig (for decision) have the same values as
  the URL param. If the JS objects and the URL param don't match, an
  error is thrown.

With this method, we can confirm at runtime that the requestedSize is
plumbed through the browser correctly, because the generated Javascript
knows what the plumbed value is supposed to be.

Change-Id: I36664db6e4a62c62e13dbcce86d78eabb8937b6b
Reviewed-on: https://chromium-review.googlesource.com/4756460
WPT-Export-Revision: 72e3d8daec151c90d1fc96f3e14ccab74564ebdd

Comment 1

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

PR 41365 applied with additional changes from upstream: 535a9abff0616c0d7f2b433f5aff0fd725ebc22b, 8f2f8e97d89c955dbe71f613502ccfb7345d741f

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=1c0ec8b135b5f777dfbbef176d822076002544bd

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 184 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] 173[GitHub]
`PASS`   : 16[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] 17[GitHub]
`CRASH`  : 1
`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] 481[GitHub]
`TIMEOUT`: 12[Gecko-windows11-32-2009-qr-debug] 13[Gecko-linux1804-64-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-debug, Gecko-windows11-64-2009-qr-opt, GitHub]
`ERROR`  : 4[Gecko-windows11-64-2009-qr-debug] 5[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-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`     : 173
`PASS`   : 18
`FAIL`   : 484
`TIMEOUT`: 14
`ERROR`  : 5
`NOTRUN` : 10

### Safari
`OK`     : 172
`PASS`   : 16
`FAIL`   : 480
`TIMEOUT`: 14
`ERROR`  : 6
`NOTRUN` : 14

## Links
[Gecko CI (Treeherder)](https://treeherder.mozilla.org/#/jobs?repo=try&revision=1c0ec8b135b5f777dfbbef176d822076002544bd)
[GitHub PR Head](https://wpt.fyi/results/?sha=1a2a2e2e31938f96a126a1735a4d10d59ffd89ea&label=pr_head)
[GitHub PR Base](https://wpt.fyi/results/?sha=1a2a2e2e31938f96a126a1735a4d10d59ffd89ea&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-64-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-debug`, `Gecko-windows11-32-2009-qr-opt`, `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`], `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-debug`, `Gecko-windows11-64-2009-qr-opt`, `GitHub`] linked bug:Bug 1847365

(See attachment for full changes)

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/0e6a18c52d03
[wpt PR 41365] - Fenced Frames: Add WPT checks for requestedSize API., a=testonly

Comment 5

[Serban Stanca [:SerbanS]]

https://hg.mozilla.org/mozilla-central/rev/0e6a18c52d03