273235 - Support XML Events also in SVG

Status: RESOLVED FIXED

(Core :: DOM: Events, defect)

Description

[Olli Pettay [:smaug][[e-mail]]]

patch coming

Comment 1

[Olli Pettay [:smaug][[e-mail]]]

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]]]

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]]]

Attachment: simple example

Comment 6

[Olli Pettay [:smaug][[e-mail]]]

Attachment: fix v2

Perhaps tor has time for a review.

Comment 7

[Olli Pettay [:smaug][[e-mail]]]

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