[Webkit-unassigned] [Bug 35071] Disable loader limiting of requests per host for chromium port

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=35071





--- Comment #7 from William Chan <willchan at chromium.org>  2010-02-17 21:32:04 PST ---
(From update of attachment 48966)
> fbdbcc22020afdbf0d706ce1ecf1a4220931f436
> diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog
> index 8f6b450..b3b6281 100644
> --- a/WebCore/ChangeLog
> +++ b/WebCore/ChangeLog
> @@ -1,3 +1,15 @@
> +2010-02-17  William Chan  <willchan at chromium.org>
> +
> +        Reviewed by NOBODY (OOPS!).
> +
> +	https://bugs.webkit.org/show_bug.cgi?id=35071
> +	Disable loader limiting of requests per host for the chromium port.

Bleaugh, used tabs on accident.  I'll fix this.  Do I need to R- this patch? 
Or should I just let a reviewer do that for me?

> +
> +        No tests because we're only changing a constant.
> +
> +        * platform/network/chromium/ResourceRequest.cpp:
> +        (WebCore::initializeMaximumHTTPConnectionCountPerHost):
> +
>  2010-02-16  Simon Fraser  <simon.fraser at apple.com>
>  
>          Reviewed by Dan Bernstein.
> diff --git a/WebCore/platform/network/chromium/ResourceRequest.cpp b/WebCore/platform/network/chromium/ResourceRequest.cpp
> index 5b27c1b..016bd34 100644
> --- a/WebCore/platform/network/chromium/ResourceRequest.cpp
> +++ b/WebCore/platform/network/chromium/ResourceRequest.cpp
> @@ -31,7 +31,10 @@ namespace WebCore {
>  // This is used by the loader to control the number of issued parallel load requests. 
>  unsigned initializeMaximumHTTPConnectionCountPerHost()
>  {
> -    return 6;
> +    // The chromium network stack already handles limiting the number of
> +    // parallel requests per host, so there's no need to do it here.  Therefore,
> +    // this is set to a high value that should never be hit in practice.
> +    return 10000;
>  }
>  
>  } // namespace WebCore

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.