Closed Bug 1615997 Opened 4 years ago Closed 4 years ago

[wpt-sync] Sync PR 21832 - [WPT/mixed-content] Remove websocket + redirect tests

Categories

(Core :: DOM: Security, task, P4)

task

Tracking

()

RESOLVED FIXED
mozilla75
Tracking Status
firefox75 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream][domsecurity-backlog])

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

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

Hiroshige Hayashizaki <hiroshige@chromium.org> wrote:

[WPT/mixed-content] Remove websocket + redirect tests

Redirects are forbidden (see Step 12 and the warning at the bottom:
https://fetch.spec.whatwg.org/#concept-websocket-establish ).

Previously, websocket + redirect tests were generated, but
their behavior was the same as websocket + no-redirect,
because the websocket subresource endpoint ignores
redirect-related parameters.
This CL removes such redundant tests.

Bug: 906850
Change-Id: I86be33d1812b4d16ba463f14320fb23369321cc0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2058707
Commit-Queue: Hiroshige Hayashizaki \<hiroshige@chromium.org>
Reviewed-by: Hiroki Nakagawa \<nhiroki@chromium.org>
Reviewed-by: Mike West \<mkwst@chromium.org>
Reviewed-by: Kenichi Ishibashi \<bashi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#741851}

PR 21832 applied with additional changes from upstream: 04433e487ca9829c6326910b436bd85cb29540f5, 89585ca77ad3e4e334e84e49e2ca19b466285051, b2331fccbc54f5b0a57d66190d30d40341cbc957
Component: web-platform-tests → DOM: Security
Product: Testing → Core
Whiteboard: [wptsync downstream] → [wptsync downstream][domsecurity-backlog]
Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/ff8bf380fbb1
[wpt PR 21832] - [WPT/mixed-content] Remove websocket + redirect tests, a=testonly
Test result changes from PR not available.
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla75
You need to log in before you can comment on or make changes to this bug.