[webkit-reviews] review requested: [Bug 57384] CFNetwork and WebCore load priorities should match : [Attachment 88111] Patch v2
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Mon Apr 4 13:22:14 PDT 2011
David Kilzer (ddkilzer) <ddkilzer at webkit.org> has asked for review:
Bug 57384: CFNetwork and WebCore load priorities should match
https://bugs.webkit.org/show_bug.cgi?id=57384
Attachment 88111: Patch v2
https://bugs.webkit.org/attachment.cgi?id=88111&action=review
------- Additional Comments from David Kilzer (ddkilzer) <ddkilzer at webkit.org>
Made a small fix in ResourceHandleMac.mm compared to Patch v1:
if (isHTTPPipeliningEnabled()) {
- int priority =
mapResourceLoadPriorityToHTTPPipeliningPriority(m_priority);
- wkSetHTTPPipeliningPriority(nsRequest,
shouldForceHTTPPipeliningPriorityHigh() ? 2 : priority);
+ int priority =
toHTTPPipeliningPriority(shouldForceHTTPPipeliningPriorityHigh() ?
ResourceLoadPriorityHigh : m_priority);
+ wkSetHTTPPipeliningPriority(nsRequest, priority);
}
More information about the webkit-reviews
mailing list