Add support for compressed GPU memory buffers.

ui/gl/gl_image_memory.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ui/gl/gl_image_memory.h"
 
 #include "base/logging.h"
 #include "base/trace_event/trace_event.h"
 #include "ui/gl/gl_bindings.h"
 #include "ui/gl/scoped_binders.h"
@@ skipping 10 matching lines @@
   switch (internalformat) {
     case GL_RGBA:
       return true;
     default:
       return false;
   }
 }
 
 bool ValidFormat(gfx::GpuMemoryBuffer::Format format) {
   switch (format) {
+    case gfx::GpuMemoryBuffer::ATC:
+    case gfx::GpuMemoryBuffer::ATCIA:
+    case gfx::GpuMemoryBuffer::DXT1:
+    case gfx::GpuMemoryBuffer::DXT5:
+    case gfx::GpuMemoryBuffer::ETC1:
     case gfx::GpuMemoryBuffer::RGBA_8888:
     case gfx::GpuMemoryBuffer::BGRA_8888:
       return true;
     case gfx::GpuMemoryBuffer::RGBX_8888:
       return false;
   }
 
   NOTREACHED();
   return false;
 }
 
+bool CompressedFormat(gfx::GpuMemoryBuffer::Format format) {
+  switch (format) {
+    case gfx::GpuMemoryBuffer::ATC:
+    case gfx::GpuMemoryBuffer::ATCIA:
+    case gfx::GpuMemoryBuffer::DXT1:
+    case gfx::GpuMemoryBuffer::DXT5:
+    case gfx::GpuMemoryBuffer::ETC1:
+      return true;
+    case gfx::GpuMemoryBuffer::RGBA_8888:
+    case gfx::GpuMemoryBuffer::BGRA_8888:
+    case gfx::GpuMemoryBuffer::RGBX_8888:
+      return false;
+  }
+
+  NOTREACHED();
+  return false;
+}
+
 GLenum TextureFormat(gfx::GpuMemoryBuffer::Format format) {
   switch (format) {
+    case gfx::GpuMemoryBuffer::ATC:
+      return GL_ATC_RGB_AMD;
+    case gfx::GpuMemoryBuffer::ATCIA:
+      return GL_ATC_RGBA_INTERPOLATED_ALPHA_AMD;
+    case gfx::GpuMemoryBuffer::DXT1:
+      return GL_COMPRESSED_RGB_S3TC_DXT1_EXT;
+    case gfx::GpuMemoryBuffer::DXT5:
+      return GL_COMPRESSED_RGBA_S3TC_DXT5_EXT;
+    case gfx::GpuMemoryBuffer::ETC1:
+      return GL_ETC1_RGB8_OES;
     case gfx::GpuMemoryBuffer::RGBA_8888:
       return GL_RGBA;
     case gfx::GpuMemoryBuffer::BGRA_8888:
       return GL_BGRA_EXT;
     case gfx::GpuMemoryBuffer::RGBX_8888:
       NOTREACHED();
       return 0;
   }
 
   NOTREACHED();
   return 0;
 }
 
 GLenum DataFormat(gfx::GpuMemoryBuffer::Format format) {
   return TextureFormat(format);
 }
 
 GLenum DataType(gfx::GpuMemoryBuffer::Format format) {
   switch (format) {
     case gfx::GpuMemoryBuffer::RGBA_8888:
     case gfx::GpuMemoryBuffer::BGRA_8888:
       return GL_UNSIGNED_BYTE;
+    case gfx::GpuMemoryBuffer::ATC:
+    case gfx::GpuMemoryBuffer::ATCIA:
+    case gfx::GpuMemoryBuffer::DXT1:
+    case gfx::GpuMemoryBuffer::DXT5:
+    case gfx::GpuMemoryBuffer::ETC1:
     case gfx::GpuMemoryBuffer::RGBX_8888:
       NOTREACHED();
       return 0;
   }
 
   NOTREACHED();
   return 0;
 }
 
 }  // namespace
