Bug 1349102 - Making read back buffer use the default framebuffer when doing snapshot; r?jgilbert draft
authorDaosheng Mu <daoshengmu@gmail.com>
Mon, 10 Apr 2017 17:45:46 +0800
changeset 561352 bf555a23e601e4fad5444e06f678e825903d9a77
parent 559365 45692c884fdd5136a64fb2f8a61a0c8183b69331
child 623953 8f9ecc9ed24de50103ffb91a6a2deefec455bd01
push id53708
push userbmo:dmu@mozilla.com
push dateWed, 12 Apr 2017 15:19:54 +0000
reviewersjgilbert
bugs1349102
milestone55.0a1
Bug 1349102 - Making read back buffer use the default framebuffer when doing snapshot; r?jgilbert MozReview-Commit-ID: EGaopHztAex
dom/canvas/WebGLContext.cpp
--- a/dom/canvas/WebGLContext.cpp
+++ b/dom/canvas/WebGLContext.cpp
@@ -1954,22 +1954,22 @@ WebGLContext::GetSurfaceSnapshot(bool* o
     {
         ScopedBindFramebuffer autoFB(gl, 0);
         ClearBackbufferIfNeeded();
 
         // Save, override, then restore glReadBuffer.
         const GLenum readBufferMode = gl->Screen()->GetReadBufferMode();
 
         if (readBufferMode != LOCAL_GL_BACK) {
-            gl->fReadBuffer(LOCAL_GL_BACK);
+            gl->Screen()->SetReadBuffer(LOCAL_GL_BACK);
         }
         ReadPixelsIntoDataSurface(gl, surf);
 
         if (readBufferMode != LOCAL_GL_BACK) {
-            gl->fReadBuffer(readBufferMode);
+            gl->Screen()->SetReadBuffer(readBufferMode);
         }
     }
 
     if (out_premultAlpha) {
         *out_premultAlpha = true;
     }
     bool srcPremultAlpha = mOptions.premultipliedAlpha;
     if (!srcPremultAlpha) {