[Webkit-unassigned] [Bug 52868] [chromium] Fix redundant video frame paint on CSS LayerChromium for <video>

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Jan 26 18:48:41 PST 2011


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





--- Comment #6 from Victoria Kirst <vrk at chromium.org>  2011-01-26 18:48:41 PST ---
Huh, there seems to be an easy fix for this!

So, normal video paint calls (i.e. originating from <video> software rendering) use the "paint()" method to render frames in a canvas. It seems like Canvas/WebGL use paintCurrentFrameIntoContext(), which by default just calls paint(). But of course we could just override it to behave slightly differently.

This patch has paintCurrentFrameIntoContext() do a software-paint into the given context blindly, while paint() does a check first to make sure we're not re-doing the work of VideoLayerChromium. Seems to work but I need to run layout tests to be confident, will update with results.

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



More information about the webkit-unassigned mailing list