@@ skipping 22 matching lines @@
   DCHECK_EQ(0u, egl_texture_id_);
 #endif
 }
 
 // static
 bool GLImageMemory::StrideInBytes(size_t width,
                                   gfx::GpuMemoryBuffer::Format format,
                                   size_t* stride_in_bytes) {
   base::CheckedNumeric<size_t> s = width;
   switch (format) {
+    case gfx::GpuMemoryBuffer::ATCIA:
+    case gfx::GpuMemoryBuffer::DXT5:
+      *stride_in_bytes = width;
+      return true;
+    case gfx::GpuMemoryBuffer::ATC:
+    case gfx::GpuMemoryBuffer::DXT1:
+    case gfx::GpuMemoryBuffer::ETC1:
+      DCHECK_EQ(width % 2, 0U);
+      s /= 2;
+      if (!s.IsValid())
+        return false;
+
+      *stride_in_bytes = s.ValueOrDie();
+      return true;
     case gfx::GpuMemoryBuffer::RGBA_8888:
     case gfx::GpuMemoryBuffer::BGRA_8888:
       s *= 4;
       if (!s.IsValid())
         return false;
 
       *stride_in_bytes = s.ValueOrDie();
       return true;
     case gfx::GpuMemoryBuffer::RGBX_8888:
       NOTREACHED();
@@ skipping 62 matching lines @@
 }
 
 bool GLImageMemory::CopyTexImage(unsigned target) {
   TRACE_EVENT0("gpu", "GLImageMemory::CopyTexImage");
 
   // GL_TEXTURE_EXTERNAL_OES is not a supported CopyTexImage target.
   if (target == GL_TEXTURE_EXTERNAL_OES)
     return false;
 
   DCHECK(memory_);
-  glTexSubImage2D(target, 0,  // level
-                  0,          // x
-                  0,          // y
-                  size_.width(), size_.height(), DataFormat(format_),
-                  DataType(format_), memory_);
+  if (CompressedFormat(format_)) {
+    glCompressedTexSubImage2D(target,

[reveman, 2015/02/11 13:58:40]

Should we avoid calling CompressedTexSubImage2D if the format is not supported
by the GL?

[christiank, 2015/02/12 08:41:52]

Yes, that's the idea. The code assumes that we only try to use a compressed
format that's supported by the GL device.

[reveman, 2015/02/12 12:51:28]

What's preventing a malicious renderer from being able to issue a
CompressedTexSubImage call with an invalid format here?

[christiank, 2015/02/12 13:54:42]

I guess that's possible. One could argue that a malicious renderer could do the
same for the non-compressed path. How is the compressed path different from the
non-compressed in this case? Is it the difference in buffer size that's cause
for concern?

I could add some checks, but I am not quite sure how to query GL features from
here. Any ideas?

[reveman, 2015/02/12 14:18:00]

Yes, that could be a problem. In general, we shouldn't allow the renderer to use
the GL in a way that generate errors. We should detect incorrect usage and
handle these errors ourselves. 
I don't think the check belong here. I think they should either be in
GLES2CmdDecoder or maybe we should prevent the GLImage from being created in the
first place with a format that is not supported by the GL. Maybe we can add a
check to GpuCommandBufferStub::OnCreateImage.

[christiank, 2015/02/13 10:08:36]

I have now added some checks to GpuCommandBufferStub::OnCreateImage, what do you
think?

+                              0,  // level
+                              0,  // x-offset
+                              0,  // y-offset
+                              size_.width(), size_.height(),
+                              DataFormat(format_), MemoryBytes(format_),
+                              memory_);
+  } else {
+    glTexSubImage2D(target, 0,  // level
+                    0,          // x
+                    0,          // y
+                    size_.width(), size_.height(), DataFormat(format_),
+                    DataType(format_), memory_);
+  }
 
   return true;
 }
 
 void GLImageMemory::WillUseTexImage() {
   DCHECK(!in_use_);
   in_use_ = true;
 
   if (!need_do_bind_tex_image_)
     return;
@@ skipping 28 matching lines @@
     if (egl_image_ == EGL_NO_IMAGE_KHR) {
       DCHECK_EQ(0u, egl_texture_id_);
       glGenTextures(1, &egl_texture_id_);
 
       {
         ScopedTextureBinder texture_binder(GL_TEXTURE_2D, egl_texture_id_);
 
         glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_MIN_FILTER, GL_LINEAR);
         glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_WRAP_S, GL_CLAMP_TO_EDGE);
         glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_WRAP_T, GL_CLAMP_TO_EDGE);
-        glTexImage2D(GL_TEXTURE_2D,
-                     0,  // mip level
-                     TextureFormat(format_),
-                     size_.width(),
-                     size_.height(),
-                     0,  // border
-                     DataFormat(format_),
-                     DataType(format_),
-                     memory_);
+        if (CompressedFormat(format_)) {
+          glCompressedTexImage2D(GL_TEXTURE_2D,
+                                 0,  // mip level
+                                 TextureFormat(format_), size_.width(),
+                                 size_.height(),
+                                 0,  // border
+                                 MemoryBytes(format_), memory_);
+        } else {
+          glTexImage2D(GL_TEXTURE_2D,
+                       0,  // mip level
+                       TextureFormat(format_),
+                       size_.width(),
+                       size_.height(),
+                       0,  // border
+                       DataFormat(format_),
+                       DataType(format_),
+                       memory_);
+        }
       }
 
       EGLint attrs[] = {EGL_IMAGE_PRESERVED_KHR, EGL_TRUE, EGL_NONE};
       // Need to pass current EGL rendering context to eglCreateImageKHR for
       // target type EGL_GL_TEXTURE_2D_KHR.
       egl_image_ =
           eglCreateImageKHR(GLSurfaceEGL::GetHardwareDisplay(),
                             eglGetCurrentContext(),
                             EGL_GL_TEXTURE_2D_KHR,
                             reinterpret_cast<EGLClientBuffer>(egl_texture_id_),
                             attrs);
       DCHECK_NE(EGL_NO_IMAGE_KHR, egl_image_)
           << "Error creating EGLImage: " << eglGetError();
     } else {
       ScopedTextureBinder texture_binder(GL_TEXTURE_2D, egl_texture_id_);
 
-      glTexSubImage2D(GL_TEXTURE_2D,
-                      0,  // mip level
-                      0,  // x-offset
-                      0,  // y-offset
-                      size_.width(),
-                      size_.height(),
-                      DataFormat(format_),
-                      DataType(format_),
-                      memory_);
+      if (CompressedFormat(format_)) {
+        glCompressedTexSubImage2D(GL_TEXTURE_2D,
+                                  0,  // mip level
+                                  0,  // x-offset
+                                  0,  // y-offset
+                                  size_.width(), size_.height(),
+                                  DataFormat(format_), MemoryBytes(format_),
+                                  memory_);
+      } else {
+        glTexSubImage2D(GL_TEXTURE_2D,
+                        0,  // mip level
+                        0,  // x-offset
+                        0,  // y-offset
+                        size_.width(),
+                        size_.height(),
+                        DataFormat(format_),
+                        DataType(format_),
+                        memory_);
+      }
     }
 
     glEGLImageTargetTexture2DOES(target, egl_image_);
     DCHECK_EQ(static_cast<GLenum>(GL_NO_ERROR), glGetError());
     return;
   }
 #endif
 
   DCHECK_NE(static_cast<GLenum>(GL_TEXTURE_EXTERNAL_OES), target);
