[webkit-reviews] review canceled: [Bug 57384] CFNetwork and WebCore load priorities should match : [Attachment 88087] Patch v1

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 canceled David Kilzer
(ddkilzer) <ddkilzer at webkit.org>'s request for review:
Bug 57384: CFNetwork and WebCore load priorities should match
https://bugs.webkit.org/show_bug.cgi?id=57384

Attachment 88087: Patch v1
https://bugs.webkit.org/attachment.cgi?id=88087&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