-
kocienda authored
Reviewed by Darin and Gramps Added some more test cases as part of the work to fix this bug: Radar 3136795 (type URL with ../ in it, get assertion) I also notice an unrelated change in ifnsurlextensions-test.chk where leading whitespace in a URL is now getting removed correctly. Bonus. * TestURLs.h * WebFoundation-Misc/ifnsurlextensions-test.chk * WebFoundation-Misc/ifnsurlextensions-test.m: (main) WebFoundation: Reviewed by Darin and Gramps Radar 3136795 (type URL with ../ in it, get assertion) The code to remove relative portions from URLs was eating leading slashes on URL paths. This caused absolute paths with relative portions to get mangled, resulting in a URL that the engine could not use. Now, leading slashes are no longer removed from URL paths, resulting in usuable URLs. * Misc.subproj/WebNSURLExtras.m: (pathRemovingDots) (-[NSURL _web_URLComponents]) WebCore: Reviewed by Darin and Gramps Fix for this bug: Radar 3136795 (type URL with ../ in it, get assertion) The code to remove relative portions from URLs was eating leading slashes on URL paths. This caused absolute paths with relative portions to get mangled, resulting in a URL that the engine could not use. Now, leading slashes are no longer removed from URL paths, resulting in usuable URLs. * kwq/KWQKURL.mm: (copyPathRemovingDots) git-svn-id: http://svn.webkit.org/repository/webkit/trunk@3211 268f45cc-cd09-0410-ab3c-d52691b4dbfc
kocienda authoredReviewed by Darin and Gramps Added some more test cases as part of the work to fix this bug: Radar 3136795 (type URL with ../ in it, get assertion) I also notice an unrelated change in ifnsurlextensions-test.chk where leading whitespace in a URL is now getting removed correctly. Bonus. * TestURLs.h * WebFoundation-Misc/ifnsurlextensions-test.chk * WebFoundation-Misc/ifnsurlextensions-test.m: (main) WebFoundation: Reviewed by Darin and Gramps Radar 3136795 (type URL with ../ in it, get assertion) The code to remove relative portions from URLs was eating leading slashes on URL paths. This caused absolute paths with relative portions to get mangled, resulting in a URL that the engine could not use. Now, leading slashes are no longer removed from URL paths, resulting in usuable URLs. * Misc.subproj/WebNSURLExtras.m: (pathRemovingDots) (-[NSURL _web_URLComponents]) WebCore: Reviewed by Darin and Gramps Fix for this bug: Radar 3136795 (type URL with ../ in it, get assertion) The code to remove relative portions from URLs was eating leading slashes on URL paths. This caused absolute paths with relative portions to get mangled, resulting in a URL that the engine could not use. Now, leading slashes are no longer removed from URL paths, resulting in usuable URLs. * kwq/KWQKURL.mm: (copyPathRemovingDots) git-svn-id: http://svn.webkit.org/repository/webkit/trunk@3211 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Loading