Bug 19978 – GTK port always ends up with # at the end of resource URLs (and hence can't load files)

• Status: RESOLVED
• Resolution: FIXED
• Component: Platform

[Jacob Refstrup]

r35040 changed KURL.cpp in setRef() (line 708) testing for isNull() rather than isEmpty() caused a problem on GTK where called setRef("") which ends up being not being null (but would have been empty). This currently causes all file:// references to have added a '#' at the end of the URL -- which means file:// URLs won't load.

[Jacob Refstrup]

Created attachment 22205 [details]
KURL::clearRef added and used to fix situation

Per Darin's suggestion added a clearRef() to KURL and called this from WebCore/platform/network/curl/ResourceHandleManager.cpp:558.

[Jacob Refstrup]

Created attachment 22206 [details]
KURL::clearRef added and used to fix situation

Per Darin's suggestion added a clearRef() to KURL and called this from
WebCore/platform/network/curl/ResourceHandleManager.cpp:558.

(this time with a request for a review)

[Darin Adler]

Comment on attachment 22206 [details]
KURL::clearRef added and used to fix situation

The ChangeLog has tab characters in it. That will make it impossible to check in as is. Otherwise, this looks good.

We should take a look at other callers of setRef and see if any should be changed to use this new function. Perhaps the function should be called removeRef instead of clearRef.

I think this patch is good enough as-is that I will say r=me

[Jacob Refstrup]

Created attachment 22229 [details]
Final patch.

Renamed to removeRef(); removed tabs. Will need to follow up on AccessibilityRenderObject.cpp seperately.

[Mark Rowe (bdash)]

*** Bug 19993 has been marked as a duplicate of this bug. ***

[Mark Rowe (bdash)]

Can you please refresh the patch against TOT.  A change in <http://trac.webkit.org/changeset/35111> added a removeRef method to KURL to address a similar issue inside AccessibilityRenderObject.

[Jacob Refstrup]

Created attachment 22243 [details]
Final patch (fixed against TOT)

Patched fixed against merge conflict caused by another person adding removeRef(); resolved duplicate and chose to use more streamlined version of removeRef().

[Jacob Refstrup]

Fixed against TOT.

[Darin Adler]

Comment on attachment 22243 [details]
Final patch (fixed against TOT)

r=me

[Jacob Refstrup]

Updated severity to "blocker" as gtk depends on this fix.

[Jan Alonzo]

Thanks for the patch (and Darin for the review). Landed in r35137