Use existing code for redirects when using NETWORK_SESSION
Created attachment 267213 [details] Patch
Comment on attachment 267213 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=267213&action=review > Source/WebKit2/NetworkProcess/NetworkLoad.cpp:169 > sharedWillSendRedirectedRequest(request, response); > - completionHandler(request); > + ASSERT(!m_redirectCompletionHandler); > + m_redirectCompletionHandler = completionHandler; I need to set m_redirectCompletionHandler before calling sharedWillSendRedirectedRequest
This fixes http/tests/cookies/set-cookie-on-redirect.html rdar://problem/23860624
Created attachment 267248 [details] Patch
Comment on attachment 267248 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=267248&action=review > Source/WebKit2/NetworkProcess/NetworkLoad.cpp:107 > + m_task->cancel(); Incorrect indentation? > Source/WebKit2/NetworkProcess/NetworkLoad.h:125 > + RedirectCompletionHandler m_redirectCompletionHandler { nullptr }; std::function doesn't need nullptr initialization to be null
https://trac.webkit.org/changeset/194043