1786775 - [wpt-sync] Sync PR 35593 - [beacon-api] Allow non-CORS-safelisted content-type with preflight requests

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

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

Ming-Ying Chung <[e-mail]> wrote:

[beacon-api] Allow non-CORS-safelisted content-type with preflight requests

According to offline discussion with yhirano@, the spec implies
that we should follow Fetch's spec which allows beacon with non-CORS-safelisted content-type that requires preflight requests.

Bug: 1352366,1293679
Change-Id: Ic3c876a5226bd855098c5fb1e9114138fca6cd50
Reviewed-on: https://chromium-review.googlesource.com/3845374
WPT-Export-Revision: 90cae5101354d1879674d0e51e318bac116b5c68

Comment 1

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

PR 35593 applied with additional changes from upstream: 74938560647321aed597aa158a4bff731edbd759, f1ef84ca14e9e7f18d82544dfa2191ccab027757, 132606abb6068abac3aad510ede995caa37032de, ac84adaa5a4190f4937db37882ba1e3f00511c68

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=f383ee954da2c51dde40211f022b455e6b41e196

Comment 3

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

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

Comment 4

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

CI Results

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

Total 8 tests

Status Summary

Firefox

ERROR: 8

Chrome

ERROR: 8

Safari

ERROR: 8

Links

GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

• /pending_beacon/pending_beacon-basic.tentative.window.html [wpt.fyi]: ERROR (Chrome: ERROR, Safari: ERROR)
• /pending_beacon/pending_beacon-deactivate.tentative.window.html [wpt.fyi]: ERROR (Chrome: ERROR, Safari: ERROR)
• /pending_beacon/pending_beacon-sendnow.tentative.window.html [wpt.fyi]: ERROR (Chrome: ERROR, Safari: ERROR)
• /pending_beacon/pending_beacon-sendondiscard.tentative.window.html [wpt.fyi]: ERROR (Chrome: ERROR, Safari: ERROR)
• /pending_beacon/pending_get_beacon-cors.tentative.https.window.html [wpt.fyi]: ERROR (Chrome: ERROR, Safari: ERROR)
• /pending_beacon/pending_get_beacon-send.tentative.window.html [wpt.fyi]: ERROR (Chrome: ERROR, Safari: ERROR)
• /pending_beacon/pending_post_beacon-cors.tentative.https.window.html [wpt.fyi]: ERROR (Chrome: ERROR, Safari: ERROR)
• /pending_beacon/pending_post_beacon-sendwithdata.tentative.window.html [wpt.fyi]: ERROR (Chrome: ERROR, Safari: ERROR)

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/46773be4c6ac
[wpt PR 35593] - [beacon-api] Allow non-CORS-safelisted content-type with preflight requests, a=testonly

Comment 6

[Cristina Cozmuta (:CrissCozmuta)]

https://hg.mozilla.org/mozilla-central/rev/46773be4c6ac