Bug 119289

Summary: Correct the elapsedTime calculation in SVG animations
Product: WebKit Reporter: Ryosuke Niwa <rniwa>
Component: SVGAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, dino, d-r, fmalita, krit, pdr, schenney, simon.fraser, thorton, zimmermann
Priority: P2 Keywords: BlinkMergeCandidate
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Ryosuke Niwa
Reported 2013-07-30 16:18:11 PDT
Consider merging https://chromium.googlesource.com/chromium/blink/+/338f9badca7fb7880abdb0cecd5f02493c1f7d2e As per the SVG standard, SVG document on receiving the pauseAnimations() should also pause the animation clock corresponding to the SVG document. Whereas current implmentation also takes into account the accumulated pause time which is not required. So replacing the m_accumulatedPauseTime with m_accumulatedActiveTime and other corresponding calculations. Added the layout Test.
Attachments
Patch (7.80 KB, patch)
2013-09-13 05:58 PDT, youenn fablet 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
youenn fablet
Comment 1 2013-09-13 05:58:21 PDT
Created attachment 211547 [details] Patch
Brent Fulgham
Comment 2 2013-10-31 14:52:45 PDT
Comment on attachment 211547 [details] Patch r=me
WebKit Commit Bot
Comment 3 2013-10-31 16:21:23 PDT
Comment on attachment 211547 [details] Patch Clearing flags on attachment: 211547 Committed r158405: <http://trac.webkit.org/changeset/158405>
WebKit Commit Bot
Comment 4 2013-10-31 16:21:25 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.