Bug 108854 - [Qt][WK2] Improve loading time by pre-fetching DNS lookups
Summary: [Qt][WK2] Improve loading time by pre-fetching DNS lookups
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit2 (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Viatcheslav Ostapenko
Depends on:
Reported: 2013-02-04 13:30 PST by Viatcheslav Ostapenko
Modified: 2013-02-11 14:52 PST (History)
7 users (show)

See Also:

Patch (1.73 KB, patch)
2013-02-04 13:38 PST, Viatcheslav Ostapenko
buildbot: commit-queue-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Comment 1 Viatcheslav Ostapenko 2013-02-04 13:38:25 PST
Created attachment 186450 [details]
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]

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.