14167 2007-06-15 07:41:29 -0700 event listeners on <use> elements wrongly return evt.target as an SVGElementInstance 2007-06-20 07:07:43 -0700 1 1 1 Unclassified WebKit SVG 523.x (Safari 3) Mac OS X 10.4 RESOLVED INVALID P2 Normal --- 1 bugzilla webkit-unassigned ahayes bugzilla oldest_to_newest 7439 0 bugzilla 2007-06-15 07:41:29 -0700 When a <use> element is accessed via javascript, its ownerDocument attribute is found undefined. 7437 1 15054 bugzilla 2007-06-15 07:42:36 -0700 Created attachment 15054 Test Case 7414 2 bugzilla 2007-06-15 10:36:05 -0700 This is probably related to Bug 14170. It appears that there is no DOM access from the javascript bindings when it comes to a <use> element. 7418 3 eric 2007-06-15 10:37:29 -0700 You're correct, this is a bug. We're incorrectly returning an SVGElementInstance, even though the listeners were set on the <use> itself, and not on the referenced content. It would be interesting to see a test which also set the listeners on the <rect>. I think we'd then show the proper behavior (returning an SVGElementInstance, which does not actually expose any ownerDocument interface). 7405 4 bugzilla 2007-06-15 10:50:01 -0700 Eric, this is a good question. If the mouse events were set on the <rect> definition as opposed to the <use> element (I must admit, I had never considered this possibility), what would the event target be? I always assumed that the event target would be the rendered element (in this case, the <use> element), but I am not sure if I am right. Another case would be if the mouse events were set on a parent <g> element. During bubbling up, the event would be sent to the parent's event handler, but the event target would remain the rendered element (the <use> element). Or, at least, that is my interpretation of the situation. What do you think the proper behaviour should be? 7395 5 bugzilla 2007-06-15 13:27:45 -0700 Re-reading the specs, I now understand better your comment, Eric. Thanks for the clarification. http://www.w3.org/TR/SVG/struct.html#UseElement "The effect of a 'use' element is as if the contents of the referenced element were deeply cloned into a separate non-exposed DOM tree which had the 'use' element as its parent and all of the 'use' element's ancestors as its higher-level ancestors." The "deep cloning" aspect leads me to believe that the ownerDocument attribute should be set to the receiving document. Also, on the related bug 14170, a cloned node should be able to support DOM methods, such as getAttribute(). 7027 6 bugzilla 2007-06-20 07:07:43 -0700 Thanks so much for the help. Yes, Safari is specs compliant. 15054 2007-06-15 07:42:36 -0700 2007-06-15 07:42:36 -0700 Test Case index.svg image/svg+xml 2508 bugzilla PD94bWwgdmVyc2lvbj0iMS4wIj8+CjwhRE9DVFlQRSBzdmcgUFVCTElDICItLy9XM0MvL0RURCBT VkcgMjAwMTA5MDQvL0VOIiAiaHR0cDovL3d3dy53My5vcmcvVFIvMjAwMS9SRUMtU1ZHLTIwMDEw OTA0L0RURC9zdmcxMC5kdGQiPgo8c3ZnIAoJeG1sbnM9Imh0dHA6Ly93d3cudzMub3JnLzIwMDAv c3ZnIgoJeG1sbnM6eGxpbms9Imh0dHA6Ly93d3cudzMub3JnLzE5OTkveGxpbmsiCgl2aWV3Qm94 PSIwIDAgNzAgNzAiIAoJb25sb2FkPSJpbml0KGV2dCkiCgk+Cgk8c2NyaXB0IHR5cGU9InRleHQv amF2YXNjcmlwdCI+CgkJPCFbQ0RBVEFbCgkJCXZhciBteURvYyA9IG51bGw7CgkJCWZ1bmN0aW9u IGluaXQoZXZ0KSB7CgkJCQlteURvYyA9IGV2dC50YXJnZXQub3duZXJEb2N1bWVudDsKCQkJfQoJ CQkKCQkJZnVuY3Rpb24gbW91c2VPdmVyKGV2dCkgewoJCQkJc2V0VGV4dChteURvYy5nZXRFbGVt ZW50QnlJZCgnbGFiZWxfZXZ0JyksJycrZXZ0KTsKCQkJCXNldFRleHQobXlEb2MuZ2V0RWxlbWVu dEJ5SWQoJ2xhYmVsX2V2dF90YXJnZXQnKSwnJytldnQudGFyZ2V0KTsKCQkJCXNldFRleHQobXlE b2MuZ2V0RWxlbWVudEJ5SWQoJ2xhYmVsX2V2dF90YXJnZXRfb3duZXJEb2N1bWVudCcpLCcnK2V2 dC50YXJnZXQub3duZXJEb2N1bWVudCk7CgkJCX0KCQkJCgkJCWZ1bmN0aW9uIG1vdXNlT3V0KGV2 dCkgewoJCQkJc2V0VGV4dChteURvYy5nZXRFbGVtZW50QnlJZCgnbGFiZWxfZXZ0JyksJycpOwoJ CQkJc2V0VGV4dChteURvYy5nZXRFbGVtZW50QnlJZCgnbGFiZWxfZXZ0X3RhcmdldCcpLCcnKTsK CQkJCXNldFRleHQobXlEb2MuZ2V0RWxlbWVudEJ5SWQoJ2xhYmVsX2V2dF90YXJnZXRfb3duZXJE b2N1bWVudCcpLCcnKTsKCQkJfQoJCQkKCQkJZnVuY3Rpb24gc2V0VGV4dChlbGVtLHN0cikgewoJ CQkJdmFyIGNoaWxkcmVuID0gZWxlbS5jaGlsZE5vZGVzOwoJCQkJdmFyIGxvb3A7CgkJCQlmb3Io bG9vcD0wO2xvb3A8Y2hpbGRyZW4ubGVuZ3RoOysrbG9vcCkgewoJCQkJCWVsZW0ucmVtb3ZlQ2hp bGQoY2hpbGRyZW4uaXRlbShsb29wKSk7CgkJCQl9CgkJCQoJCQkJZWxlbS5hcHBlbmRDaGlsZCgg ZWxlbS5vd25lckRvY3VtZW50LmNyZWF0ZVRleHROb2RlKHN0cikgKTsKCQkJfQoJCV1dPgoJPC9z Y3JpcHQ+Cgk8ZGVmcz4KCQk8cmVjdAoJCQlpZD0icmVjdDEiCgkJCXdpZHRoPSIyMCIgaGVpZ2h0 PSIyMCIKCQkvPgoJPC9kZWZzPgoJPHJlY3QKCQl4PSIxMCIgeT0iMTAiIHdpZHRoPSIyMCIgaGVp Z2h0PSIyMCIKCQlmaWxsPSIjY2NjY2NjIgoJCXN0cm9rZS13aWR0aD0iMSIKCQlzdHJva2U9IiMw MDAwMDAiCgkJb25tb3VzZW92ZXI9Im1vdXNlT3ZlcihldnQpIgoJCW9ubW91c2VvdXQ9Im1vdXNl T3V0KGV2dCkiCgkvPgoJPHVzZSAKCQl4bGluazpocmVmPSIjcmVjdDEiCgkJeD0iNTAiIHk9IjEw IgoJCWZpbGw9IiNjY2NjY2MiCgkJc3Ryb2tlLXdpZHRoPSIxIgoJCXN0cm9rZT0iIzAwMDAwMCIK CQlvbm1vdXNlb3Zlcj0ibW91c2VPdmVyKGV2dCkiCgkJb25tb3VzZW91dD0ibW91c2VPdXQoZXZ0 KSIKCQkvPgoJPGcgdHJhbnNmb3JtPSJtYXRyaXgoMC4yIDAgMCAwLjIgMTAgNDUpIj4KCQk8dGV4 dCBmb250LWZhbWlseT0iJ0NvdXJpZXInIiBmb250LXNpemU9IjkiPgoJCQk8dHNwYW4geD0iMCIg eT0iLTQ1Ij5ldnQ6PC90c3Bhbj48dHNwYW4gaWQ9ImxhYmVsX2V2dCIgeD0iMTUwIiB5PSItNDUi PjwvdHNwYW4+CgkJCTx0c3BhbiB4PSIwIiB5PSItMzAiPmV2dC50YXJnZXQ6PC90c3Bhbj48dHNw YW4gaWQ9ImxhYmVsX2V2dF90YXJnZXQiIHg9IjE1MCIgeT0iLTMwIj48L3RzcGFuPgoJCQk8dHNw YW4geD0iMCIgeT0iLTE1Ij5ldnQudGFyZ2V0Lm93bmVyRG9jdW1lbnQ6PC90c3Bhbj48dHNwYW4g aWQ9ImxhYmVsX2V2dF90YXJnZXRfb3duZXJEb2N1bWVudCIgeD0iMTUwIiB5PSItMTUiPjwvdHNw YW4+CgkJCTx0c3BhbiB4PSIwIiB5PSIwIj5Gb3IgdGhpcyB0ZXN0IGNhc2UgdG8gYmUgc3VjY2Vz c2Z1bCwgYm90aCBzcXVhcmVzPC90c3Bhbj4KCQkJPHRzcGFuIHg9IjAiIHk9IjE1Ij5tdXN0IHJl YWN0IHRvIHRoZSAnbW91c2Ugb3ZlcicgZXZlbnQgaWRlbnRpY2FsbHkgYW5kPC90c3Bhbj4KCQkJ PHRzcGFuIHg9IjAiIHk9IjMwIj5yZXBvcnQgdGhlIGV2ZW50IHByb3BlcnRpZXMuIE9uZSB0aGUg bGVmdCwgdGhlIHNxdWFyZSBpczwvdHNwYW4+CgkJCTx0c3BhbiB4PSIwIiB5PSI0NSI+YSA8IVtD REFUQVs8cmVjdD5dXT4gZWxlbWVudCwgd2hpbGUgb24gdGhlIHJpZ2h0IGl0IGlzIGE8L3RzcGFu PgoJCQk8dHNwYW4geD0iMCIgeT0iNjAiPjwhW0NEQVRBWzx1c2U+XV0+IGVsZW1lbnQuIEluIFNh ZmFyaSBCZXRhIDMsIHRoZSB0YXJnZXQgPC90c3Bhbj4KCQkJPHRzcGFuIHg9IjAiIHk9Ijc1Ij5v d25lciBkb2N1bWVudCBpcyBub3Qgc2V0LCBhcyBpdCBzaG91bGQgYmUuPC90c3Bhbj4KCQkJPHRz cGFuIHg9IjAiIHk9IjkwIj5CdWcgMTQxNjc8L3RzcGFuPgoJCTwvdGV4dD4KCTwvZz4KPC9zdmc+