Summary: | Chromium HTTP Connection Count erroneously throttled to 4. | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Mike Belshe <mbelshe> | ||||||||
Component: | WebCore Misc. | Assignee: | Mike Belshe <mbelshe> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | commit-queue | ||||||||
Priority: | P2 | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | PC | ||||||||||
OS: | Other | ||||||||||
Attachments: |
|
Description
Mike Belshe
2009-10-21 22:08:18 PDT
Created attachment 41641 [details]
patch
Simple patch to do the same thing that the mac platform does - have this function be controlled outside of webkit.
A corresponding change will be made in the chromium api to provide this function.
Comment on attachment 41641 [details] patch > Index: WebCore/ChangeLog ... > + Make the number of connections for chromium controlled by > + chromium rather than by ReosurceRequestBase. > + https://bugs.webkit.org/show_bug.cgi?id=30661 nit: ^^^ remove the tabs > Index: WebCore/platform/network/ResourceRequestBase.cpp ... > +#if !PLATFORM(MAC) && !USE(CFNETWORK) && !USE(SOUP) && !PLATFORM(CHROMIUM) > unsigned initializeMaximumHTTPConnectionCountPerHost() This method should probably be implemented in chromium/ResourceRequest.h as either defined to return 6 or defined in terms of a ChromiumBridge function call of exactly the same signature. Created attachment 41661 [details]
Patch 2; this time with the chromium implementation.
Comment on attachment 41661 [details]
Patch 2; this time with the chromium implementation.
9 No new tests. (OOPS!)
will have to be removed before commit. Our documentation isn't very good on that subject, but the OOPS will cause the commit to fail.
Better would be to replace that with a short comment explaining why this is untestable (it actually probably isn't untestable, but could be kinda tricky).
Created attachment 41674 [details]
Patch 3
Comment on attachment 41674 [details] Patch 3 Rejecting patch 41674 from commit-queue. mike@belshe.com does not have committer permissions according to http://trac.webkit.org/browser/trunk/WebKitTools/Scripts/modules/committers.py. Comment on attachment 41674 [details]
Patch 3
I can't remember who is and who is not a committer these days. But if you're a committer mike, feel free to add yourself to that file! Otherwise you can set commit-queue? and someone will come along and cq+ your patches.
cq+ing this one now.
Comment on attachment 41674 [details] Patch 3 Clearing flags on attachment: 41674 Committed r50079: <http://trac.webkit.org/changeset/50079> All reviewed patches have been landed. Closing bug. |