[Fetch API] FetchResponse should return a ReadableStream even if disturbed

[youenn fablet]

As per the specification, FetchResponse body should be a ReadableStream even if disturbed.
Currently, body is null if body is consumed.

[youenn fablet]

Created attachment 277060 [details]
Patch

[Darin Adler]

Comment on attachment 277060 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=277060&action=review

> Source/WebCore/Modules/fetch/FetchResponse.cpp:296
> +    ASSERT(!FetchBodyOwner::isDisturbed());

Why not just ASSERT(!isDisturbed())?

[youenn fablet]

Thanks for the review.

(In reply to comment #2)
> Comment on attachment 277060 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=277060&action=review
> 
> > Source/WebCore/Modules/fetch/FetchResponse.cpp:296
> > +    ASSERT(!FetchBodyOwner::isDisturbed());
> 
> Why not just ASSERT(!isDisturbed())?

Right, this is only an ASSERT.
I'll update the patch accordingly.

[youenn fablet]

Created attachment 277677 [details]
Patch for landing

[WebKit Commit Bot]

Comment on attachment 277677 [details]
Patch for landing

Clearing flags on attachment: 277677

Committed r200235: <http://trac.webkit.org/changeset/200235>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.