Bug 108854

Summary: [Qt][WK2] Improve loading time by pre-fetching DNS lookups
Product: WebKit Reporter: Viatcheslav Ostapenko <ostap73>
Component: WebKit2Assignee: Viatcheslav Ostapenko <ostap73>
Status: RESOLVED INVALID    
Severity: Normal CC: abecsi, ap, cmarcelo, hausmann, menard, rich, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch buildbot: commit-queue-

Comment 1 Viatcheslav Ostapenko 2013-02-04 13:38:25 PST
Created attachment 186450 [details]
Patch
Comment 2 Richard Moore 2013-02-04 13:57:27 PST
Please don't submit the patch as-is and just review the concept. I'll add an api to QHostInfo specifically for this since you get a warning otherwise.
Comment 3 Richard Moore 2013-02-04 14:41:14 PST
The change to Qt to make this patch work without warnings is now up for review at https://codereview.qt-project.org/#change,46776
Comment 4 Build Bot 2013-02-04 17:15:04 PST
Comment on attachment 186450 [details]
Patch

Attachment 186450 [details] did not pass win-ews (win):
Output: http://queues.webkit.org/results/16365536
Comment 5 Alexey Proskuryakov 2013-02-04 17:32:36 PST
This is already implemented in Chrome::mouseDidMoveOverElement(). Is this version in client code different?
Comment 6 Simon Hausmann 2013-02-05 00:34:13 PST
I think Alexey is right. The existing code issues the prefetch request through the correct layer, which in the current design for the Qt port will result in a DNS lookup in the web process, which is also where we currently do networking. This patch on the other hand will issue a DNS request in the ui process, populating the in-process QHostInfoCache in the wrong process.