[webkit-reviews] review granted: [Bug 64263] libsoup-CRITICAL **: soup_message_io_pause: assertion `io != NULL' failed : [Attachment 109401] Patch

[bugzilla-daemon at webkit.org]

Xan Lopez <xan.lopez at gmail.com> has granted Martin Robinson
<mrobinson at webkit.org>'s request for review:
Bug 64263: libsoup-CRITICAL **: soup_message_io_pause: assertion `io != NULL'
failed
https://bugs.webkit.org/show_bug.cgi?id=64263

Attachment 109401: Patch
https://bugs.webkit.org/attachment.cgi?id=109401&action=review

------- Additional Comments from Xan Lopez <xan.lopez at gmail.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=109401&action=review


> Source/WebCore/platform/network/soup/ResourceHandleSoup.cpp:-708
> -    if (d->m_finished || soupMessage->status_code == SOUP_STATUS_NONE)

It's not immediately obvious why it's OK to remove the status_code check, so
please add a comment in the ChangeLog about it.