[webkit-reviews] review denied: [Bug 187790] FetchResponse should close its stream when loading finishes : [Attachment 345299] Patch

[bugzilla-daemon at webkit.org]

Chris Dumez <cdumez at apple.com> has denied youenn fablet <youennf at gmail.com>'s
request for review:
Bug 187790: FetchResponse should close its stream when loading finishes
https://bugs.webkit.org/show_bug.cgi?id=187790

Attachment 345299: Patch

https://bugs.webkit.org/attachment.cgi?id=345299&action=review




--- Comment #3 from Chris Dumez <cdumez at apple.com> ---
Comment on attachment 345299
  --> https://bugs.webkit.org/attachment.cgi?id=345299
Patch

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

> Source/WebCore/Modules/fetch/FetchBodyOwner.cpp:50
> +    if (m_readableStreamSource)

m_readableStreamSource gets nulled out in several methods in this class, in
which case, the source may be kept alive (due to refcounting) but will still
have a pointer to this owner. I think we want to clear the source's body owner
in all cases where m_readableStreamSource gets cleared, not used in the
BodyOwner destructor.

> Source/WebCore/Modules/fetch/FetchBodySource.cpp:89
> +void FetchBodySource::cleanBodyOwner()

I do not like "clean", I'd prefer "clear" or "detachFrom"