Bug 61078

Summary: Use toHTTPPipeliningPriority() in initializeMaximumHTTPConnectionCountPerHost()
Product: WebKit Reporter: David Kilzer (:ddkilzer) <ddkilzer>
Component: Page LoadingAssignee: David Kilzer (:ddkilzer) <ddkilzer>
Status: RESOLVED FIXED    
Severity: Normal CC: joepeck, psolanki
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 57384    
Bug Blocks:    
Attachments:
Description Flags
Patch v1
none
Patch v2 joepeck: review+

Description David Kilzer (:ddkilzer) 2011-05-18 10:17:17 PDT
Instead of passing a ResourceLoadPriority directly into wkSetHTTPPipeliningMaximumPriority(), we should convert it using Use toHTTPPipeliningPriority().

This should have been done in r82900 for Bug 57384.  <http://trac.webkit.org/changeset/82900>

The method is in Source/WebCore/platform/network/cf/ResourceRequestCFNet.cpp.
Comment 1 David Kilzer (:ddkilzer) 2011-05-18 10:38:29 PDT
Created attachment 93933 [details]
Patch v1
Comment 2 Joseph Pecoraro 2011-05-18 10:40:01 PDT
Comment on attachment 93933 [details]
Patch v1

r=me. Thanks!
Comment 3 David Kilzer (:ddkilzer) 2011-05-18 11:02:12 PDT
Comment on attachment 93933 [details]
Patch v1

We should also fix wkSetHTTPPipeliningMinimumFastLanePriority() as well while we're here.
Comment 4 David Kilzer (:ddkilzer) 2011-05-18 11:09:47 PDT
Created attachment 93939 [details]
Patch v2
Comment 5 Joseph Pecoraro 2011-05-18 11:14:27 PDT
Comment on attachment 93939 [details]
Patch v2

Oh, excellent. Good catch.
Comment 6 David Kilzer (:ddkilzer) 2011-05-18 11:34:57 PDT
Committed r86778: <http://trac.webkit.org/changeset/86778>