Bug 82237

Summary: [chromium] assertion being hit in CCLayerAnimationController
Product: WebKit Reporter: Shawn Singh <shawnsingh>
Component: Layout and RenderingAssignee: vollick
Status: RESOLVED FIXED    
Severity: Normal CC: cc-bugs, danakj, enne, jamesr, nduca, vollick, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

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.