Bug 113485

Summary: Hang when calling setCurrentTime on SVG with cyclic animation dependency chain
Product: WebKit Reporter: David Vest <davve>
Component: SVGAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: d-r, fmalita, pdr, schenney, webkit.review.bot, zimmermann
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Linux   
Attachments:
Description Flags
Test
none
Patch
none
Patch
none
Patch none

Description David Vest 2013-03-28 00:57:50 PDT
Created attachment 195496 [details]
Test

Discovered when writing tests for <https://bugs.webkit.org/show_bug.cgi?id=93528>.
Comment 1 David Vest 2013-03-28 07:41:00 PDT
Created attachment 195565 [details]
Patch
Comment 2 David Vest 2013-03-28 10:36:23 PDT
Created attachment 195595 [details]
Patch
Comment 3 David Vest 2013-04-02 01:37:55 PDT
Created attachment 196102 [details]
Patch
Comment 4 Stephen Chenney 2013-04-02 07:31:57 PDT
Comment on attachment 196102 [details]
Patch

Good catch. R=me. Let me know ASAP if you need me to land this.
Comment 5 David Vest 2013-04-02 07:52:50 PDT
Comment on attachment 196102 [details]
Patch

I've now added "cq: ?" to the patch, as you instructed me in bug 93528. It would be great if you could help me land it.

I realize now that I possibly should have done so at the same time as adding "r: ?"?
Comment 6 WebKit Review Bot 2013-04-02 08:16:59 PDT
Comment on attachment 196102 [details]
Patch

Clearing flags on attachment: 196102

Committed r147434: <http://trac.webkit.org/changeset/147434>
Comment 7 WebKit Review Bot 2013-04-02 08:17:03 PDT
All reviewed patches have been landed.  Closing bug.