Bug 89846 - fill=freeze animations cannot be rewound
Summary: fill=freeze animations cannot be rewound
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: SVG (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Philip Rogers
URL:
Keywords:
Depends on:
Blocks: 89687
  Show dependency tree / graph
 
Reported: 2012-06-24 20:00 PDT by Philip Rogers
Modified: 2012-06-25 03:31 PDT (History)
3 users (show)

See Also:

Attachments
Testcase (1.48 KB, text/html)
2012-06-24 20:00 PDT, Philip Rogers 		no flags Details
Fix rewinding of SVG animations (4.44 KB, patch)
2012-06-24 20:16 PDT, Philip Rogers 		no flags Details | Formatted Diff | Diff
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 Philip Rogers 2012-06-24 20:00:47 PDT 
Created attachment 149226 [details]
Testcase

If an SVG animation containing fill=freeze is rewound using setTimeout(), we fail to reset to the base animated value.

Because of how we handle accumulating multiple animations, this can only be hit if there are at least 2 animations.

See the attached testcase for an example.
Comment 1 Philip Rogers 2012-06-24 20:16:31 PDT 
Created attachment 149228 [details]
Fix rewinding of SVG animations
Comment 2 Nikolas Zimmermann 2012-06-25 02:36:34 PDT 
Comment on attachment 149228 [details]
Fix rewinding of SVG animations

Woha, great catch! r=me.
Comment 3 WebKit Review Bot 2012-06-25 03:31:22 PDT 
Comment on attachment 149228 [details]
Fix rewinding of SVG animations

Clearing flags on attachment: 149228

Committed r121146: <http://trac.webkit.org/changeset/121146>
Comment 4 WebKit Review Bot 2012-06-25 03:31:26 PDT 
All reviewed patches have been landed.  Closing bug.