[Webkit-unassigned] [Bug 168115] HTTPHeaderMap add methods are not consistent to each other

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=168115

--- Comment #13 from Carlos Garcia Campos <cgarcia at igalia.com> ---
Ok, looking at the tests in detail now I understand better what is going on

http/tests/xmlhttprequest/check-combining-headers.html
imported/w3c/web-platform-tests/XMLHttpRequest/setrequestheader-header-allowed.htm

These have FAIL lines in the expectations, like:

FAIL XMLHttpRequest: setRequestHeader() - combining headers (Authorization) assert_equals: Combined header value should be t1,t2 expected "t1,t2" but got "t1, t2"

Whatever patch we write for this bug should make those pass.

The other ones failing expect ',' not ", ".

And finally the one I was trying to fix expects ", " but it should really expect ','. According to the HTTP spec, the space is optional, but fetch spec says it shouldn't be any trailing/leaving whitespace in values, see https://fetch.spec.whatwg.org/#concept-header-list-combine. That's why it was changed in w3c web platform tests, but I think they forgot about imported/w3c/web-platform-tests/XMLHttpRequest/getresponseheader-case-insensitive.htm. I guess it's not a good idea to change imported tests, so we can simply add a failure expectation for this test.

So, let's be consistent and use ',' everywhere. I'll submit a new patch.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-unassigned/attachments/20170211/2de72a4f/attachment.html>