[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