No-op BufferSubData after validation.

We were skipping some validation, which was confusing dEQP. We can still no-op, but do this after all validation. BUG=angleproject:901 Change-Id: I9888c1aca851519e9c7da210c983ba6012e95932 Reviewed-on: https://chromium-review.googlesource.com/266926Reviewed-by: Geoff Lang <geofflang@chromium.org> Reviewed-by: Brandon Jones <bajones@chromium.org> Tested-by: Jamie Madill <jmadill@chromium.org>

No-op BufferSubData after validation.
0be9a8a0 · Jamie Madill · a7441353 · 0be9a8a0
Commit 0be9a8a0 authored Apr 23, 2015 by Jamie Madill
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 5 deletions

entry_points_gles_2_0.cpp src/libGLESv2/entry_points_gles_2_0.cpp +5 -5

No files found.
--- a/src/libGLESv2/entry_points_gles_2_0.cpp
+++ b/src/libGLESv2/entry_points_gles_2_0.cpp
@@ -537,11 +537,6 @@ void GL_APIENTRY BufferSubData(GLenum target, GLintptr offset, GLsizeiptr size, 
            return;
        }

-        if (data == NULL)
-        {
-            return;
-        }
-
        if (!ValidBufferTarget(context, target))
        {
            context->recordError(Error(GL_INVALID_ENUM));
@@ -575,6 +570,11 @@ void GL_APIENTRY BufferSubData(GLenum target, GLintptr offset, GLsizeiptr size, 
            return;
        }

+        if (data == NULL)
+        {
+            return;
+        }
+
        Error error = buffer->bufferSubData(data, size, offset);
        if (error.isError())
        {