Bug 28401 - Add support for DNS prefetching with CFNetwork
Summary: Add support for DNS prefetching with CFNetwork
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Page Loading (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac OS X 10.5
: P2 Enhancement
Assignee: Alexey Proskuryakov
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-08-17 11:22 PDT by Alexey Proskuryakov
Modified: 2009-08-17 15:09 PDT (History)
0 users

See Also:

Attachments
proposed patch (2.03 KB, patch)
2009-08-17 11:26 PDT, Alexey Proskuryakov 		ggaren: review+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Alexey Proskuryakov 2009-08-17 11:22:37 PDT 
prefetchDNS() in DNSCFNet.cpp is not implemented.
Comment 1 Alexey Proskuryakov 2009-08-17 11:26:36 PDT 
Created attachment 34981 [details]
proposed patch
Comment 2 Geoffrey Garen 2009-08-17 12:07:48 PDT 
Comment on attachment 34981 [details]
proposed patch

r=me

>  54     host.releaseRef(); // The host will be released from callback.

I'd change this to "The host will be released in clientCallback".
Comment 3 Alexey Proskuryakov 2009-08-17 15:09:17 PDT 
Committed revision 47391.

Made one post-review change: with a null callback info, CFHostStartInfoResolution() works synchronously, even if the client has been set.