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+

David Kilzer (:ddkilzer)
Reported 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.
Attachments
Patch v1 (1.69 KB, patch)
2011-05-18 10:38 PDT, David Kilzer (:ddkilzer) 		no flags
DetailsFormatted DiffDiff
Patch v2 (2.03 KB, patch)
2011-05-18 11:09 PDT, David Kilzer (:ddkilzer) 		joepeck: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
David Kilzer (:ddkilzer)
Comment 1 2011-05-18 10:38:29 PDT
Created attachment 93933 [details] Patch v1
Joseph Pecoraro
Comment 2 2011-05-18 10:40:01 PDT
Comment on attachment 93933 [details] Patch v1 r=me. Thanks!
David Kilzer (:ddkilzer)
Comment 3 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.
David Kilzer (:ddkilzer)
Comment 4 2011-05-18 11:09:47 PDT
Created attachment 93939 [details] Patch v2
Joseph Pecoraro
Comment 5 2011-05-18 11:14:27 PDT
Comment on attachment 93939 [details] Patch v2 Oh, excellent. Good catch.
David Kilzer (:ddkilzer)
Comment 6 2011-05-18 11:34:57 PDT
Committed r86778: <http://trac.webkit.org/changeset/86778>
Note You need to log in before you can comment on or make changes to this bug.