Progress towards implementing downloads with NetworkSession
Created attachment 266657 [details] Patch
Created attachment 266661 [details] Patch
Comment on attachment 266661 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=266661&action=review > Source/WebKit2/NetworkProcess/NetworkSession.h:-106 > - ~NetworkSession() { ASSERT(m_dataTaskMap.isEmpty()); } This ASSERT seems to be missing from the new ~NetworkSession. Intentional? > Source/WebKit2/NetworkProcess/cocoa/NetworkSessionCocoa.mm:226 > + [m_session invalidateAndCancel]; The ASSERT I mentioned above seems to be missing here. Intentional?
(In reply to comment #3) > Comment on attachment 266661 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=266661&action=review > > > Source/WebKit2/NetworkProcess/NetworkSession.h:-106 > > - ~NetworkSession() { ASSERT(m_dataTaskMap.isEmpty()); } > > This ASSERT seems to be missing from the new ~NetworkSession. Intentional? Yes, intentional. That is because I was running into destroyed network sessions that had data tasks still, so I call [m_session invalidateAndCancel]
Comment on attachment 266661 [details] Patch Clearing flags on attachment: 266661 Committed r193787: <http://trac.webkit.org/changeset/193787>
All reviewed patches have been landed. Closing bug.