Bug 63705 - SVGAnimatedLengthListAnimator does not take additive="sum" into accout
Summary: SVGAnimatedLengthListAnimator does not take additive="sum" into accout
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: SVG (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Dirk Schulze
URL:
Keywords:
Depends on:
Blocks: 41761
  Show dependency tree / graph
 
Reported: 2011-06-30 05:54 PDT by Dirk Schulze
Modified: 2011-07-01 01:51 PDT (History)
1 user (show)

See Also:

Attachments
Patch (20.24 KB, patch)
2011-06-30 06:22 PDT, Dirk Schulze 		no flags Details | Formatted Diff | Diff
Patch (17.14 KB, patch)
2011-06-30 06:38 PDT, Dirk Schulze 		zimmermann: review+
Details | Formatted Diff | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Dirk Schulze 2011-06-30 05:54:37 PDT 
SVGAnimatedLengthListAnimator does not take additive="sum" into accout.
Comment 1 Dirk Schulze 2011-06-30 06:22:18 PDT 
Created attachment 99285 [details]
Patch
Comment 2 Dirk Schulze 2011-06-30 06:38:33 PDT 
Created attachment 99290 [details]
Patch
Comment 3 Nikolas Zimmermann 2011-06-30 07:50:05 PDT 
Comment on attachment 99290 [details]
Patch

Looks great, it seems to move you solved your length problem? What was the specific trick to get it working?
Comment 4 Dirk Schulze 2011-06-30 08:12:05 PDT 
(In reply to comment #3)
> (From update of attachment 99290 [details])
> Looks great, it seems to move you solved your length problem? What was the specific trick to get it working?

Yes. additive sum is useable with one animation as well. Was a bit confused. :-P See http://www.w3.org/TR/SVG/animate.html#AdditionAttributes
Comment 5 Nikolas Zimmermann 2011-06-30 12:10:12 PDT 
Comment on attachment 99290 [details]
Patch

Oops, I wanted to set r+.
Comment 6 Dirk Schulze 2011-07-01 01:51:52 PDT 
Committed r90219: <http://trac.webkit.org/changeset/90219>