Bug 34550 – REGRESSION (r51644): WebCore/manual-tests/linkjump-1.html fails

• Status: RESOLVED
• Resolution: FIXED
• Component: Page Loading

[Brady Eidson]

http://trac.webkit.org/changeset/51644 caused manual-tests/linkjump to fail.

Previously, fragment scrolls didn't actually update a Document's URL.  After r51644, they did, causing improper relative URL calculation when clicking on anchors that only contain an (empty) fragment identifier.

Turns out that when computing a new URL with a base and relative URL, KURL didn't correctly handle the case of an empty fragment ID.

A comment in the relevant code correctly called out the fact that the RFC notes that an empty reference relative to an absolute URL means the new URL refers to the same document, but left out the fact that it refers to the same document *with any existing fragment identifier removed*

One line fix + layout tests are coming up.

[Brady Eidson]

<rdar://problem/7595694>

[Brady Eidson]

Created attachment 48088 [details]
1 line fix, updated comment, and a veritable cornucopia of layout tests

[Alexey Proskuryakov]

Comment on attachment 48088 [details]
1 line fix, updated comment, and a veritable cornucopia of layout tests 

r=me

Do these tests cover cases that used to be covered by manual tests? If not, can the latter be converted to automated ones, as well?

[Brady Eidson]

These tests definitely (largely) overlap with the original manual-tests/linkjump-1.html test using "EventSender Technology™" unavailable in the dark ages, and I suspect they cover the original manual test.

If you think it should be removed, I'd like to do that in a separate patch.

[Brady Eidson]

http://trac.webkit.org/changeset/54321

[Alexey Proskuryakov]

>If you think it should be removed, I'd like to do that in a separate patch.

It's nice to reduce the number of manual tests when possible.

[Brady Eidson]

rs=you on removing it and logging that commit here?  :)