[Webkit-unassigned] [Bug 147824] NetworkProcess: DNS prefetch happens in the Web Process

[bugzilla-daemon at webkit.org]

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

Alexey Proskuryakov <ap at webkit.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #258612|review?                     |review-
              Flags|                            |

--- Comment #3 from Alexey Proskuryakov <ap at webkit.org> ---
Comment on attachment 258612
  --> https://bugs.webkit.org/attachment.cgi?id=258612
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=258612&action=review

Thank you for tackling this, it's good cleanup! I think that this need to be updated to not use strategies though.

> Source/WebCore/html/HTMLAnchorElement.cpp:259
> +                    platformStrategies()->loaderStrategy()->prefetchDNS(document().completeURL(parsedURL).host());

We are gradually getting rid of strategies in favor of client calls. Strategies were needed for the chromium port, and even then, I wasn't quite sure why.

> Source/WebCore/loader/LoaderStrategy.h:31
>  #include <wtf/Vector.h>
> +#include <wtf/text/WTFString.h>

Can probably use wtf/Forward.h here.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-unassigned/attachments/20150810/41fe38cf/attachment.html>