https://issues.apache.org/bugzilla/show_bug.cgi?id=46791
--- Comment #7 from Cameron McCormack <[email protected]> 2009-03-03 16:08:21 PST --- (In reply to comment #5) > On that point, Cameron - should the initialization of > the live attributes be tied to the booting of the SVG > and CSS DOM? That would be better, since you could then avoid constructing the various SVGLength, etc. objects which wouldn't be useful without the SVG DOM being initialised anyway. Before animation was implemented, these objects were lazily created. So it'd be good to go back to something like that. (In reply to comment #6) > the initializeAllLiveAttributes method is still called by the deepCopyInto > method of SVGOMElement. When the newNode() method of BindableElement used the > parameterized constructor, the initializeAllLiveAttributes method would > actually have been called twice, if that first call had not caused the NPE due > to the owner document not being set yet. I only took a very quick look, and I think Daniel is correct. The pattern of overloading those four methods to ensure some extra data is copied across to the cloned node is also used in SVGOMElement. -- Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the assignee for the bug. --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
