RESOLVED FIXED89538
[GTK] [WebKit1] Schedule next 60FPS sync before rendering contents to the window in AcceleratedCompositingContextGL 
https://bugs.webkit.org/show_bug.cgi?id=89538
Summary [GTK] [WebKit1] Schedule next 60FPS sync before rendering contents to the win...
Changwan Hong
Reported 2012-06-19 18:11:49 PDT
Refine syncLayersTimeoutCallback for Accelerated Compositing.
Attachments
Patch (2.02 KB, patch)
2012-06-19 18:12 PDT, Changwan Hong 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Changwan Hong
Comment 1 2012-06-19 18:12:13 PDT
Created attachment 148478 [details] Patch
Martin Robinson
Comment 2 2012-06-19 20:13:29 PDT
Comment on attachment 148478 [details] Patch Does this have some observable change in behavior?
Changwan Hong
Comment 3 2012-06-19 21:06:52 PDT
(In reply to comment #2) > (From update of attachment 148478 [details]) > Does this have some observable change in behavior? If rendering takes long time (Contents are complex or performance of hardware is low like as embedded system), it delays timer so it can not draw in correct time.
Martin Robinson
Comment 4 2012-06-19 21:30:18 PDT
(In reply to comment #3) > If rendering takes long time (Contents are complex or performance of hardware is low like as embedded system), it delays timer so it can not draw in correct time. I guess what I mean is are you actually seeing this happen?
Changwan Hong
Comment 5 2012-06-19 21:38:06 PDT
(In reply to comment #4) > (In reply to comment #3) > > > If rendering takes long time (Contents are complex or performance of hardware is low like as embedded system), it delays timer so it can not draw in correct time. > > I guess what I mean is are you actually seeing this happen? Yes. I saw that. I tested it on Exynos.
WebKit Review Bot
Comment 6 2012-06-19 22:51:24 PDT
Comment on attachment 148478 [details] Patch Clearing flags on attachment: 148478 Committed r120803: <http://trac.webkit.org/changeset/120803>
WebKit Review Bot
Comment 7 2012-06-19 22:51:29 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.