Bug 19995 – hixie.ch : anchor Link fails with fragment identifiers inside SVG fragments.

• Component: SVG
• Status: RESOLVED FIXED

[Calvin Anderson]

I. Steps:
 -----------
 
 1. Launch Safari
 2. Go to the URL http://www.hixie.ch/tests/adhoc/svg/links/003.xml
 
 
 
 II. Issue:
 -----------------
 The test fails, When you click on Blue circle, will take you near to the Red circle.
 Expected - When you click, shouldn't do anything.
 
 
 
 
 III. Other browsers:
 -----------------------
 IE7 : not working
 FF3 : not working: The window scrolls downwards
 Opera 9.27: ok
 
 IV. Safari nightly tested: version 3.1.1(525.17 )- r34388.
 
 Not working properly on Safari.
 
 
 
 V. Safari screenshot : Avalible

[Calvin Anderson]

Created attachment 22231 [details]
Screenshot

[Calvin Anderson]

Created attachment 22232 [details]
reduction.zip

[Nikolas Zimmermann]

Changed component to SVG, so it shows up in my all-svg-bugs search.

[Rob Buis]

Created attachment 94863 [details]
Patch

[Dirk Schulze]

Comment on attachment 94863 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=94863&action=review

r- because of a missing valid DRT test

> LayoutTests/svg/hixie/links/003.xml:24
> +  <svg xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink"
> +       height="20000" width="200">
> +   <!-- Spec says that <a> can only link to animation elements,
> +        otherwise it is invalid; that known attributes with
> +        unsupported values are treated as if they hadn't been
> +        specified when rendering; that if the attribute is not
> +        specified, the effect is as if an empty value ("") was
> +        specified; and that an empty attribute value (xlink:href="")
> +        means that no link traversal occurs upon activation of the 'a'
> +        element. So. Nothing should happen if you click it: -->
> +   <a xlink:href="#test">
> +    <circle cx="50" cy="50" r="50" fill="blue"/>
> +   </a>

Can't you write an automatic test with mousedown and check if "test" has the focus() afterwards?

[Rob Buis]

Created attachment 95035 [details]
Patch

[Nikolas Zimmermann]

Comment on attachment 95035 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=95035&action=review

Good catch, r=me.

> LayoutTests/svg/hixie/links/003.xml:17
> +  window.layoutTestController.waitUntilDone();
> +  window.layoutTestController.dumpAsText();

You can omit window. here, it's in the default lookup chain, not in the if() line though.

> LayoutTests/svg/hixie/links/003.xml:25
> +    window.layoutTestController.notifyDone();

Ditto.

[Rob Buis]

Committed r87518: <http://trac.webkit.org/changeset/87518>