Last Comment Bug 698195 - Use AfterSetAttr to process script href changes
: Use AfterSetAttr to process script href changes
Status: RESOLVED FIXED
:
Product: Core
Classification: Components
Component: SVG (show other bugs)
: Trunk
: All All
: -- normal (vote)
: mozilla11
Assigned To: Robert Longson
:
Mentors:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2011-10-29 08:05 PDT by Robert Longson
Modified: 2011-11-19 05:08 PST (History)
1 user (show)
longsonr: in‑testsuite-
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
patch (2.51 KB, patch)
2011-10-29 08:06 PDT, Robert Longson
dholbert: review+
Details | Diff | Review

Description Robert Longson 2011-10-29 08:05:51 PDT

    
Comment 1 Robert Longson 2011-10-29 08:06:34 PDT
Created attachment 570478 [details] [diff] [review]
patch
Comment 2 Daniel Holbert [:dholbert] 2011-10-29 10:45:53 PDT
Comment on attachment 570478 [details] [diff] [review]
patch

I take it theree's no way to change the target of <script xlink:href="..."> other than setAttributeNS?

I know it's not animatable, and it doesn't seem to be exposed as an attribute on the DOM node.

Assuming that there's no other way to do it (or all ways end up triggering AfterSetAttr), then r=me.
Comment 3 Robert Longson 2011-11-19 00:58:04 PST
You can change it with SVG DOM but you still end up in AfterSetAttr.
Comment 5 Ed Morley [:emorley] 2011-11-19 05:08:11 PST
https://hg.mozilla.org/mozilla-central/rev/64103eac30cf

Note You need to log in before you can comment on or make changes to this bug.