RESOLVED FIXED 82237
[chromium] assertion being hit in CCLayerAnimationController 
https://bugs.webkit.org/show_bug.cgi?id=82237
Summary [chromium] assertion being hit in CCLayerAnimationController
Shawn Singh
Reported 2012-03-26 13:24:25 PDT
Approximately line 244, CCLayerAnimationController::pushNewAnimationsToImplThread(...) ASSERT(m_activeAnimations[i]0>needsSynchronizedStartTime()) I'm hitting this assertion on desktop aura. To reproduce: 1. open poster circle, apple iphone, gmail (with --force-compositing-mode) 2. switch tabs a few times. At some point when switching to poster circle, it triggers this assert and crashes the page.
Attachments
Patch (6.09 KB, patch)
2012-03-30 08:51 PDT, vollick 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Shawn Singh
Comment 1 2012-03-26 13:52:46 PDT
Dana did you mean to dupe this one instead?
Dana Jansens
Comment 2 2012-03-26 13:58:51 PDT
Nope this is different, I didn't hear of this before. Ian may have to look at this.
vollick
Comment 3 2012-03-30 08:51:31 PDT
Created attachment 134821 [details] Patch When pushing 'new' animations to the impl thread, the old code did not consider that the animation may be missing on the impl thread because it has already completed. In that case, the old animation would have already notified that it had started and the main thread animation would no longer need a synchronized start time.
James Robinson
Comment 4 2012-03-30 10:37:32 PDT
Comment on attachment 134821 [details] Patch Cool, R=me
WebKit Review Bot
Comment 5 2012-03-30 11:39:58 PDT
Comment on attachment 134821 [details] Patch Clearing flags on attachment: 134821 Committed r112701: <http://trac.webkit.org/changeset/112701>
WebKit Review Bot
Comment 6 2012-03-30 11:40:02 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.