Closed Bug 992731 Opened 7 years ago Closed 7 years ago
Element .get Total Length from returning Na N
2.65 KB, image/svg+xml
328 bytes, image/svg+xml
1.29 KB, patch
|Details | Diff | Splinter Review|
285.34 KB, image/png
Steps To Reproduce: 1. Open http://cmx.io/edit/sample.html Actual Results: a single green line is drawn And an error shown: Error: TypeError: Argument 1 of SVGPathElement.getPointAtLength is not a finite floating-point value. Source File: http://cmx.io/v/0.1/cmx.js Line: 22654 Expected Results: Cartoon should be drawn http://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=2c4c218a7a09&tochange=f853e8ff85ea Regressed by: f853e8ff85ea Bas Schouten — Bug 987077: Correctly deal with a constant acceleration. r=jwatt
Bas, Can you have a look at this?
jwatt, can you come up with a minimal testcase for me?
Flags: needinfo?(bas) → needinfo?(jwatt)
Jonathan, could you have a look here? I can still reproduce it with today aurora. Thanks
Summary: SVG cartoon fails to draw with Error: TypeError: Argument 1 of SVGPathElement.getPointAtLength is not a finite floating-point value. → Stop SVGPathElement.getTotalLength from returning NaN
(I can reproduce in 64-bit Linux nightly builds. OS-->All, version-->Trunk)
OS: Windows 7 → All
Version: 31 Branch → Trunk
Assignee: nobody → bas
Status: NEW → ASSIGNED
Attachment #8431918 - Flags: review?(jwatt)
This kind of thing should really have a test case when it's checked in.
I have a testcase in my queue on the Moz2D repo, to land once I do a merge from m-c to moz2d.
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla32
There several differences comparing the drawings in chrome vs nightly 32.0a1 (2014-06-02). Also the values returned by the reduced testcases are different. Does it actually matter?
Thanks, Paul -- I see the same results (much too tiny box around "One Lazy Morning" in Nightly). Firefox release version matches Chrome, with a normal-sized border-box. Probably best to track that in a separate bug, though. Mind filing?
Done - bug 1020255
Bas, can we get an uplift request for 31? Thanks.
You need to log in before you can comment on or make changes to this bug.