We have two different paths to do the same, when building without OpenGL, the VideoRenderRequestScheduler has its own timer and we need to add ifdefs to handle that. But when building with OpenGL we still support non-accelerated rendering that does the same than the VideoRenderRequestScheduler timer, but using the media player timer instead. We can simplify the code and remove more ifdefs by using the media player timer and mutex in all cases for non-accelerated rendering.
Created attachment 312012 [details]
Comment on attachment 312012 [details]
Nice! r=me, conditional on Miguel's approval.
Looks good and usual use cases work :)
I built without OpenGL yesterday to try it out.
Committed r217824: <http://trac.webkit.org/changeset/217824>