Bug 708124 - Remove unnecessary glFlushs from Layers - r=bjacob

gfx/layers/basic/BasicLayers.cpp

@@ -1140,10 +1140,6 @@ BasicCanvasLayer::UpdateSurface(gfxASurface* aDestSurface)
 
     NS_ASSERTION(isurf->Stride() == mBounds.width * 4, "gfxImageSurface stride isn't what we expect!");
 
-    // We have to flush to ensure that any buffered GL operations are
-    // in the framebuffer before we read.
-    mGLContext->fFlush();
-
     PRUint32 currentFramebuffer = 0;
 
     mGLContext->fGetIntegerv(LOCAL_GL_FRAMEBUFFER_BINDING, (GLint*)&currentFramebuffer);

gfx/layers/d3d10/CanvasLayerD3D10.cpp

@@ -181,10 +181,6 @@ CanvasLayerD3D10::UpdateSurface()
 
     mGLContext->MakeCurrent();
 
-    // We have to flush to ensure that any buffered GL operations are
-    // in the framebuffer before we read.
-    mGLContext->fFlush();
-
     PRUint32 currentFramebuffer = 0;
 
     mGLContext->fGetIntegerv(LOCAL_GL_FRAMEBUFFER_BINDING, (GLint*)&currentFramebuffer);

gfx/layers/d3d9/CanvasLayerD3D9.cpp

@@ -119,10 +119,6 @@ CanvasLayerD3D9::UpdateSurface()
 
     mGLContext->MakeCurrent();
 
-    // We have to flush to ensure that any buffered GL operations are
-    // in the framebuffer before we read.
-    mGLContext->fFlush();
-
     PRUint32 currentFramebuffer = 0;
 
     mGLContext->fGetIntegerv(LOCAL_GL_FRAMEBUFFER_BINDING, (GLint*)&currentFramebuffer);