Bug 81350 - SVG setCurrentTime with pauseAnimations bug.
Summary: SVG setCurrentTime with pauseAnimations bug.
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:
 
Reported: 2012-03-16 07:40 PDT by Philip Rogers
Modified: 2012-06-26 03:06 PDT (History)
4 users (show)

See Also:

Attachments
Repro case (1.42 KB, text/html)
2012-03-16 16:10 PDT, Philip Rogers 		no flags Details
Fix setCurrentTime for paused animations (4.13 KB, patch)
2012-06-25 21:47 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-03-16 07:40:40 PDT 
Unconfirmed defect report from user: when I call setCurrentTime(0), a call to unpauseAnimations() afterwards introduces a 1s pause.

May be related to: https://bugs.webkit.org/show_bug.cgi?id=12073
Comment 1 Philip Rogers 2012-03-16 16:10:50 PDT 
Created attachment 132410 [details]
Repro case

Attaching a small repro case: it looks like calling setCurrentTime while paused is broken.
Comment 2 Raks 2012-06-04 06:21:50 PDT 
Actually its not 1s. If in the code setCurrentTime() and unPauseAnimation() are called together means they are in consecutive lines of code then the animation starts after a time which is the time difference between the call to pauseAnimation() and unPauseAnimation(). Pretty strange !!
Comment 3 Raks 2012-06-04 06:25:04 PDT 
Hi Joel, Can you take a look at this ? Maybe it is part of BUG 12073
Comment 4 Philip Rogers 2012-06-04 06:28:27 PDT 
(In reply to comment #3)
> Hi Joel, Can you take a look at this ? Maybe it is part of BUG 12073

I don't think Joel is able to look at this so it is up for grabs :)

Also, a good starter bug for new webkittens.
Comment 5 Philip Rogers 2012-06-25 21:47:16 PDT 
Created attachment 149448 [details]
Fix setCurrentTime for paused animations
Comment 6 Nikolas Zimmermann 2012-06-26 02:44:42 PDT 
Comment on attachment 149448 [details]
Fix setCurrentTime for paused animations

pdr++, r=me.
Comment 7 WebKit Review Bot 2012-06-26 03:06:49 PDT 
Comment on attachment 149448 [details]
Fix setCurrentTime for paused animations

Clearing flags on attachment: 149448

Committed r121246: <http://trac.webkit.org/changeset/121246>
Comment 8 WebKit Review Bot 2012-06-26 03:06:54 PDT 
All reviewed patches have been landed.  Closing bug.