Fix framebuffer completeness ES3 behavior.

gpu/command_buffer/service/framebuffer_manager.cc

 // Copyright (c) 2012 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/framebuffer_manager.h"
 #include "base/logging.h"
 #include "base/strings/stringprintf.h"
 #include "gpu/command_buffer/common/gles2_cmd_utils.h"
 #include "gpu/command_buffer/service/renderbuffer_manager.h"
 #include "gpu/command_buffer/service/texture_manager.h"
@@ skipping 242 matching lines @@
   GLsizei samples_;
 
   DISALLOW_COPY_AND_ASSIGN(TextureAttachment);
 };
 
 FramebufferManager::TextureDetachObserver::TextureDetachObserver() {}
 
 FramebufferManager::TextureDetachObserver::~TextureDetachObserver() {}
 
 FramebufferManager::FramebufferManager(
-    uint32 max_draw_buffers, uint32 max_color_attachments)
+    uint32 max_draw_buffers, uint32 max_color_attachments,
+    ContextGroup::ContextType context_type)
     : framebuffer_state_change_count_(1),
       framebuffer_count_(0),
       have_context_(true),
       max_draw_buffers_(max_draw_buffers),
-      max_color_attachments_(max_color_attachments) {
+      max_color_attachments_(max_color_attachments),
+      context_type_(context_type) {
   DCHECK_GT(max_draw_buffers_, 0u);
   DCHECK_GT(max_color_attachments_, 0u);
 }
 
 FramebufferManager::~FramebufferManager() {
   DCHECK(framebuffers_.empty());
   // If this triggers, that means something is keeping a reference to a
   // Framebuffer belonging to this.
   CHECK_EQ(framebuffer_count_, 0u);
 }
@@ skipping 190 matching lines @@
     if (!attachment->ValidForAttachmentType(attachment_type,
                                             manager_->max_color_attachments_)) {
       return GL_FRAMEBUFFER_INCOMPLETE_ATTACHMENT;
     }
     if (width < 0) {
       width = attachment->width();
       height = attachment->height();
       if (width == 0 || height == 0) {
         return GL_FRAMEBUFFER_INCOMPLETE_ATTACHMENT;
       }
-    } else {
+    } else if (manager_->context_type() != ContextGroup::CONTEXT_TYPE_WEBGL2) {
+      // TODO(zmo): revisit this if we create ES3 contexts for clients other
+      // than WebGL 2.
       if (attachment->width() != width || attachment->height() != height) {
         return GL_FRAMEBUFFER_INCOMPLETE_DIMENSIONS_EXT;
       }
     }
 
     if (!attachment->CanRenderTo()) {
       return GL_FRAMEBUFFER_UNSUPPORTED;
     }
   }
 
@@ skipping 251 matching lines @@
        ++it) {
     TextureDetachObserver* observer = *it;
     observer->OnTextureRefDetachedFromFramebuffer(texture);
   }
 }
 
 }  // namespace gles2
 }  // namespace gpu