[Webkit-unassigned] [Bug 23846] [GTK]Enable DNS prefetching

[bugzilla-daemon at webkit.org]

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





--- Comment #7 from Holger Freyther <zecke at selfish.org>  2009-10-13 18:48:53 PDT ---
(From update of attachment 41109)

>          Reviewed by Simon Hausmann.
> Index: WebCore/page/Chrome.cpp
> ===================================================================
> --- WebCore/page/Chrome.cpp	(revision 49502)
> +++ WebCore/page/Chrome.cpp	(working copy)
> @@ -311,8 +311,11 @@ void Chrome::mouseDidMoveOverElement(con
>  {
>      if (result.innerNode()) {
>          Document* document = result.innerNode()->document();
> -        if (document && document->isDNSPrefetchEnabled())
> -            prefetchDNS(result.absoluteLinkURL().host());
> +        if (document && document->isDNSPrefetchEnabled()) {
> +            String host = result.absoluteLinkURL().host();
> +            if (!host.isEmpty())
> +                prefetchDNS(host);
> +        }
>      }

If you move the check from prefetchDNS to here you will have to update the
existing prefetchDNS implementations too to not make them check the string
twice.

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