273235 - Support XML Events also in SVG

Status: RESOLVED FIXED

(Core :: DOM: Events, defect)

Description

[Olli Pettay [:smaug][[e-mail]](vacation until Nov 16)]

patch coming

Comment 1

[Olli Pettay [:smaug][[e-mail]](vacation until Nov 16)]

Attachment: fix

Note, this does not add support for 
SVG 1.2 modifications to XML events.
http://www.w3.org/TR/SVG12/events.html#handler-element

Comment 2

[Hixie (not reading bugmail)]

Shouldn't this just be automatic for all elements? Why does each namespace have
to be special-cased? Does MathML need this? or HTML?

Comment 3

[Olli Pettay [:smaug][[e-mail]](vacation until Nov 16)]

Yes, it should be "automatic" for all elements, but unfortunately we have several
SetAttr() implementations. And yes, (X)HTML does have this already, so does 
generic XML (so XML Events do work with any XTF based elements and also with
MathML).

Comment 4

[Boris Zbarsky [:bzbarsky]]

Why mNodeinfo->GetDocument()?  Why not just call GetOwnerDoc()?

Comment 5

[Olli Pettay [:smaug][[e-mail]](vacation until Nov 16)]

Attachment: simple example

Comment 6

[Olli Pettay [:smaug][[e-mail]](vacation until Nov 16)]

Attachment: fix v2

Perhaps tor has time for a review.

Comment 7

[Olli Pettay [:smaug][[e-mail]](vacation until Nov 16)]

Attachment: this one

Comment 8

[Hixie (not reading bugmail)]

Comment on attachment 171540 [details]
simple example

This is an awesome document.

Comment 9

[tor]

Checked in.

Checking in nsSVGElement.cpp;
/cvsroot/mozilla/content/svg/content/src/nsSVGElement.cpp,v  <--  nsSVGElement.cpp
new revision: 1.71; previous revision: 1.70
done