Bug 80997 – [GTK] Use the same DNS prefetching path than the other ports.

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKitGTK

[Sergio Villar Senin]

After https://bugs.webkit.org/show_bug.cgi?id=35589 the GTK+ port is using a different code path than the other ports for DNS pre-fetching. It was done back then because there were plans to add some features to soup_session_prepare_for_uri() that would require a full URL instead of just the host name. That work in libsoup was tracked in bug https://bugzilla.gnome.org/show_bug.cgi?id=605065.

After reviewing both patches and double checking with Dan, we agreed that it does not longer make any sense as adding more features to _prepare_for_uri() is really unlikely now.

So it looks like going back to the previous state, and thus, using the same code paths than the other ports is desirable.

[Sergio Villar Senin]

Created attachment 131670 [details]
Patch

[Sergio Villar Senin]

BTW another advantage of this "revert" is that it brings DNSSoup.cpp back to life. That would be the perfect place to host the DNS prefetching queue that will fix bug #80587  or bug #41630

[Gyuyoung Kim]

Comment on attachment 131670 [details]
Patch

Attachment 131670 [details] did not pass efl-ews (efl):
Output: http://queues.webkit.org/results/11953018

[Sergio Villar Senin]

Committed r110669: <http://trac.webkit.org/changeset/110669>

[Sergio Villar Senin]

(In reply to comment #3)
> (From update of attachment 131670 [details])
> Attachment 131670 [details] did not pass efl-ews (efl):
> Output: http://queues.webkit.org/results/11953018

I forgot to update EFL's PlatformEfl.cmake. Should be fixed in r110671