-  glTexImage2D(target,
-               0,  // mip level
-               TextureFormat(format_),
-               size_.width(),
-               size_.height(),
-               0,  // border
-               DataFormat(format_),
-               DataType(format_),
-               memory_);
+  if (CompressedFormat(format_)) {
+    glCompressedTexImage2D(target,
+                           0,  // mip level
+                           TextureFormat(format_), size_.width(),
+                           size_.height(),
+                           0,  // border
+                           MemoryBytes(format_), memory_);
+  } else {
+    glTexImage2D(target,
+                 0,  // mip level
+                 TextureFormat(format_),
+                 size_.width(),
+                 size_.height(),
+                 0,  // border
+                 DataFormat(format_),
+                 DataType(format_),
+                 memory_);
+  }
+}
+
+size_t GLImageMemory::MemoryBytes(gfx::GpuMemoryBuffer::Format format) {

[reveman, 2015/02/11 13:58:40]

Please move this to anonymous namespace, rename it to SizeInBytes and pass size
to it as parameter.

[christiank, 2015/02/12 08:41:52]

Sounds good, fixed.

+  size_t stride_in_bytes = 0;
+  bool valid_stride = StrideInBytes(size_.width(), format, &stride_in_bytes);
+  DCHECK(valid_stride);
+  return stride_in_bytes * size_.height();
 }
 
 }  // namespace gfx