[Fetch] Check / Set .bodyUsed in Request construction if the src body is null.

[Fetch] Check / Set .bodyUsed in Request construction if the src body is null.

We need to check / set .bodyUsed in RequestConstruction when a Request is given,
even if the Request has a null body. See [1] for details.

1: https://github.com/whatwg/fetch/issues/61

BUG=501195
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=197325

[yhirano, 2015-06-17 09:11:35]

yhirano@chromium.org changed reviewers:
+ tyoshino@chromium.org

[yhirano, 2015-06-17 09:11:35]

PTAL

[tyoshino (SeeGerritForStatus), 2015-06-18 06:34:35]

lgtm

https://codereview.chromium.org/1187653003/diff/1/Source/modules/fetch/Reques...
File Source/modules/fetch/Request.cpp (right):

https://codereview.chromium.org/1187653003/diff/1/Source/modules/fetch/Reques...
Source/modules/fetch/Request.cpp:55: if (inputRequest->bodyUsed()) {
are these if-s separated intentionally?

[yhirano, 2015-06-18 06:44:24]

https://codereview.chromium.org/1187653003/diff/1/Source/modules/fetch/Reques...
File Source/modules/fetch/Request.cpp (right):

https://codereview.chromium.org/1187653003/diff/1/Source/modules/fetch/Reques...
Source/modules/fetch/Request.cpp:55: if (inputRequest->bodyUsed()) {
On 2015/06/18 06:34:34, tyoshino wrote:
> are these if-s separated intentionally?

Yes.

[yhirano, 2015-06-18 07:32:55]

The CQ bit was checked by yhirano@chromium.org

[commit-bot: I haz the power, 2015-06-18 07:33:13]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1187653003/1

[commit-bot: I haz the power, 2015-06-18 07:37:53]

Committed patchset #1 (id:1) as
https://src.chromium.org/viewvc/blink?view=rev&revision=197325