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-

Attachments
Patch (1.73 KB, patch)
2013-02-04 13:38 PST, Viatcheslav Ostapenko 		buildbot: commit-queue-
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Viatcheslav Ostapenko
Comment 1 2013-02-04 13:38:25 PST
Created attachment 186450 [details] Patch
Richard Moore
Comment 2 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.
Richard Moore
Comment 3 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
Build Bot
Comment 4 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
Alexey Proskuryakov
Comment 5 2013-02-04 17:32:36 PST
This is already implemented in Chrome::mouseDidMoveOverElement(). Is this version in client code different?
Simon Hausmann
Comment 6 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.
Note You need to log in before you can comment on or make changes to this bug.