webgl: use immutable texture for the default FBO.

third_party/WebKit/Source/platform/graphics/gpu/DrawingBuffer.cpp

Index: third_party/WebKit/Source/platform/graphics/gpu/DrawingBuffer.cpp
diff --git a/third_party/WebKit/Source/platform/graphics/gpu/DrawingBuffer.cpp b/third_party/WebKit/Source/platform/graphics/gpu/DrawingBuffer.cpp
index c8b928e6ba2d410cba0eae8b5697184f5e3e19a7..3a62898318e34cae2523f1aaafa0e343ab0c2a05 100644
--- a/third_party/WebKit/Source/platform/graphics/gpu/DrawingBuffer.cpp
+++ b/third_party/WebKit/Source/platform/graphics/gpu/DrawingBuffer.cpp
@@ -151,6 +151,7 @@ DrawingBuffer::DrawingBuffer(PassOwnPtr<WebGraphicsContext3D> context,
     , m_multisampleExtensionSupported(multisampleExtensionSupported)
     , m_packedDepthStencilExtensionSupported(packedDepthStencilExtensionSupported)
     , m_discardFramebufferSupported(discardFramebufferSupported)
+    , m_storageTextureSupported(false)
     , m_fbo(0)
     , m_depthStencilBuffer(0)
     , m_depthBuffer(0)
@@ -305,7 +306,8 @@ bool DrawingBuffer::prepareMailbox(WebExternalTextureMailbox* outMailbox, WebExt
             m_context->discardFramebufferEXT(GL_FRAMEBUFFER, 3, attachments);
         }
     } else {
-        m_context->copyTextureCHROMIUM(GL_TEXTURE_2D, m_colorBuffer.textureId, frontColorBufferMailbox->textureInfo.textureId, GL_RGBA, GL_UNSIGNED_BYTE, GL_FALSE, GL_FALSE, GL_FALSE);
+        m_context->copySubTextureCHROMIUM(GL_TEXTURE_2D, m_colorBuffer.textureId, frontColorBufferMailbox->textureInfo.textureId,

[dshwang, 2015/11/19 12:33:00]

copyTextureCHROMIUM must not used for immutable texture as target, because it
can redefine the texture.

[Ken Russell (switch to Gerrit), 2015/11/19 21:47:41]

If this code was relying on the copyTextureCHROMIUM call to resize the texture
in the mailbox, then what's doing the same when the canvas is resized now? It
looks to me like this introduces a bug when preserveDrawingBuffer=true and the
canvas is resized, since nobody clears out the recycled mailboxes. I think you
can catch the bug by drawing to a 128x128 canvas, resizing it larger (to
256x256), re-rendering so that a different color is drawn for fragments with x
or y > 128, and reading back the lower-right corner to verify the canvas was
resized correctly. Can you please write such a test and verify that it passes?
Especially if it doesn't, could you submit a pull request to
https://github.com/KhronosGroup/WebGL under either conformance/context or
conformance/canvas adding one?

[dshwang, 2015/11/20 09:31:32]

This code doesn't rely on copyTextureCHROMIUM call to resize the texture.
copyTextureCHROMIUM emits GL_INVALID_OPERATION if the target texture is
immutable.
https://code.google.com/p/chromium/codesearch#chromium/src/gpu/command_buffer...

As this CL changes target texture to immutable, we can not use
copyTextureCHROMIUM. the source and target texture always existsm so we can use
copySubTextureCHROMIUM.

On the other hands, the test idea is good. 
WebglConformance.conformance_textures_webgl_canvas_tex_image_and_sub_image_2d_with_webgl_canvas_*
covers what you explain in discard case.
If you still think the test is needed, let me extend
WebglConformance.conformance_textures_webgl_canvas_tex_image_and_sub_image_2d_with_webgl_canvas_*
or WebglConformance.conformance_textures_misc_texture_size.

[Ken Russell (switch to Gerrit), 2015/11/23 22:26:31]

I went ahead and wrote the test I had in mind in
https://github.com/KhronosGroup/WebGL/pull/1325 . Please verify that this test
continues to pass with your CL. We should roll forward the WebGL conformance
tests to pick this up.

[dshwang, 2015/11/24 12:05:11]

Thank you for adding test! The test works well with this CL. In addition, I
checked the test cover this code path.

+            0, 0, 0, 0, m_size.width(), m_size.height(), GL_FALSE, GL_FALSE, GL_FALSE);
     }
 
     restoreFramebufferBindings();
@@ -449,6 +451,7 @@ bool DrawingBuffer::initialize(const IntSize& size)
         }
     }
     m_sampleCount = std::min(4, maxSampleCount);
+    m_storageTextureSupported = m_extensionsUtil->supportsExtension("GL_EXT_texture_storage");
 
     m_fbo = m_context->createFramebuffer();
 
@@ -959,12 +962,6 @@ void DrawingBuffer::flipVertically(uint8_t* framebuffer, int width, int height)
     }
 }
 
-void DrawingBuffer::texImage2DResourceSafe(GLenum target, GLint level, GLenum internalformat, GLsizei width, GLsizei height, GLint border, GLenum format, GLenum type, GLint unpackAlignment)
-{
-    ASSERT(unpackAlignment == 1 || unpackAlignment == 2 || unpackAlignment == 4 || unpackAlignment == 8);
-    m_context->texImage2D(target, level, internalformat, width, height, border, format, type, 0);
-}
-
 void DrawingBuffer::allocateTextureMemory(TextureInfo* info, const IntSize& size)
 {
     if (RuntimeEnabledFeatures::webGLImageChromiumEnabled()) {
@@ -977,7 +974,18 @@ void DrawingBuffer::allocateTextureMemory(TextureInfo* info, const IntSize& size
         }
     }
 
-    texImage2DResourceSafe(GL_TEXTURE_2D, 0, m_internalColorFormat, size.width(), size.height(), 0, m_colorFormat, GL_UNSIGNED_BYTE);
+    if (m_storageTextureSupported) {
+        if (info->immutable) {
+            m_context->deleteTexture(info->textureId);
+            info->textureId = createColorTexture();
+        }
+
+        // TODO(dshwang): GL_BGRA8_EXT can be better in some platforms. crbug.com/557848
+        m_context->texStorage2DEXT(GL_TEXTURE_2D, 1, m_internalRenderbufferFormat, size.width(), size.height());
+        info->immutable = true;
+        return;
+    }
+    m_context->texImage2D(GL_TEXTURE_2D, 0, m_internalColorFormat, size.width(), size.height(), 0, m_colorFormat, GL_UNSIGNED_BYTE, 0);
 }
 
 void DrawingBuffer::deleteChromiumImageForTexture(TextureInfo* info)