[Streams] Pass WebDataConsumerHandle when the response arrives.

[Streams] Pass WebDataConsumerHandle when the response arrives.

We are implementing the Streams API. To make back-pressure take effect, we
should allow the resource loading customer such as XHR or the Fetch API
to read the data manually.
This CL adds a WebDataConsumerHandle to didReceiveResponse parameters.
This CL does not change the behavior as of now.

The tricky part is how to deal with a handle. As WebDataConsumerHandle
cannot be shared by multiple clients, we can't add the handle in the
response. Moreover, we assert that handle must be null when more than
one client exists in RawResource.cpp.

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

[yhirano, 2014-10-09 08:23:26]

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

[yhirano, 2014-10-09 08:25:38]

PTAL

This CL depends on https://codereview.chromium.org/626803002/

[tyoshino (SeeGerritForStatus), 2014-10-29 07:37:50]

lgtm

[yhirano, 2014-10-29 08:37:46]

yhirano@chromium.org changed reviewers:
+ japhet@chromium.org, mkwst@chromium.org

[yhirano, 2014-10-29 08:37:47]

+japhet, mkwst for OWNER review.

[Mike West, 2014-11-04 15:09:31]

LGTM % nits

https://codereview.chromium.org/603903003/diff/60001/Source/core/fetch/RawRes...
File Source/core/fetch/RawResource.cpp (right):

https://codereview.chromium.org/603903003/diff/60001/Source/core/fetch/RawRes...
Source/core/fetch/RawResource.cpp:102: ASSERT(count() == 1 || !handle);
Can 'count()' be 0?

https://codereview.chromium.org/603903003/diff/60001/Source/core/fileapi/File...
File Source/core/fileapi/FileReaderLoader.cpp (right):

https://codereview.chromium.org/603903003/diff/60001/Source/core/fileapi/File...
Source/core/fileapi/FileReaderLoader.cpp:176: ASSERT_UNUSED(handle, !handle);
Why this ASSERT?

[yhirano, 2014-11-05 04:38:40]

https://codereview.chromium.org/603903003/diff/60001/Source/core/fetch/RawRes...
File Source/core/fetch/RawResource.cpp (right):

https://codereview.chromium.org/603903003/diff/60001/Source/core/fetch/RawRes...
Source/core/fetch/RawResource.cpp:102: ASSERT(count() == 1 || !handle);
On 2014/11/04 15:09:30, Mike West wrote:
> Can 'count()' be 0?

Done.

https://codereview.chromium.org/603903003/diff/60001/Source/core/fileapi/File...
File Source/core/fileapi/FileReaderLoader.cpp (right):

https://codereview.chromium.org/603903003/diff/60001/Source/core/fileapi/File...
Source/core/fileapi/FileReaderLoader.cpp:176: ASSERT_UNUSED(handle, !handle);
On 2014/11/04 15:09:31, Mike West wrote:
> Why this ASSERT?

I'm planning to use this parameter only for XHR and fetch API (at least, at
first). FileReaderLoaderClient has no interface for WebDataConsumerHandle and
having non-null handle here means something wrong happens. Hence I added this
assertion.

[Mike West, 2014-11-05 08:23:25]

Still LGTM.

[yhirano, 2014-11-06 01:14:28]

The CQ bit was checked by yhirano@chromium.org

[commit-bot: I haz the power, 2014-11-06 01:15:55]

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

[commit-bot: I haz the power, 2014-11-06 02:33:54]

Committed patchset #6 (id:100001) as 184892