[webkit-reviews] review granted: [Bug 203423] VeryHigh priority loads are actually loading at VeryLow priority : [Attachment 382249] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Oct 30 02:04:05 PDT 2019


Antti Koivisto <koivisto at iki.fi> has granted Ben Nham <nham at apple.com>'s
request for review:
Bug 203423: VeryHigh priority loads are actually loading at VeryLow priority
https://bugs.webkit.org/show_bug.cgi?id=203423

Attachment 382249: Patch

https://bugs.webkit.org/attachment.cgi?id=382249&action=review




--- Comment #7 from Antti Koivisto <koivisto at iki.fi> ---
Comment on attachment 382249
  --> https://bugs.webkit.org/attachment.cgi?id=382249
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=382249&action=review

> Source/WebCore/platform/network/cf/ResourceRequestCFNet.h:43
> +    // TODO(nham): switch VeryLow back to 0 priority when CFNetwork fixes
<rdar://problem/56621205>

We usually just say FIXME: without a name. Anyone is free to fix problems and
if the FIXME originator is important it can be found with blame.


More information about the webkit-reviews mailing list