125515 2013-12-10 07:28:45 -0800 [WK2] [SOUP] Asserts in DocumentLoader::dataReceived() if the network process is enabled 2014-01-07 06:30:12 -0800 1 1 1 Unclassified WebKit WebKit2 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 108832 1 berto webkit-unassigned cgarcia oldest_to_newest 958362 0 berto 2013-12-10 07:28:45 -0800 Here's the backtrace: ASSERTION FAILED: !m_response.isNull() ../../Source/WebCore/loader/DocumentLoader.cpp(819) : virtual void WebCore::DocumentLoader::dataReceived(WebCore::CachedResource*, const char*, int) 1 0x7fef70de3014 /home/berto/devel/WebKit/build/.libs/libjavascriptcoregtk-3.0.so.0(WTFCrash+0x1e) [0x7fef70de3014] 2 0x7fef6be911af /home/berto/devel/WebKit/build/.libs/libwebkit2gtk-3.0.so.25(_ZN7WebCore14DocumentLoader12dataReceivedEPNS_14CachedResourceEPKci+0xe9) [0x7fef6be911af] 3 0x7fef6be6ff87 /home/berto/devel/WebKit/build/.libs/libwebkit2gtk-3.0.so.25(_ZN7WebCore17CachedRawResource28notifyClientsDataWasReceivedEPKcj+0x79) [0x7fef6be6ff87] 4 0x7fef6be6fd1c /home/berto/devel/WebKit/build/.libs/libwebkit2gtk-3.0.so.25(_ZN7WebCore17CachedRawResource13addDataBufferEPNS_14ResourceBufferE+0xce) [0x7fef6be6fd1c] 5 0x7fef6bef4c79 /home/berto/devel/WebKit/build/.libs/libwebkit2gtk-3.0.so.25(_ZN7WebCore17SubresourceLoader22didReceiveDataOrBufferEPKciN3WTF10PassRefPtrINS_12SharedBufferEEExNS_15DataPayloadTypeE+0x1f1) [0x7fef6bef4c79] 6 0x7fef6bef4a17 /home/berto/devel/WebKit/build/.libs/libwebkit2gtk-3.0.so.25(_ZN7WebCore17SubresourceLoader14didReceiveDataEPKcixNS_15DataPayloadTypeE+0x51) [0x7fef6bef4a17] 7 0x7fef6b5a9ba6 /home/berto/devel/WebKit/build/.libs/libwebkit2gtk-3.0.so.25(_ZN6WebKit17WebResourceLoader14didReceiveDataERKN7CoreIPC13DataReferenceEl+0xea) [0x7fef6b5a9ba6] [...] The problem is that DocumentLoader::responseReceived() is never reached. We do handle->client()->didReceiveResponse() in ResourceHandleSoup, but that's a no-op since NetworkResourceLoader does not implement that method. 958431 1 berto 2013-12-10 11:30:54 -0800 Actually NetworkResourceLoader used to implement that but it was renamed to didReceiveResponseAsync() in r147952. Either way, we also assert in ResourceHandle::continueDidReceiveResponse() via AsynchronousNetworkLoaderClient::didReceiveResponse(). 961491 2 cgarcia 2013-12-19 12:11:20 -0800 Patch in bug https://bugs.webkit.org/show_bug.cgi?id=126006 probably fixes these asserts too. 964984 3 berto 2014-01-07 06:30:12 -0800 Fixed after bug 126006