1438085 - [wpt-sync] PR 9453 - Fetch API: Don't DCHECK on a 302 response with a Location header with empty value.

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect, P3)

Description

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

Blink was failing a DCHECK when it expected a redirect to be taken because
the request mode was 'follow' and the response had a redirect status code
and Location header. However, Chrome's //net considers a Location header
with an empty value to be the same as no Location header at all. See the
comments on bug 810288 about net::HttpResponseHeaders::IsRedirect().

WPT tests are added. The spec is unclear (https://github.com/whatwg/url/issues/373)
but seems to either say to fail with network error or enter a loop of self-redirects
that will eventually hit the redirect limit and fail with network error.
Chrome currently fails this test because it treats the Location as
as non-existent, i.e., not a redirect response.

Bug: 810288,707185
Change-Id: Iaca93976aaa697380b1542ab45d63c318f5050b0

Reviewed-on: https://chromium-review.googlesource.com/910753
WPT-Export-Revision: 807e3e33fd628d68b22cba980a4a9583f907b63b

Comment 1

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

Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=c1056c06da6018d05c16afdd7a5b07a6f737dd7d

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/794c66f8f16e
[wpt PR 9453] - Fetch API: Don't DCHECK on a 302 response with a Location header with empty value., a=testonly

Comment 3

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/794c66f8f16e