[webkit-reviews] review requested: [Bug 20690] Support for DNS prefetch : [Attachment 23504] Updated patch more comments from Antti

[bugzilla-daemon at webkit.org]

Collin Jackson <collinj at webkit.org> has asked  for review:
Bug 20690: Support for DNS prefetch
https://bugs.webkit.org/show_bug.cgi?id=20690

Attachment 23504: Updated patch more comments from Antti
https://bugs.webkit.org/attachment.cgi?id=23504&action=edit

------- Additional Comments from Collin Jackson <collinj at webkit.org>
(In reply to comment #8)
> I don't think it is good to have two callbacks that do essentially the same
> thing. Also didParseAnchor() tells nothing about what should be implemented
> there where as prefetchDNS() is a good name.

Ok, I got rid of didParseAnchor.

> I think relative link URLs (or actually anything that does not start with
> http:// or https://) can be ignored since the domain is the same as the
current
> document's. That would help cutting out many links on many pages. 

Ok, I've implemented this suggestion. I also added a check for scheme-relative
URLs (starting with "//"), since they are used heavily on certain sites (e.g.
Slashdot).