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

[bugzilla-daemon at webkit.org]

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


Huang Dongsung <luxtella at company100.net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |luxtella at company100.net




--- Comment #2 from Huang Dongsung <luxtella at company100.net>  2012-10-16 19:25:26 PST ---
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.

2. I think you may decide threaded animations in Web process because of accurate callback for requestAnimationFrame. Can't Messages::LayerTreeCoordinator::RenderNextFrame() satisfy the requirements for requestAnimationFrame?

I'm looking forward Coordinated Graphics running animations smoothly regardless of Web process's main thread :)

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