Curl port didn't have this implementation. Now we have.
Created attachment 332199 [details] Patch
Comment on attachment 332199 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=332199&action=review > Source/WebCore/platform/network/curl/ResourceHandleCurlDelegate.cpp:190 > + m_handle->client()->didSendData(m_handle, bytesSent, totalBytesToBeSent); You might want to check these for null.
Comment on attachment 332199 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=332199&action=review >> Source/WebCore/platform/network/curl/ResourceHandleCurlDelegate.cpp:190 >> + m_handle->client()->didSendData(m_handle, bytesSent, totalBytesToBeSent); > > You might want to check these for null. Looks like cancelledOrClientless does that correct Basuke?
Null check was done in callClient utility method. It is also guarded by protectThis.
Oh, I misunderstood the point. I'm checking.
Don is correct. It was checked by cancelledOrClientless().
Comment on attachment 332199 [details] Patch Clearing flags on attachment: 332199 Committed r227591: <https://trac.webkit.org/changeset/227591>
All reviewed patches have been landed. Closing bug.
<rdar://problem/36853125>