Bug 150035 – [Curl] Increase limit of parallel network requests.

• Status: RESOLVED
• Resolution: FIXED
• Component: WebCore Misc.

If the limit is low, other network requests will often be blocked until active requests finishes. This can affect performance in a negative way.

Created attachment 262902 [details]
Patch

[Alex Christensen]

Comment on attachment 262902 [details]
Patch

What if the limit is too big?  Why is there a limit?

(In reply to comment #2)
> Comment on attachment 262902 [details]
> Patch
> 
> What if the limit is too big?  Why is there a limit?

Good point. It might be better to leave this decision to Curl, since it is probably handled well there. In that case, I dont't think a big limit would hurt :)

It seems there already is a Curl option for this (with default 0, meaning infinite number of connections):

http://curl.haxx.se/libcurl/c/CURLMOPT_MAX_TOTAL_CONNECTIONS.html

Since Curl doesn't have a limit here, maybe we shouldn't either? If we limit the number of connections, I think the limit should be higher that it is now at least.

Another problem with the limit is that it can effectively block all future network requests if the current connections are never closed (for some reason).

Thanks for reviewing :)

[WebKit Commit Bot]

Comment on attachment 262902 [details]
Patch

Clearing flags on attachment: 262902

Committed r190887: <http://trac.webkit.org/changeset/190887>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.