[Webkit-unassigned] [Bug 93146] Coordinated Graphics: Enable threaded animations

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=93146





--- Comment #3 from Noam Rosenthal <noam.rosenthal at nokia.com>  2012-10-16 21:06:17 PST ---
(In reply to comment #2)
> Looks good! This patch makes CSS animation smoother.
> 
> However, I have few questions.
> 
> 1. Why did you decide Web process makes progress animations instead of UI process. If we pass a GraphicsLayer::addAnimation message to UI process, LayerTreeRenderer can make progress animations like PageClientQWidget::syncLayers() or AcceleratedCompositingContext::renderLayersToWindow(). I think the later approach can reuse existing code and reduce animation IPC messages and may make css animations slightly faster than threaded animations.

Yes, that's they way it was before. The problem (which is solvable, but tricky) is in synchronization.
We've had cases where, for example, an animated layer stops being composited, and then it gets repainted with the rest of the "non-composited" content.
The issue is that if painting the non-composited contents takes too much time, there might be a few extra animation frames in the UI process, which would go beyond the end point of the animation. If we can fix that, we can bring back UI-side animations...

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.