Fetch, XHR: Use ", " as header list value separator.

Fetch, XHR: Use ", " as header list value separator.

Adapt to https://github.com/whatwg/fetch/pull/504 and
https://github.com/whatwg/xhr/pull/130, which started requiring combined
header list values to be separated by ", " instead of just ",".

The XHR code handling responses already did that, but |ResourceRequest| and
|FetchHeaderList| both had to be updated.

As a bonus, this also makes us pass w-p-t's headers-combine.html.

BUG=700434, 705490
R=mkwst@chromium.org,tyoshino@chromium.org,yhirano@chromium.org

Review-Url: https://codereview.chromium.org/2822183002
Cr-Commit-Position: refs/heads/master@{#465537}
Committed: https://chromium.googlesource.com/chromium/src/+/fcbfde6ea277a07cddc3db613a3b4ff3de5d3933

[Raphael Kubo da Costa (rakuco), 2017-04-18 14:35:21]

Description was changed from

==========
Fetch, XHR: Use ", " as header list value separator.

Adapt to https://github.com/whatwg/fetch/pull/504 and
https://github.com/whatwg/xhr/pull/130, which started requiring combined
header list values to be separated by ", " instead of just ",".

The XHR code handling responses already did that, but |ResourceRequest| and
|FetchHeaderList| both had to be updated.

As a bonus, this also makes us to pass w-p-t's headers-combine.html, as well
as most of the tests in w-p-t's XMLHttpRequest/getallresponseheaders-cl.htm,
which is currently not imported into LayoutTests.

BUG=700434,705490
R=mkwst@chromium.org,tyoshino@chromium.org,yhirano@chromium.org
==========

to

==========
Fetch, XHR: Use ", " as header list value separator.

Adapt to https://github.com/whatwg/fetch/pull/504 and
https://github.com/whatwg/xhr/pull/130, which started requiring combined
header list values to be separated by ", " instead of just ",".

The XHR code handling responses already did that, but |ResourceRequest| and
|FetchHeaderList| both had to be updated.

As a bonus, this also makes us pass w-p-t's headers-combine.html, as well
as most of the tests in w-p-t's XMLHttpRequest/getallresponseheaders-cl.htm,
which is currently not imported into LayoutTests.

BUG=700434,705490
R=mkwst@chromium.org,tyoshino@chromium.org,yhirano@chromium.org
==========

[Raphael Kubo da Costa (rakuco), 2017-04-18 14:43:42]

Description was changed from

==========
Fetch, XHR: Use ", " as header list value separator.

Adapt to https://github.com/whatwg/fetch/pull/504 and
https://github.com/whatwg/xhr/pull/130, which started requiring combined
header list values to be separated by ", " instead of just ",".

The XHR code handling responses already did that, but |ResourceRequest| and
|FetchHeaderList| both had to be updated.

As a bonus, this also makes us pass w-p-t's headers-combine.html, as well
as most of the tests in w-p-t's XMLHttpRequest/getallresponseheaders-cl.htm,
which is currently not imported into LayoutTests.

BUG=700434,705490
R=mkwst@chromium.org,tyoshino@chromium.org,yhirano@chromium.org
==========

to

==========
Fetch, XHR: Use ", " as header list value separator.

Adapt to https://github.com/whatwg/fetch/pull/504 and
https://github.com/whatwg/xhr/pull/130, which started requiring combined
header list values to be separated by ", " instead of just ",".

The XHR code handling responses already did that, but |ResourceRequest| and
|FetchHeaderList| both had to be updated.

As a bonus, this also makes us pass w-p-t's headers-combine.html.

BUG=700434,705490
R=mkwst@chromium.org,tyoshino@chromium.org,yhirano@chromium.org
==========

[Raphael Kubo da Costa (rakuco), 2017-04-18 14:43:50]

The CQ bit was checked by raphael.kubo.da.costa@intel.com to run a CQ dry run

[commit-bot: I haz the power, 2017-04-18 14:44:08]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Raphael Kubo da Costa (rakuco), 2017-04-18 14:44:14]

PTAL

[commit-bot: I haz the power, 2017-04-18 16:27:49]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-04-18 16:27:50]

Dry run: This issue passed the CQ dry run.

[Mike West, 2017-04-19 08:30:19]

lgtm

[Raphael Kubo da Costa (rakuco), 2017-04-19 08:54:24]

The CQ bit was checked by raphael.kubo.da.costa@intel.com

[commit-bot: I haz the power, 2017-04-19 08:54:51]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-19 08:58:59]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1492592064432690, "parent_rev":
"14bcd90c3e83b60fb9ecce8504356b81af4c6209", "commit_rev":
"fcbfde6ea277a07cddc3db613a3b4ff3de5d3933"}

[commit-bot: I haz the power, 2017-04-19 08:59:07]

Description was changed from

==========
Fetch, XHR: Use ", " as header list value separator.

Adapt to https://github.com/whatwg/fetch/pull/504 and
https://github.com/whatwg/xhr/pull/130, which started requiring combined
header list values to be separated by ", " instead of just ",".

The XHR code handling responses already did that, but |ResourceRequest| and
|FetchHeaderList| both had to be updated.

As a bonus, this also makes us pass w-p-t's headers-combine.html.

BUG=700434,705490
R=mkwst@chromium.org,tyoshino@chromium.org,yhirano@chromium.org
==========

to

==========
Fetch, XHR: Use ", " as header list value separator.

Adapt to https://github.com/whatwg/fetch/pull/504 and
https://github.com/whatwg/xhr/pull/130, which started requiring combined
header list values to be separated by ", " instead of just ",".

The XHR code handling responses already did that, but |ResourceRequest| and
|FetchHeaderList| both had to be updated.

As a bonus, this also makes us pass w-p-t's headers-combine.html.

BUG=700434,705490
R=mkwst@chromium.org,tyoshino@chromium.org,yhirano@chromium.org

Review-Url: https://codereview.chromium.org/2822183002
Cr-Commit-Position: refs/heads/master@{#465537}
Committed:
https://chromium.googlesource.com/chromium/src/+/fcbfde6ea277a07cddc3db613a3b...
==========

[commit-bot: I haz the power, 2017-04-19 08:59:08]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/fcbfde6ea277a07cddc3db613a3b...

[tyoshino (SeeGerritForStatus), 2017-04-20 08:40:31]

lgtm