.
Created attachment 427994 [details] Patch
Comment on attachment 427994 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=427994&action=review > Source/WebKit/NetworkProcess/soup/NetworkDataTaskSoup.cpp:286 > + if (!g_cancellable_is_cancelled(protectCancellable.get()) && !m_networkLoadMetrics.fetchStart) { The fact that the operation can be cancelled before returning to the main loop is strange and confusing, but you do at least have a warning comment up above, so... I guess that's OK.
Comment on attachment 427994 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=427994&action=review >> Source/WebKit/NetworkProcess/soup/NetworkDataTaskSoup.cpp:286 >> + if (!g_cancellable_is_cancelled(protectCancellable.get()) && !m_networkLoadMetrics.fetchStart) { > > The fact that the operation can be cancelled before returning to the main loop is strange and confusing, but you do at least have a warning comment up above, so... I guess that's OK. I agree. It only happens when hsts cancels the load to start a https one.
Committed r277225 (237496@main): <https://commits.webkit.org/237496@main>