Don't ask the driver about FB attachment color encoding. draft
authorjdashg <jdashg+github@gmail.com>
Thu, 17 Dec 2015 16:16:56 -0800
changeset 316125 9b37c002eba27ff769345b81b4afb955400efe9c
parent 316124 da9945f4715cf103134da3e08871fcd5bf1cec80
child 512131 07f308c41e4a571cd91933e72c445eb49343c714
push id8514
push userjgilbert@mozilla.com
push dateFri, 18 Dec 2015 00:24:33 +0000
milestone45.0a1
Don't ask the driver about FB attachment color encoding.
dom/canvas/WebGLFramebuffer.cpp
--- a/dom/canvas/WebGLFramebuffer.cpp
+++ b/dom/canvas/WebGLFramebuffer.cpp
@@ -404,17 +404,19 @@ WebGLFBAttachPoint::GetParameter(const c
         MOZ_ASSERT(attachment != LOCAL_GL_DEPTH_STENCIL_ATTACHMENT);
         isPNameValid = true;
         break;
 
     case LOCAL_GL_FRAMEBUFFER_ATTACHMENT_COLOR_ENCODING:
         if (webgl->IsWebGL2() ||
             webgl->IsExtensionEnabled(WebGLExtensionID::EXT_sRGB))
         {
-            isPNameValid = true;
+            const auto format = Format()->format;
+            return JS::Int32Value(format->isSRGB ? LOCAL_GL_SRGB
+                                                 : LOCAL_GL_LINEAR);
         }
         break;
 
     ////////////////
 
     case LOCAL_GL_FRAMEBUFFER_ATTACHMENT_TEXTURE_LEVEL:
         if (mTexturePtr)
             return JS::Int32Value(MipLevel());