Make CopyTexImage2D and CopyTexSubImage2D fail if...

gpu/command_buffer/service/gles2_cmd_decoder.cc

 // Copyright (c) 2010 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 "gpu/command_buffer/service/gles2_cmd_decoder.h"
 
 #include <stdio.h>
 
 #include <algorithm>
 #include <list>
@@ skipping 760 matching lines @@
 
   // Deletes the texture info for the given texture.
   void RemoveTextureInfo(GLuint client_id) {
     texture_manager()->RemoveTextureInfo(feature_info_, client_id);
   }
 
   // Get the size (in pixels) of the currently bound frame buffer (either FBO
   // or regular back buffer).
   gfx::Size GetBoundReadFrameBufferSize();
 
+  // Get the format of the currently bound frame buffer (either FBO or regular
+  // back buffer)
+  GLenum GetBoundReadFrameBufferInternalFormat();
+
   // Wrapper for CompressedTexImage2D commands.
   error::Error DoCompressedTexImage2D(
     GLenum target,
     GLint level,
     GLenum internal_format,
     GLsizei width,
     GLsizei height,
     GLint border,
     GLsizei image_size,
     const void* data);
@@ skipping 1397 matching lines @@
   } else {
     last_id = 0;
   }
 
   glBindTexture(GL_TEXTURE_2D, last_id);
   glActiveTexture(GL_TEXTURE0 + active_texture_unit_);
 }
 
 gfx::Size GLES2DecoderImpl::GetBoundReadFrameBufferSize() {
   if (bound_read_framebuffer_ != 0) {
-    int width = 0;
-    int height = 0;
-
-    GLenum target =
-        feature_info_->feature_flags().chromium_framebuffer_multisample ?
-            GL_READ_FRAMEBUFFER_EXT : GL_FRAMEBUFFER;
-
-    // Assume we have to have COLOR_ATTACHMENT0. Should we check for depth and
-    // stencil.
-    GLint fb_type = 0;
-    glGetFramebufferAttachmentParameterivEXT(
-        target,
-        GL_COLOR_ATTACHMENT0,
-        GL_FRAMEBUFFER_ATTACHMENT_OBJECT_TYPE,
-        &fb_type);
-    switch (fb_type) {
-      case GL_RENDERBUFFER:
-        {
-          GLint renderbuffer_id = 0;
-          glGetFramebufferAttachmentParameterivEXT(
-              target,
-              GL_COLOR_ATTACHMENT0,
-              GL_FRAMEBUFFER_ATTACHMENT_OBJECT_NAME,
-              &renderbuffer_id);
-          if (renderbuffer_id != 0) {
-            glGetRenderbufferParameterivEXT(
-                GL_RENDERBUFFER,
-                GL_RENDERBUFFER_WIDTH,
-                &width);
-            glGetRenderbufferParameterivEXT(
-                GL_RENDERBUFFER,
-                GL_RENDERBUFFER_HEIGHT,
-                &height);
-          }
-          break;
-        }
-      case GL_TEXTURE:
-        {
-          GLint texture_id = 0;
-          glGetFramebufferAttachmentParameterivEXT(
-              target,
-              GL_COLOR_ATTACHMENT0,
-              GL_FRAMEBUFFER_ATTACHMENT_OBJECT_NAME,
-              &texture_id);
-          if (texture_id != 0) {
-            GLuint client_id = 0;
-            if (texture_manager()->GetClientId(texture_id, &client_id)) {
-              TextureManager::TextureInfo* texture_info =
-                 GetTextureInfo(client_id);
-              if (texture_info) {
-                GLint level = 0;
-                GLint face = 0;
-                glGetFramebufferAttachmentParameterivEXT(
-                    target,
-                    GL_COLOR_ATTACHMENT0,
-                    GL_FRAMEBUFFER_ATTACHMENT_TEXTURE_LEVEL,
-                    &level);
-                glGetFramebufferAttachmentParameterivEXT(
-                    target,
-                    GL_COLOR_ATTACHMENT0,
-                    GL_FRAMEBUFFER_ATTACHMENT_TEXTURE_CUBE_MAP_FACE,
-                    &face);
-                texture_info->GetLevelSize(
-                    face ? face : GL_TEXTURE_2D, level, &width, &height);
-              }
-            }
-          }
-          break;
-        }
-      default:
-        // unknown so assume width and height are zero.
-        break;
+    const FramebufferManager::FramebufferInfo::Attachment* attachment =
+        bound_read_framebuffer_->GetAttachment(GL_COLOR_ATTACHMENT0);
+    if (attachment) {
+      return gfx::Size(attachment->width(), attachment->height());
     }
-
-    return gfx::Size(width, height);
+    return gfx::Size(0, 0);
   } else if (offscreen_target_frame_buffer_.get()) {
     return offscreen_size_;
   } else {
     return context_->GetSize();
   }
 }
 
+GLenum GLES2DecoderImpl::GetBoundReadFrameBufferInternalFormat() {
+  if (bound_read_framebuffer_ != 0) {
+    const FramebufferManager::FramebufferInfo::Attachment* attachment =
+        bound_read_framebuffer_->GetAttachment(GL_COLOR_ATTACHMENT0);
+    if (attachment) {
+      return attachment->internal_format();
+    }
+    return 0;
+  } else if (offscreen_target_frame_buffer_.get()) {
+    return offscreen_target_color_format_;
+  } else {
+    // TODO(gman): return correct format
+    return GL_RGBA;
+  }
+}
+
 bool GLES2DecoderImpl::UpdateOffscreenFrameBufferSize() {
   if (offscreen_size_ == pending_offscreen_size_)
     return true;
 
   offscreen_size_ = pending_offscreen_size_;
 
   // Reallocate the offscreen target buffers.
   DCHECK(offscreen_target_color_format_);
   if (IsOffscreenBufferMultisampled()) {
     if (!offscreen_target_color_render_buffer_->AllocateStorage(
@@ skipping 1044 matching lines @@
   RenderbufferManager::RenderbufferInfo* info = NULL;
   if (client_renderbuffer_id) {
     info = GetRenderbufferInfo(client_renderbuffer_id);
     if (!info) {
       SetGLError(GL_INVALID_OPERATION,
                  "glFramebufferRenderbuffer: unknown renderbuffer");
       return;
     }
     service_id = info->service_id();
   }
+  CopyRealGLErrorsToWrapper();
   glFramebufferRenderbufferEXT(
       target, attachment, renderbuffertarget, service_id);
-  if (service_id == 0 ||
-      glCheckFramebufferStatusEXT(target) == GL_FRAMEBUFFER_COMPLETE) {
+  GLenum error = glGetError();
+  if (error == GL_NO_ERROR) {
     framebuffer_info->AttachRenderbuffer(attachment, info);
-    if (info) {
-      ClearUnclearedRenderbuffers(target, framebuffer_info);
+    if (service_id == 0 ||
+        glCheckFramebufferStatusEXT(target) == GL_FRAMEBUFFER_COMPLETE) {
+      if (info) {
+        ClearUnclearedRenderbuffers(target, framebuffer_info);
+      }
     }
   }
 }
 
 void GLES2DecoderImpl::SetCapabilityState(GLenum cap, bool enabled) {
   switch (cap) {
     case GL_SCISSOR_TEST:
       enable_scissor_test_ = enabled;
       break;
     default:
@@ skipping 136 matching lines @@
   TextureManager::TextureInfo* info = NULL;
   if (client_texture_id) {
     info = GetTextureInfo(client_texture_id);
     if (!info) {
       SetGLError(GL_INVALID_OPERATION,
                  "glFramebufferTexture2D: unknown texture");
       return;
     }
     service_id = info->service_id();
   }
+  CopyRealGLErrorsToWrapper();
   glFramebufferTexture2DEXT(target, attachment, textarget, service_id, level);
-  if (service_id != 0 &&
-      glCheckFramebufferStatusEXT(target) == GL_FRAMEBUFFER_COMPLETE) {
-    ClearUnclearedRenderbuffers(target, framebuffer_info);
+  GLenum error = glGetError();
+  if (error == GL_NO_ERROR) {
+    framebuffer_info->AttachTexture(attachment, info, textarget, level);
+    if (service_id != 0 &&
+        glCheckFramebufferStatusEXT(target) == GL_FRAMEBUFFER_COMPLETE) {
+      ClearUnclearedRenderbuffers(target, framebuffer_info);
+    }
   }
 }
 
 void GLES2DecoderImpl::DoGetFramebufferAttachmentParameteriv(
     GLenum target, GLenum attachment, GLenum pname, GLint* params) {
   FramebufferManager::FramebufferInfo* framebuffer_info =
       GetFramebufferInfoForTarget(target);
   if (!framebuffer_info) {
     SetGLError(GL_INVALID_OPERATION,
                "glFramebufferAttachmentParameteriv: no framebuffer bound");
@@ skipping 53 matching lines @@
 }
 
 void GLES2DecoderImpl::DoRenderbufferStorageMultisample(
     GLenum target, GLsizei samples, GLenum internalformat,
     GLsizei width, GLsizei height) {
   if (!feature_info_->feature_flags().chromium_framebuffer_multisample) {
     SetGLError(GL_INVALID_OPERATION,
                "glRenderbufferStorageMultisampleEXT: function not available");
     return;
   }
-  bound_renderbuffer_->set_internal_format(internalformat);
 
+  GLenum impl_format = internalformat;
   if (gfx::GetGLImplementation() != gfx::kGLImplementationEGLGLES2) {
-    switch (internalformat) {
+    switch (impl_format) {
       case GL_DEPTH_COMPONENT16:
-        internalformat = GL_DEPTH_COMPONENT;
+        impl_format = GL_DEPTH_COMPONENT;
         break;
       case GL_RGBA4:
       case GL_RGB5_A1:
-        internalformat = GL_RGBA;
+        impl_format = GL_RGBA;
         break;
       case GL_RGB565:
-        internalformat = GL_RGB;
+        impl_format = GL_RGB;
         break;
     }
   }
 
+  CopyRealGLErrorsToWrapper();
   glRenderbufferStorageMultisampleEXT(
-      target, samples, internalformat, width, height);
-  // TODO(gman) should not set internal format unless this succeeds
+      target, samples, impl_format, width, height);
+  GLenum error = glGetError();
+  if (error == GL_NO_ERROR) {
+    bound_renderbuffer_->SetInfo(samples, internalformat, width, height);
+  }
 }
 
 void GLES2DecoderImpl::DoRenderbufferStorage(
   GLenum target, GLenum internalformat, GLsizei width, GLsizei height) {
   if (!bound_renderbuffer_) {
     SetGLError(GL_INVALID_OPERATION,
                "glGetRenderbufferStorage: no renderbuffer bound");
     return;
   }
-  bound_renderbuffer_->set_internal_format(internalformat);
 
+  GLenum impl_format = internalformat;
   if (gfx::GetGLImplementation() != gfx::kGLImplementationEGLGLES2) {
-    switch (internalformat) {
+    switch (impl_format) {
       case GL_DEPTH_COMPONENT16:
-        internalformat = GL_DEPTH_COMPONENT;
+        impl_format = GL_DEPTH_COMPONENT;
         break;
       case GL_RGBA4:
       case GL_RGB5_A1:
-        internalformat = GL_RGBA;
+        impl_format = GL_RGBA;
         break;
       case GL_RGB565:
-        internalformat = GL_RGB;
+        impl_format = GL_RGB;
         break;
     }
   }
 
-  glRenderbufferStorageEXT(target, internalformat, width, height);
-  // TODO(gman) should not set internal format unless this succeeds
+  CopyRealGLErrorsToWrapper();
+  glRenderbufferStorageEXT(target, impl_format, width, height);
+  GLenum error = glGetError();
+  if (error == GL_NO_ERROR) {
+    bound_renderbuffer_->SetInfo(0, internalformat, width, height);
+  }
 }
 
 void GLES2DecoderImpl::DoLinkProgram(GLuint program) {
   ProgramManager::ProgramInfo* info = GetProgramInfoNotShader(
       program, "glLinkProgram");
   if (!info) {
     return;
   }
   if (!info->CanLink()) {
     return;
@@ skipping 1838 matching lines @@
                "glCopyTexImage2D: unknown texture for target");
     return;
   }
   if (!texture_manager()->ValidForTarget(
         feature_info_, target, level, width, height, 1) ||
       border != 0) {
     SetGLError(GL_INVALID_VALUE, "glCopyTexImage2D: dimensions out of range");
     return;
   }
 
-  // TODO(gman): Need to check that current FBO is compatible with
-  // internal_format.
-  // TODO(gman): Type needs to match format for FBO.
+  // Check we have compatible formats.
+  GLenum read_format = GetBoundReadFrameBufferInternalFormat();
+  uint32 channels_exist = GLES2Util::GetChannelsForFormat(read_format);
+  uint32 channels_needed = GLES2Util::GetChannelsForFormat(internal_format);
+
+  if ((channels_needed & channels_exist) != channels_needed) {
+    SetGLError(GL_INVALID_OPERATION, "glCopyTexImage2D: incompatible format");
+    return;
+  }
+
   CopyRealGLErrorsToWrapper();
   ScopedResolvedFrameBufferBinder binder(this);
+  gfx::Size size = GetBoundReadFrameBufferSize();
+
   // Clip to size to source dimensions
-  gfx::Size size = GetBoundReadFrameBufferSize();
   GLint copyX = 0;
   GLint copyY = 0;
   GLint copyWidth = 0;
   GLint copyHeight = 0;
   Clip(x, width, size.width(), &copyX, &copyWidth);
   Clip(y, height, size.height(), &copyY, &copyHeight);
 
   if (copyX != x ||
       copyY != y ||
       copyWidth != width ||
@@ skipping 48 matching lines @@
   }
   GLenum type = 0;
   GLenum format = 0;
   if (!info->GetLevelType(target, level, &type, &format) ||
       !info->ValidForTexture(
           target, level, xoffset, yoffset, width, height, format, type)) {
     SetGLError(GL_INVALID_VALUE,
                "glCopyTexSubImage2D: bad dimensions.");
     return;
   }
-  // TODO(gman): Should we check that x, y, width, and height are in range
-  // for current FBO?
+
+  // Check we have compatible formats.
+  GLenum read_format = GetBoundReadFrameBufferInternalFormat();
+  uint32 channels_exist = GLES2Util::GetChannelsForFormat(read_format);
+  uint32 channels_needed = GLES2Util::GetChannelsForFormat(format);
+
+  if ((channels_needed & channels_exist) != channels_needed) {
+    SetGLError(
+        GL_INVALID_OPERATION, "glCopyTexSubImage2D: incompatible format");
+    return;
+  }
+
   ScopedResolvedFrameBufferBinder binder(this);
   gfx::Size size = GetBoundReadFrameBufferSize();
   GLint copyX = 0;
   GLint copyY = 0;
   GLint copyWidth = 0;
   GLint copyHeight = 0;
   Clip(x, width, size.width(), &copyX, &copyWidth);
   Clip(y, height, size.height(), &copyY, &copyHeight);
   if (copyX != x ||
       copyY != y ||
@@ skipping 511 matching lines @@
   return error::kNoError;
 }
 
 // Include the auto-generated part of this file. We split this because it means
 // we can easily edit the non-auto generated parts right here in this file
 // instead of having to edit some template or the code generator.
 #include "gpu/command_buffer/service/gles2_cmd_decoder_autogen.h"
 
 }  // namespace gles2
 }  // namespace gpu