[Webkit-unassigned] [Bug 225510] [SOUP] Use the new libsoup network metrics API

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sat May 8 03:30:34 PDT 2021


https://bugs.webkit.org/show_bug.cgi?id=225510

--- Comment #3 from Carlos Garcia Campos <cgarcia at igalia.com> ---
Comment on attachment 427994
  --> https://bugs.webkit.org/attachment.cgi?id=427994
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.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20210508/b0130be7/attachment.htm>


More information about the webkit-unassigned mailing list