The copyPreviousContentsToBackSurfaceOfTile() call will also modify buffer contents, move it after the blit generation condvar to make sure the GPU is done with it before modifying the buffer.
Created attachment 154033 [details] Patch
Hmm, looking at the patch again, it doesn't make sense to move the call that gets the buffer pointer, because the whole condvar business could be replaced by a method that takes the buffer pointer and performs the synchronization to encapsulate the mechanism.
Created attachment 154035 [details] Patch
This is a pretty minimal patch =) Internally reviewed by Jakob Petsovits, what do you think?
Comment on attachment 154035 [details] Patch Rejecting attachment 154035 [details] from commit-queue. Failed to run "['/mnt/git/webkit-commit-queue/Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '-..." exit_code: 2 Last 500 characters of output: Kit/chromium/third_party/yasm/source/patched-yasm --revision 134927 --non-interactive --force --accept theirs-conflict --ignore-externals' in '/mnt/git/webkit-commit-queue/Source/WebKit/chromium' 48>At revision 134927. ________ running '/usr/bin/python tools/clang/scripts/update.py --mac-only' in '/mnt/git/webkit-commit-queue/Source/WebKit/chromium' ________ running '/usr/bin/python gyp_webkit' in '/mnt/git/webkit-commit-queue/Source/WebKit/chromium' Updating webkit projects from gyp files... Full output: http://queues.webkit.org/results/13345220
OOPS, I said OOPS in the Changelog!
Created attachment 154348 [details] Patch
Comment on attachment 154348 [details] Patch Same patch, this time without OOPS. If I understand correctly, only CQ+ is needed now, since it's already reviewied before.
Comment on attachment 154348 [details] Patch Clearing flags on attachment: 154348 Committed r123617: <http://trac.webkit.org/changeset/123617>
All reviewed patches have been landed. Closing bug.