Summary: | URL parsing from adress input in GdkLauncher | ||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Kulyk Nazar <schamane> | ||||||||||||
Component: | New Bugs | Assignee: | Nobody <webkit-unassigned> | ||||||||||||
Status: | RESOLVED FIXED | ||||||||||||||
Severity: | Normal | CC: | mrowe | ||||||||||||
Priority: | P2 | ||||||||||||||
Version: | 523.x (Safari 3) | ||||||||||||||
Hardware: | PC | ||||||||||||||
OS: | Linux | ||||||||||||||
Attachments: |
|
Description
Kulyk Nazar
2007-04-26 11:36:17 PDT
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://.
|