Bug 13507 – URL parsing from adress input in GdkLauncher

• Status: RESOLVED
• Resolution: FIXED
• Component: New Bugs

[Kulyk Nazar]

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.

[Kulyk Nazar]

Created attachment 14207 [details]
gdklaunch-fixurlparsing.patch

This patch should do initial parsing for url input.

[Mark Rowe (bdash)]

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.

[Kulyk Nazar]

Created attachment 14223 [details]
gdklauncher-urlparsing-0.1.patch

Initial patch for gdklauncher to parse URLs. Please check if something like this is ok.

[Kulyk Nazar]

Created attachment 14224 [details]
gdklauncher-urlparsing-0.2.patch

Fix for review

[Kulyk Nazar]

Created attachment 14225 [details]
gdklauncher-urlparsing-0.5.patch

Initial work for url correction.

[Kulyk Nazar]

Created attachment 14226 [details]
gdklauncher-urlparsing-0.7.patch

i hope this is last try :)

[Mark Rowe (bdash)]

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://.

[Mark Rowe (bdash)]

Landed in r21148 with some minor tweaks.