RESOLVED FIXED70560
[chromium] Route onSwapBuffersComplete from LayerRenderer to CCScheduler 
https://bugs.webkit.org/show_bug.cgi?id=70560
Summary [chromium] Route onSwapBuffersComplete from LayerRenderer to CCScheduler
Nat Duca
Reported 2011-10-20 15:55:25 PDT
[chromium] Route onSwapBuffersComplete from LayerRenderer to CCScheduler
Attachments
Patch (27.06 KB, patch)
2011-10-20 15:57 PDT, Nat Duca 		jamesr: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Nat Duca
Comment 1 2011-10-20 15:57:46 PDT
Created attachment 111863 [details] Patch
Nat Duca
Comment 2 2011-10-20 15:58:40 PDT
I also took the liberty of getting rid of word "present", which is a directx-ism that crept into our code and really means swapbuffers.
James Robinson
Comment 3 2011-10-20 16:18:11 PDT
Comment on attachment 111863 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=111863&action=review R=me, cool > Source/WebCore/platform/graphics/chromium/LayerRendererChromium.h:76 > +class LayerRendererChromium { you haz an extra space here > Source/WebCore/platform/graphics/chromium/cc/CCLayerTreeHostImpl.h:54 > + nit: extra newline
Nat Duca
Comment 4 2011-10-20 16:36:33 PDT
Mucked up changelog, ~sigh~ Committed r98039: <http://trac.webkit.org/changeset/98039>
Note You need to log in before you can comment on or make changes to this bug.