[Webkit-unassigned] [Bug 12350] Add calcMode support to SVG Animation

[bugzilla-daemon at webkit.org]

http://bugs.webkit.org/show_bug.cgi?id=12350


macdome at opendarwin.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #12677|                            |review?
               Flag|                            |
             Status|NEW                         |ASSIGNED
         AssignedTo|webkit-                     |macdome at opendarwin.org
                   |unassigned at lists.webkit.org |
  Attachment #12660|0                           |1
        is obsolete|                            |




------- Comment #7 from macdome at opendarwin.org  2007-01-26 02:05 PDT -------
Created an attachment (id=12677)
 --> (http://bugs.webkit.org/attachment.cgi?id=12677&action=view)
final patch, with test cases

I'm not sure if anyone in particular is particularly qualified to review this. 
WildFox originally wrote this code.  This is a total re-write, so mostly I
expect folks to sanity check the individual *Distance classes, and give a
rubber stamp to the "oh, that architecture more-or-less looks sane".  I'm happy
to talk about any of these changes over IRC.  The resulting state of animation
is certainly better than TOT.  calcMode, keyTimes work (which they didn't
before).

Some thoughts on what remains broken:
keySplines is a few lines of code away from working (code to find a point on a
line given an x value).  freeze and repeat accumulations do not work (but I'm
not sure they did before either).  The underlying animVal/baseVal system is
still broken.  <animate> can't really be wired in correctly until a system for
handling animated mapped attributes, and animated styles is devised.


-- 
Configure bugmail: http://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
You are the assignee for the bug, or are watching the assignee.