WebGL: remove validation code for vertexAttribPointer and vertexAttribIPointer

third_party/WebKit/Source/modules/webgl/WebGLVertexArrayObjectBase.cpp

 // Copyright 2015 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 "modules/webgl/WebGLVertexArrayObjectBase.h"
 
 #include "modules/webgl/WebGLRenderingContextBase.h"
 
 namespace blink {
 
 WebGLVertexArrayObjectBase::WebGLVertexArrayObjectBase(WebGLRenderingContextBase* ctx, VaoType type)
     : WebGLContextObject(ctx)
     , m_object(0)
     , m_type(type)
     , m_hasEverBeenBound(false)
     , m_destructionInProgress(false)
     , m_boundElementArrayBuffer(nullptr)
 {
-    m_vertexAttribState.reserveCapacity(ctx->maxVertexAttribs());
+    m_arrayBufferList.resize(ctx->maxVertexAttribs());
 
     switch (m_type) {
     case VaoTypeDefault:
         break;
     default:
         m_object = context()->webContext()->createVertexArrayOES();
         break;
     }
 }
 
 WebGLVertexArrayObjectBase::~WebGLVertexArrayObjectBase()
 {
     m_destructionInProgress = true;
 
     // Delete the platform framebuffer resource, in case
     // where this vertex array object isn't detached when it and
     // the WebGLRenderingContextBase object it is registered with
     // are both finalized.
     detachAndDeleteObject();
 }
 
 void WebGLVertexArrayObjectBase::dispatchDetached(WebGraphicsContext3D* context3d)
 {
     if (m_boundElementArrayBuffer)
         m_boundElementArrayBuffer->onDetached(context3d);
 
-    for (size_t i = 0; i < m_vertexAttribState.size(); ++i) {
-        VertexAttribState* state = m_vertexAttribState[i].get();
-        if (state->bufferBinding)
-            state->bufferBinding->onDetached(context3d);
+    for (size_t i = 0; i < m_arrayBufferList.size(); ++i) {
+        if (m_arrayBufferList[i])
+            m_arrayBufferList[i]->onDetached(context3d);
     }
 }
 
 void WebGLVertexArrayObjectBase::deleteObjectImpl(WebGraphicsContext3D* context3d)
 {
     switch (m_type) {
     case VaoTypeDefault:
         break;
     default:
         context3d->deleteVertexArrayOES(m_object);
@@ skipping 11 matching lines @@
 
 void WebGLVertexArrayObjectBase::setElementArrayBuffer(WebGLBuffer* buffer)
 {
     if (buffer)
         buffer->onAttached();
     if (m_boundElementArrayBuffer)
         m_boundElementArrayBuffer->onDetached(context()->webContext());
     m_boundElementArrayBuffer = buffer;
 }
 
-WebGLVertexArrayObjectBase::VertexAttribState* WebGLVertexArrayObjectBase::getVertexAttribState(size_t index)
+WebGLBuffer* WebGLVertexArrayObjectBase::getArrayBufferForAttrib(size_t index)
 {
     ASSERT(index < context()->maxVertexAttribs());
-    // Lazily create the vertex attribute states.
-    for (size_t i = m_vertexAttribState.size(); i <= index; i++)
-        m_vertexAttribState.append(new VertexAttribState);
-    return m_vertexAttribState[index].get();
+    return m_arrayBufferList[index].get();
 }
 
-void WebGLVertexArrayObjectBase::setVertexAttribState(
-    GLuint index, GLsizei bytesPerElement, GLint size, GLenum type, GLboolean normalized, GLsizei stride, GLintptr offset, WebGLBuffer* buffer)
+void WebGLVertexArrayObjectBase::setArrayBufferForAttrib(GLuint index, WebGLBuffer* buffer)
 {
-    GLsizei validatedStride = stride ? stride : bytesPerElement;
-    VertexAttribState* state = getVertexAttribState(index);
-
     if (buffer)
         buffer->onAttached();
-    if (state->bufferBinding)
-        state->bufferBinding->onDetached(context()->webContext());
+    if (m_arrayBufferList[index])
+        m_arrayBufferList[index]->onDetached(context()->webContext());
 
-    state->bufferBinding = buffer;
-    state->bytesPerElement = bytesPerElement;
-    state->size = size;
-    state->type = type;
-    state->normalized = normalized;
-    state->stride = validatedStride;
-    state->originalStride = stride;
-    state->offset = offset;
+    m_arrayBufferList[index] = buffer;
 }
 
 void WebGLVertexArrayObjectBase::unbindBuffer(WebGLBuffer* buffer)
 {
     if (m_boundElementArrayBuffer == buffer) {
         m_boundElementArrayBuffer->onDetached(context()->webContext());
         m_boundElementArrayBuffer = nullptr;
     }
 
-    for (size_t i = 0; i < m_vertexAttribState.size(); ++i) {
-        VertexAttribState* state = m_vertexAttribState[i];
-        if (state->bufferBinding == buffer) {
-            buffer->onDetached(context()->webContext());
-            state->bufferBinding = nullptr;
+    for (size_t i = 0; i < m_arrayBufferList.size(); ++i) {
+        if (m_arrayBufferList[i] == buffer) {
+            m_arrayBufferList[i]->onDetached(context()->webContext());
+            m_arrayBufferList[i] = nullptr;
         }
     }
 }
 
-void WebGLVertexArrayObjectBase::setVertexAttribDivisor(GLuint index, GLuint divisor)
-{
-    VertexAttribState* state = getVertexAttribState(index);
-    state->divisor = divisor;
-}
-
-DEFINE_TRACE(WebGLVertexArrayObjectBase::VertexAttribState)
-{
-    visitor->trace(bufferBinding);
-}
-
 DEFINE_TRACE(WebGLVertexArrayObjectBase)
 {
     visitor->trace(m_boundElementArrayBuffer);
-    visitor->trace(m_vertexAttribState);
+    visitor->trace(m_arrayBufferList);
     WebGLContextObject::trace(visitor);
 }
 
 } // namespace blink