81122 2012-03-14 09:10:22 -0700 SVG anchor on use fails 2012-04-20 09:11:11 -0700 1 1 1 Unclassified WebKit SVG 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED http://www.thesokolows.com/colorresearch/pedigrees/AARON_OF_QUIETUDE.xhtml P2 Normal --- 1 schenney schenney rwlbuis schenney zimmermann oldest_to_newest 578466 0 schenney 2012-03-14 09:10:22 -0700 http://code.google.com/p/chromium/issues/detail?id=118181 Chrome Version : 18.0.1025.56 beta URLs (if applicable) : http://www.thesokolows.com/colorresearch/pedigrees/AARON_OF_QUIETUDE.xhtml Other browsers tested: Add OK or FAIL after other browsers where you have tested this issue: Safari 5: FAIL Firefox 4.x: OK IE 7/8/9: NA What steps will reproduce the problem? 1. From the example URL, click on the little eye symbol What is the expected result? Navigate to a picture of the horse (follow the link in the <a> element). What happens instead? The link in the containing element is followed. To aid in producing a reduction, the example URL contains this, where the "#male" bbox contains the "#eye" bbox: <use xlink:href="#couple" x="11" y="14" /> <a xlink:href="ADRIAN_ASHMORE.xhtml"> <use xlink:href= "#male" class="maleStyle nonflaxenStyle " x="11" y="14" /></a> <a xlink:href="ADRIAN_ASHMORE.xhtml"> <text font-size=".7" font-family="monospace" x="11.5" y="14.8" >ADRIAN ASHMORE</text></a> <text font-size=".6" font-family="monospace" x="15" y="15.6" >12877 -</text> <text font-size=".7" font-family="monospace" x="11.3" y="15.6" >CH</text> <a xlink:href="http://www.morganhorse.com/upload/photos/3922AdrianAshmoreNew.jpg" > <use xlink:href="#eye" x="13" y="14.9" /></a> 578497 1 zimmermann 2012-03-14 09:43:58 -0700 I think the reduction is confusing. Don't you mean <a xlink:href="outer.svg">Click <a xlink:href="inner.svg">Here</a></a> that we always navigate to outer.svg even when clicking on "Here"? 578522 2 schenney 2012-03-14 10:15:10 -0700 Yeah, my reduction is not really a reduction. Note that your anchor inside anchor is in fact illegal according to the spec. This might work, but I haven't tried yet. Still working through over 100 WebKit SVG bugs in crbug figuring out which ones are not yet in Bugzilla. <svg ...> <defs> <rect name="inner" x="10" y="10" width="20" height="20"/> </defs> <a href="outerHref"><rect x="5" y="5" width="30" height="30"/></a> <a href="innerHref"><use xlink:href="#inner"/></a> </svg> When you click at 15,15 you will apparently go to outerHref. Regardless, there is some bug that the referenced site reveals. 604323 3 rwlbuis 2012-04-17 14:04:01 -0700 Is this still a problem in trunk? 604994 4 schenney 2012-04-18 12:42:13 -0700 This does indeed seem to have been fixed. Both the original bug report test case and a layout test I constructed both work as expected in Chrome 19.0.1084.24 beta. I think I'll upload the test anyway, just for future regression testing. 605011 5 137747 schenney 2012-04-18 13:00:35 -0700 Created attachment 137747 Patch 606208 6 137747 rwlbuis 2012-04-19 17:08:48 -0700 Comment on attachment 137747 Patch Looks good. 606614 7 137747 schenney 2012-04-20 09:11:06 -0700 Comment on attachment 137747 Patch Clearing flags on attachment: 137747 Committed r114750: <http://trac.webkit.org/changeset/114750> 606615 8 schenney 2012-04-20 09:11:11 -0700 All reviewed patches have been landed. Closing bug. 137747 2012-04-18 13:00:35 -0700 2012-04-20 09:11:06 -0700 Patch bug-81122-20120418160033.patch text/plain 2312 schenney SW5kZXg6IExheW91dFRlc3RzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9D aGFuZ2VMb2cJKHJldmlzaW9uIDExNDU0NykKKysrIExheW91dFRlc3RzL0NoYW5nZUxvZwkod29y a2luZyBjb3B5KQpAQCAtMSwzICsxLDE2IEBACisyMDEyLTA0LTE4ICBTdGVwaGVuIENoZW5uZXkg IDxzY2hlbm5leUBjaHJvbWl1bS5vcmc+CisKKyAgICAgICAgU1ZHIGFuY2hvciBvbiB1c2UgZmFp bHMKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTgxMTIy CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgQWRkaW5n IGEgcmVmIHRlc3QgZm9yIHRoaXMgYnVnLiBUaGlzIHNlZW1zIHRvIGhhdmUgYmVlbiBmaXhlZCwg YnV0IGEKKyAgICAgICAgcmVncmVzc2lvbiB0ZXN0IGlzIGluIG9yZGVyLgorCisgICAgICAgICog c3ZnL2N1c3RvbS9hbmNob3Itb24tdXNlLWV4cGVjdGVkLnN2ZzogQWRkZWQuCisgICAgICAgICog c3ZnL2N1c3RvbS9hbmNob3Itb24tdXNlLnN2ZzogQWRkZWQuCisKIDIwMTItMDQtMTggIEFsbGFu IFNhbmRmZWxkIEplbnNlbiAgPGFsbGFuLmplbnNlbkBub2tpYS5jb20+CiAKICAgICAgICAgQ2xl YW4tdXAgV2hlZWxFdmVudCBDb252ZXJzaW9uLgpJbmRleDogTGF5b3V0VGVzdHMvc3ZnL2N1c3Rv bS9hbmNob3Itb24tdXNlLWV4cGVjdGVkLnN2Zwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9z dmcvY3VzdG9tL2FuY2hvci1vbi11c2UtZXhwZWN0ZWQuc3ZnCShyZXZpc2lvbiAwKQorKysgTGF5 b3V0VGVzdHMvc3ZnL2N1c3RvbS9hbmNob3Itb24tdXNlLWV4cGVjdGVkLnN2ZwkocmV2aXNpb24g MCkKQEAgLTAsMCArMSw0IEBACis8c3ZnIHhtbG5zPSJodHRwOi8vd3d3LnczLm9yZy8yMDAwL3N2 ZyIgeG1sbnM6eGxpbms9Imh0dHA6Ly93d3cudzMub3JnLzE5OTkveGxpbmsiPgorICAgIDxpbWFn ZSB4PSIwIiB5PSIwIiB3aWR0aD0iNTAzIiBoZWlnaHQ9IjQxMCIgeGxpbms6aHJlZj0icmVzb3Vy Y2VzL2dyZWVuLWNoZWNrZXIucG5nIi8+Cis8L3N2Zz4KKwpJbmRleDogTGF5b3V0VGVzdHMvc3Zn L2N1c3RvbS9hbmNob3Itb24tdXNlLnN2Zwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9zdmcv Y3VzdG9tL2FuY2hvci1vbi11c2Uuc3ZnCShyZXZpc2lvbiAwKQorKysgTGF5b3V0VGVzdHMvc3Zn L2N1c3RvbS9hbmNob3Itb24tdXNlLnN2ZwkocmV2aXNpb24gMCkKQEAgLTAsMCArMSwyMSBAQAor PHN2ZyB4bWxucz0iaHR0cDovL3d3dy53My5vcmcvMjAwMC9zdmciIHhtbG5zOnhsaW5rPSJodHRw Oi8vd3d3LnczLm9yZy8xOTk5L3hsaW5rIiBvbmxvYWQ9ImNsaWNrTGlua05vdygpIj4KKyAgICA8 c2NyaXB0PgorICAgICAgICBpZiAod2luZG93LmxheW91dFRlc3RDb250cm9sbGVyKQorICAgICAg ICAgICAgd2luZG93LmxheW91dFRlc3RDb250cm9sbGVyLndhaXRVbnRpbERvbmUoKTsKKworICAg ICAgICBmdW5jdGlvbiBjbGlja0xpbmtOb3coKSB7CisgICAgICAgICAgICBpZiAod2luZG93LmV2 ZW50U2VuZGVyKSB7CisgICAgICAgICAgICAgICAgZXZlbnRTZW5kZXIubW91c2VNb3ZlVG8oMjAs IDIwKTsKKyAgICAgICAgICAgICAgICBldmVudFNlbmRlci5tb3VzZURvd24oKTsKKyAgICAgICAg ICAgICAgICBldmVudFNlbmRlci5tb3VzZVVwKCk7CisgICAgICAgICAgICAgICAgaWYgKHdpbmRv dy5sYXlvdXRUZXN0Q29udHJvbGxlcikKKyAgICAgICAgICAgICAgICAgICAgbGF5b3V0VGVzdENv bnRyb2xsZXIubm90aWZ5RG9uZSgpOworICAgICAgICAgICAgfQorICAgICAgICB9CisgICAgPC9z Y3JpcHQ+CisgICAgPGRlZnM+CisgICAgICAgIDxyZWN0IGlkPSJyZWN0LXRvLXVzZSIgeD0iMTAi IHk9IjEwIiB3aWR0aD0iMjAiIGhlaWdodD0iMjAiIGZpbGw9ImdyZWVuIiAvPgorICAgIDwvZGVm cz4KKyAgICA8YSB4bGluazpocmVmPSJyZXNvdXJjZXMvcmVkLWNoZWNrZXIucG5nIj48cmVjdCB4 PSI1IiB5PSI1IiB3aWR0aD0iMzAiIGhlaWdodD0iMzAiIGZpbGw9InJlZCIvPjwvYT4KKyAgICA8 YSB4bGluazpocmVmPSJyZXNvdXJjZXMvZ3JlZW4tY2hlY2tlci5wbmciPjx1c2UgeGxpbms6aHJl Zj0iI3JlY3QtdG8tdXNlIiAvPjwvYT4KKzwvc3ZnPgo=