Closed Bug 1853655 Opened 1 year ago Closed 1 year ago

Check WPT failure on dedicated-worker-options-credentials.html shared-worker-options-credentials.html

Categories

(Core :: DOM: Workers, task)

task

Tracking

()

RESOLVED FIXED
119 Branch
Tracking Status
firefox119 --- fixed

People

(Reporter: allstars.chh, Assigned: allstars.chh)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

This reverts the following commits from WPT
https://github.com/web-platform-tests/wpt/commit/0531410985f4a93b0859acfb9182fc6c6a0fe334
https://github.com/web-platform-tests/wpt/commit/cb9a716bfe448032c5d323d4db1331df986d05f3

As it causes regressions and broke the following WPT tests between all browsers

  • dedicated-worker-options-credentials.html
  • shared-worker-options-credentials.html

https://wpt.fyi/results/workers/modules/dedicated-worker-options-credentials.html?label=master&label=experimental&product=chrome&product=firefox&product=safari&aligned&view=interop&q=label%3Ainterop-2023-modules
https://wpt.fyi/results/workers/modules/shared-worker-options-credentials.html?label=master&label=experimental&product=chrome&product=firefox&product=safari&aligned&view=interop&q=label%3Ainterop-2023-modules

The commits @0531410 and @cb9a716 use a different host "{{hosts[alt][]}}",
which will be resolved to "not-web-platform.test" as the remote origin,
but the test in dedicated-worker-options-credentials.html will test with
HTTP cookies, which are associated with HTTP domains, and "{{hosts[alt][]}}"
is a different domain so the cookies cannot shared and causes the
failure.

See https://github.com/web-platform-tests/wpt/issues/41877

Pushed by allstars.chh@gmail.com: https://hg.mozilla.org/integration/autoland/rev/879e77eba24e Revert wpt commits @0531410 and @cb9a716. r=dom-worker-reviewers,smaug
Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 119 Branch
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/42057 for changes under testing/web-platform/tests
Upstream PR merged by moz-wptsync-bot
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: