it cause a lot of curl error if you request not full URL. Reproduce: start gdklauncher and input some url in to adressline, like : osnews.com without "http://" it opens page, but fails on downloading css and images.
Created attachment 14207 [details] gdklaunch-fixurlparsing.patch This patch should do initial parsing for url input.
Comment on attachment 14207 [details] gdklaunch-fixurlparsing.patch Please take a look at the WebKit coding style guidelines at <http://webkit.org/coding/coding-style.html>. Your change is not consistent with many of the guidelines there, nor with the style used elsewhere in the file you modified. As for the substance of the change, I'm not sure that I understand the need for many of the cases in the is_url function. It seems to me that the logic should be to trim whitespace from the beginning/end of the string, check whether the string starts with a protocol that WebKit can handle and if not assume it's intended to be HTTP and prepend http:// to the string. I can't see the point in checking for irc:/ftp: nor irc./ftp. at this point as there's no support for either of those protocols in WebKit. It seems to me that this patch does nothing to address the original report: entering osnews.com results in GTKURL_HOST being returned from is_url, which leads to osnews.com being passed through to FrameLoader::load as happens with the existing code.
Created attachment 14223 [details] gdklauncher-urlparsing-0.1.patch Initial patch for gdklauncher to parse URLs. Please check if something like this is ok.
Created attachment 14224 [details] gdklauncher-urlparsing-0.2.patch Fix for review
Created attachment 14225 [details] gdklauncher-urlparsing-0.5.patch Initial work for url correction.
Created attachment 14226 [details] gdklauncher-urlparsing-0.7.patch i hope this is last try :)
Comment on attachment 14226 [details] gdklauncher-urlparsing-0.7.patch I'm going to say r=me, but there is one minor change I will make when I land this: I'll allow https:// and file:// URLs through unchanged in the same manner as http:// and ftp://.
Landed in r21148 with some minor tweaks.