Bug 175635 - [GTK] Regression in imported/w3c/web-platform-tests/fetch/api/basic/request-headers.any.html
Summary: [GTK] Regression in imported/w3c/web-platform-tests/fetch/api/basic/request-h...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-08-16 13:11 PDT by Alicia Boya García
Modified: 2017-08-16 17:44 PDT (History)
4 users (show)

See Also:


Attachments
Patch (2.20 KB, patch)
2017-08-16 13:17 PDT, Alicia Boya García
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Alicia Boya García 2017-08-16 13:11:52 PDT
The following layout test is failing on GTK.

imported/w3c/web-platform-tests/fetch/api/basic/request-headers.any.html

Probable cause:

r220779 introduced a workaround for the Fetch & Beacon API:

> Use "application/octet-stream" Content-Type for payloads of type
> ArrayBuffer / ArrayBufferView in Fetch & Beacon.  If we don't set a
> Content-Type header, our underlying network stack is going to add a
> "application/x-www-form-urlencoded" Content-Type header, which is worse.
> 
> Chrome is already using "application/octet-stream" Content-Type in Beacon,
> but use no Content-Type in Fetch.  The Fetch/Beacon specification says we
> should have no Content-Type header in this case but this is unfortunately not
> something we can support at the moment. Using "application/octet-stream"
> Content-Type for now seems like the lesser evil.

This workaround is unnecessary in WebKitGTK, which does not have the mentioned
limitation. The related tests were passing in WebKitGTK, but this workaround
has turned them into failures.

We should either:

a) Introduce the workaround only for platforms where it is needed.

b) Update tests expectations in all ports.

I think a) is preferrable in this case, as the workaround is for port-specific
issues.
Comment 1 Alicia Boya García 2017-08-16 13:17:57 PDT
Created attachment 318285 [details]
Patch
Comment 2 youenn fablet 2017-08-16 13:26:14 PDT
r220779 will probably be rolled out.
If not, we should indeed restrict it to platform not being able to comply with the specs.
Comment 3 Chris Dumez 2017-08-16 13:41:45 PDT
Patch is being reworked via https://bugs.webkit.org/show_bug.cgi?id=175628. Will likely land today.
Comment 4 Michael Catanzaro 2017-08-16 17:44:49 PDT
Comment on attachment 318285 [details]
Patch

So the patch in bug #175628 has landed. Hopefully that means we can close this bug.