Summary: | REGRESSION (Safari 5.0.5 - 5.1): No animation on svg-wow.org/text-effects/text-effects.xhtml | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Dirk Schulze <krit> | ||||
Component: | SVG | Assignee: | Nikolas Zimmermann <zimmermann> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | ap, schenney, thorton, zimmermann | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
URL: | http://svg-wow.org/text-effects/text-effects.xhtml | ||||||
Bug Depends on: | |||||||
Bug Blocks: | 41761 | ||||||
Attachments: |
|
Description
Dirk Schulze
2011-07-24 03:05:33 PDT
An exception is thrown, NOT_SUPPORTED_ERR here "result = attrVal.baseVal.getItem(0).value;" in node-svg-debug.js, line 107. I debugged using inspector and found it that this example queries a SVGTextElement grabbings its y.baseVal, which is a SVGLengthList, and calls getItem(0) to get the SVGLength object, and then value() to get a float. As it throws an exception on SVGLength.value, SVGLength couldn't resolve the unit. Quick debugging shows that: alert(this._node.getAttribute("y")); indeed returns "-0.5em". this._node has no renderer, at the point where the script tries to query the SVG DOM, that's why it can't resolve ems/exs/percentage units. Maybe we shouldn't throw though, I'll investigate. I have a fix for this, will tidy it up tomorrow. Created attachment 127151 [details]
Patch
Comment on attachment 127151 [details]
Patch
r=me
Committed r107923: <http://trac.webkit.org/changeset/107923> Committed r113043: <http://trac.webkit.org/changeset/113043> |