InstancedArray crashes attempting to draw out of bounds
authordino@apple.com <dino@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 11 Dec 2014 19:29:18 +0000 (19:29 +0000)
committerdino@apple.com <dino@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 11 Dec 2014 19:29:18 +0000 (19:29 +0000)
https://bugs.webkit.org/show_bug.cgi?id=139521
Source/WebCore:

Reviewed by Simon Fraser.

We were not doing index validation correctly for instancing.

Test: fast/canvas/webgl/angle-instanced-arrays-out-of-bounds.html

* html/canvas/WebGLRenderingContext.cpp:
(WebCore::WebGLRenderingContext::validateVertexAttributes): We need to check
the number of instances drawn against the amount of instance data that has
been provided, taking into account the number of repeats (the divisor).
(WebCore::WebGLRenderingContext::drawArrays): Added some whitespace to make it more clear.
(WebCore::WebGLRenderingContext::validateDrawElements): This needs to take a primcount
parameter so that it can correctly validate the call (when used from drawElementsInstanced).
(WebCore::WebGLRenderingContext::drawElements): New signature to validate.
(WebCore::WebGLRenderingContext::drawArraysInstanced): Rearrange this a bit. The
primcount validation is already being done by the validateDrawArrays call. Also, there
was a bogus UNUSED_PARAM hanging around.
(WebCore::WebGLRenderingContext::drawElementsInstanced): Similar rearrangement. Use
the primcount parameter.
* html/canvas/WebGLRenderingContext.h:

LayoutTests:

<rdar://problem/17540398>

Reviewed by Simon Fraser.

This is a copy of the official webgl/1.0.3 test.

* platform/mac-mountainlion/fast/canvas/webgl/angle-instanced-arrays-out-of-bounds-expected.txt: Added. This extension is not available on Mountain Lion.
* fast/canvas/webgl/angle-instanced-arrays-out-of-bounds-expected.txt: Added.
* fast/canvas/webgl/angle-instanced-arrays-out-of-bounds.html: Added.
* fast/canvas/webgl/resources/out-of-bounds-test.js: Added.
(OutOfBoundsTest):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@177165 268f45cc-cd09-0410-ab3c-d52691b4dbfc

LayoutTests/ChangeLog
LayoutTests/fast/canvas/webgl/angle-instanced-arrays-out-of-bounds-expected.txt [new file with mode: 0644]
LayoutTests/fast/canvas/webgl/angle-instanced-arrays-out-of-bounds.html [new file with mode: 0644]
LayoutTests/fast/canvas/webgl/resources/out-of-bounds-test.js [new file with mode: 0644]
LayoutTests/platform/mac-mountainlion/fast/canvas/webgl/angle-instanced-arrays-out-of-bounds-expected.txt [new file with mode: 0644]
Source/WebCore/ChangeLog
Source/WebCore/html/canvas/WebGLRenderingContext.cpp
Source/WebCore/html/canvas/WebGLRenderingContext.h

index f0cd870..da1424c 100644 (file)
@@ -1,3 +1,19 @@
+2014-12-10  Dean Jackson  <dino@apple.com>
+
+        InstancedArray crashes attempting to draw out of bounds
+        https://bugs.webkit.org/show_bug.cgi?id=139521
+        <rdar://problem/17540398>
+
+        Reviewed by Simon Fraser.
+
+        This is a copy of the official webgl/1.0.3 test.
+
+        * platform/mac-mountainlion/fast/canvas/webgl/angle-instanced-arrays-out-of-bounds-expected.txt: Added. This extension is not available on Mountain Lion.
+        * fast/canvas/webgl/angle-instanced-arrays-out-of-bounds-expected.txt: Added.
+        * fast/canvas/webgl/angle-instanced-arrays-out-of-bounds.html: Added.
+        * fast/canvas/webgl/resources/out-of-bounds-test.js: Added.
+        (OutOfBoundsTest):
+
 2014-12-11  Daniel Bates  <dabates@apple.com>
 
         [iOS] Update expected results for LayoutTests/{js, loader, mathml, pageoverlay, scrollbars}
diff --git a/LayoutTests/fast/canvas/webgl/angle-instanced-arrays-out-of-bounds-expected.txt b/LayoutTests/fast/canvas/webgl/angle-instanced-arrays-out-of-bounds-expected.txt
new file mode 100644 (file)
index 0000000..9414b01
--- /dev/null
@@ -0,0 +1,386 @@
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawArraysInstanced: attempt to access out of bounds arrays
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawArraysInstanced: attempt to access out of bounds arrays
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawArraysInstanced: attempt to access out of bounds arrays
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawArraysInstanced: first or count < 0
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawArraysInstanced: first or count < 0
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawArraysInstanced: first or count < 0
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawArraysInstanced: first or count < 0
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawArraysInstanced: first or count < 0
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawArraysInstanced: attempt to access out of bounds arrays
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawArraysInstanced: attempt to access out of bounds arrays
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawArraysInstanced: attempt to access out of bounds arrays
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawArraysInstanced: first or count < 0
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawArraysInstanced: first or count < 0
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawArraysInstanced: first or count < 0
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawArraysInstanced: first or count < 0
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawArraysInstanced: first or count < 0
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawArraysInstanced: first or count < 0
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawArraysInstanced: first or count < 0
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawArraysInstanced: first or count < 0
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawArraysInstanced: first or count < 0
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawArraysInstanced: first or count < 0
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawArraysInstanced: first or count < 0
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawArraysInstanced: attempt to access out of bounds arrays
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawArraysInstanced: attempt to access out of bounds arrays
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawArraysInstanced: attempt to access out of bounds arrays
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawArraysInstanced: attempt to access out of bounds arrays
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawElementsInstanced: request out of bounds for current ELEMENT_ARRAY_BUFFER
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawElementsInstanced: request out of bounds for current ELEMENT_ARRAY_BUFFER
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawElementsInstanced: request out of bounds for current ELEMENT_ARRAY_BUFFER
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawElementsInstanced: request out of bounds for current ELEMENT_ARRAY_BUFFER
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawElementsInstanced: count or offset < 0
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawElementsInstanced: count or offset < 0
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawElementsInstanced: count or offset < 0
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawElementsInstanced: count or offset < 0
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawElementsInstanced: count or offset < 0
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawElementsInstanced: request out of bounds for current ELEMENT_ARRAY_BUFFER
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawElementsInstanced: request out of bounds for current ELEMENT_ARRAY_BUFFER
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawElementsInstanced: request out of bounds for current ELEMENT_ARRAY_BUFFER
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawElementsInstanced: count or offset < 0
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawElementsInstanced: count or offset < 0
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawElementsInstanced: count or offset < 0
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawElementsInstanced: count or offset < 0
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawElementsInstanced: count or offset < 0
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawElementsInstanced: request out of bounds for current ELEMENT_ARRAY_BUFFER
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawElementsInstanced: request out of bounds for current ELEMENT_ARRAY_BUFFER
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawElementsInstanced: attempt to access out of bounds arrays
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawElementsInstanced: request out of bounds for current ELEMENT_ARRAY_BUFFER
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawElementsInstanced: attempt to access out of bounds arrays
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawElementsInstanced: attempt to access out of bounds arrays
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawElementsInstanced: attempt to access out of bounds arrays
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawElementsInstanced: attempt to access out of bounds arrays
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawElementsInstanced: count or offset < 0
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawElementsInstanced: request out of bounds for current ELEMENT_ARRAY_BUFFER
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawElementsInstanced: request out of bounds for current ELEMENT_ARRAY_BUFFER
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawElementsInstanced: request out of bounds for current ELEMENT_ARRAY_BUFFER
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawElementsInstanced: no ELEMENT_ARRAY_BUFFER bound
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawElementsInstanced: no ELEMENT_ARRAY_BUFFER bound
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawArraysInstanced: attempt to access out of bounds arrays
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawArraysInstanced: attempt to access out of bounds arrays
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawArraysInstanced: attempt to access out of bounds arrays
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawArraysInstanced: first or count < 0
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawArraysInstanced: first or count < 0
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawArraysInstanced: first or count < 0
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawArraysInstanced: first or count < 0
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawArraysInstanced: first or count < 0
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawArraysInstanced: attempt to access out of bounds arrays
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawArraysInstanced: attempt to access out of bounds arrays
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawArraysInstanced: attempt to access out of bounds arrays
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawArraysInstanced: first or count < 0
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawArraysInstanced: first or count < 0
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawArraysInstanced: first or count < 0
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawArraysInstanced: first or count < 0
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawArraysInstanced: first or count < 0
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawArraysInstanced: first or count < 0
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawArraysInstanced: first or count < 0
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawArraysInstanced: first or count < 0
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawArraysInstanced: first or count < 0
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawArraysInstanced: first or count < 0
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawArraysInstanced: first or count < 0
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawArraysInstanced: attempt to access out of bounds arrays
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawArraysInstanced: attempt to access out of bounds arrays
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawArraysInstanced: attempt to access out of bounds arrays
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawArraysInstanced: attempt to access out of bounds arrays
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawElementsInstanced: request out of bounds for current ELEMENT_ARRAY_BUFFER
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawElementsInstanced: request out of bounds for current ELEMENT_ARRAY_BUFFER
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawElementsInstanced: request out of bounds for current ELEMENT_ARRAY_BUFFER
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawElementsInstanced: request out of bounds for current ELEMENT_ARRAY_BUFFER
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawElementsInstanced: count or offset < 0
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawElementsInstanced: count or offset < 0
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawElementsInstanced: count or offset < 0
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawElementsInstanced: count or offset < 0
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawElementsInstanced: count or offset < 0
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawElementsInstanced: request out of bounds for current ELEMENT_ARRAY_BUFFER
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawElementsInstanced: request out of bounds for current ELEMENT_ARRAY_BUFFER
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawElementsInstanced: request out of bounds for current ELEMENT_ARRAY_BUFFER
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawElementsInstanced: count or offset < 0
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawElementsInstanced: count or offset < 0
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawElementsInstanced: count or offset < 0
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawElementsInstanced: count or offset < 0
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawElementsInstanced: count or offset < 0
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawElementsInstanced: request out of bounds for current ELEMENT_ARRAY_BUFFER
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawElementsInstanced: request out of bounds for current ELEMENT_ARRAY_BUFFER
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawElementsInstanced: attempt to access out of bounds arrays
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawElementsInstanced: request out of bounds for current ELEMENT_ARRAY_BUFFER
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawElementsInstanced: attempt to access out of bounds arrays
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawElementsInstanced: attempt to access out of bounds arrays
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawElementsInstanced: attempt to access out of bounds arrays
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawElementsInstanced: attempt to access out of bounds arrays
+CONSOLE MESSAGE: WebGL: INVALID_VALUE: drawElementsInstanced: count or offset < 0
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawElementsInstanced: request out of bounds for current ELEMENT_ARRAY_BUFFER
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawElementsInstanced: request out of bounds for current ELEMENT_ARRAY_BUFFER
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawElementsInstanced: request out of bounds for current ELEMENT_ARRAY_BUFFER
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawElementsInstanced: no ELEMENT_ARRAY_BUFFER bound
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawElementsInstanced: no ELEMENT_ARRAY_BUFFER bound
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawArraysInstanced: attempt to access out of bounds arrays
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawArraysInstanced: attempt to access out of bounds arrays
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawArraysInstanced: attempt to access out of bounds arrays
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawArraysInstanced: attempt to access out of bounds arrays
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawArraysInstanced: attempt to access out of bounds arrays
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawArraysInstanced: attempt to access out of bounds arrays
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawArraysInstanced: attempt to access out of bounds arrays
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawArraysInstanced: attempt to access out of bounds arrays
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawElementsInstanced: attempt to access out of bounds arrays
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawElementsInstanced: attempt to access out of bounds arrays
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawElementsInstanced: attempt to access out of bounds arrays
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawElementsInstanced: attempt to access out of bounds arrays
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawElementsInstanced: attempt to access out of bounds arrays
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawElementsInstanced: attempt to access out of bounds arrays
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawElementsInstanced: attempt to access out of bounds arrays
+CONSOLE MESSAGE: WebGL: INVALID_OPERATION: drawElementsInstanced: attempt to access out of bounds arrays
+Test of drawArraysInstancedANGLE and drawElementsInstancedANGLE with out-of-bounds parameters
+
+On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
+
+ PASS Successfully enabled ANGLE_instanced_arrays extension
+
+Test with 1 instance without instanced attributes
+
+Test empty buffer
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0, 1, 1) was expected value: INVALID_OPERATION.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0, 10000, 1) was expected value: INVALID_OPERATION.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0, 10000000000000, 1) was expected value: INVALID_OPERATION.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 1, 0, 1) was expected value: NO_ERROR.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0, 0, 1) was expected value: NO_ERROR.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 100, 0, 1) was expected value: NO_ERROR.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0, -1, 1) was expected value: INVALID_VALUE.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, -1, 0, 1) was expected value: INVALID_VALUE.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 1, -1, 1) was expected value: INVALID_VALUE.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, -1, 1, 1) was expected value: INVALID_VALUE.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0, 0xffffffff, 1) was expected value: INVALID_VALUE.
+
+Test buffer with 3 float vectors
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0, 3, 1) was expected value: NO_ERROR.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 3, 2, 1) was expected value: INVALID_OPERATION.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0, 10000, 1) was expected value: INVALID_OPERATION.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0, 10000000000000, 1) was expected value: INVALID_OPERATION.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0, 0, 1) was expected value: NO_ERROR.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 100, 0, 1) was expected value: NO_ERROR.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0, -1, 1) was expected value: INVALID_VALUE.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, -1, 0, 1) was expected value: INVALID_VALUE.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 1, -1, 1) was expected value: INVALID_VALUE.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, -1, 1, 1) was expected value: INVALID_VALUE.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0, 0xffffffff, 1) was expected value: INVALID_VALUE.
+
+Test buffer with interleaved (3+2) float vectors
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0, 9, 1) was expected value: NO_ERROR.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0, -500, 1) was expected value: INVALID_VALUE.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, -200, 1, 1) was expected value: INVALID_VALUE.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, -200, -500, 1) was expected value: INVALID_VALUE.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0, 0xffffffff, 1) was expected value: INVALID_VALUE.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0xffffffff, 0xffffffff, 1) was expected value: INVALID_VALUE.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0xffffffff, 1, 1) was expected value: INVALID_VALUE.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0, 200, 1) was expected value: INVALID_OPERATION.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0, 0x7fffffff, 1) was expected value: INVALID_OPERATION.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0x7fffffff, 1, 1) was expected value: INVALID_OPERATION.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0x7fffffff, 0x7fffffff, 1) was expected value: INVALID_OPERATION.
+
+Test empty index buffer
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 0, gl.UNSIGNED_BYTE, 0, 1) was expected value: NO_ERROR.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 3, gl.UNSIGNED_BYTE, 0, 1) was expected value: INVALID_OPERATION.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 10000, gl.UNSIGNED_BYTE, 0, 1) was expected value: INVALID_OPERATION.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 10000000000000, gl.UNSIGNED_BYTE, 0, 1) was expected value: INVALID_OPERATION.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 1, gl.UNSIGNED_BYTE, 0, 1) was expected value: INVALID_OPERATION.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, -1, gl.UNSIGNED_BYTE, 0, 1) was expected value: INVALID_VALUE.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 0, gl.UNSIGNED_BYTE, -1, 1) was expected value: INVALID_VALUE.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, -1, gl.UNSIGNED_BYTE, 1, 1) was expected value: INVALID_VALUE.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 1, gl.UNSIGNED_BYTE, -1, 1) was expected value: INVALID_VALUE.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 0xffffffff, gl.UNSIGNED_BYTE, 0, 1) was expected value: INVALID_VALUE.
+
+Test buffer with 3 byte indexes
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 3, gl.UNSIGNED_BYTE, 0, 1) was expected value: NO_ERROR.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 3, gl.UNSIGNED_BYTE, 2, 1) was expected value: INVALID_OPERATION.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 10000, gl.UNSIGNED_BYTE, 0, 1) was expected value: INVALID_OPERATION.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 10000000000000, gl.UNSIGNED_BYTE, 0, 1) was expected value: INVALID_OPERATION.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, -1, gl.UNSIGNED_BYTE, 0, 1) was expected value: INVALID_VALUE.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 0, gl.UNSIGNED_BYTE, -1, 1) was expected value: INVALID_VALUE.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, -1, gl.UNSIGNED_BYTE, 1, 1) was expected value: INVALID_VALUE.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 1, gl.UNSIGNED_BYTE, -1, 1) was expected value: INVALID_VALUE.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 0xffffffff, gl.UNSIGNED_BYTE, 0, 1) was expected value: INVALID_VALUE.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 0, gl.UNSIGNED_BYTE, 4, 1) was expected value: NO_ERROR.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 0x7fffffff, gl.UNSIGNED_BYTE, 0, 1) was expected value: INVALID_OPERATION.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 0x7fffffff, gl.UNSIGNED_BYTE, 0x7fffffff, 1) was expected value: INVALID_OPERATION.
+PASS gl.bufferData(gl.ELEMENT_ARRAY_BUFFER, (new Uint8Array([ 3, 0, 1, 2 ])).subarray(1), gl.STATIC_DRAW) was expected value: NO_ERROR.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 3, gl.UNSIGNED_BYTE, 0, 1) was expected value: NO_ERROR.
+PASS gl.bufferSubData(gl.ELEMENT_ARRAY_BUFFER, 0, new Uint8Array([ 3, 0, 1])) was expected value: NO_ERROR.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 3, gl.UNSIGNED_BYTE, 0, 1) was expected value: INVALID_OPERATION.
+PASS gl.bufferSubData(gl.ELEMENT_ARRAY_BUFFER, 0, (new Uint8Array([ 3, 0, 1, 2 ])).subarray(1)) was expected value: NO_ERROR.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 3, gl.UNSIGNED_BYTE, 0, 1) was expected value: NO_ERROR.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 0, gl.UNSIGNED_BYTE, 0, 1) was expected value: NO_ERROR.
+
+Test buffer with interleaved (3+2) float vectors
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 9, gl.UNSIGNED_SHORT, 0, 1) was expected value: NO_ERROR.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 9, gl.UNSIGNED_SHORT, 1000, 1) was expected value: INVALID_OPERATION.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 12, gl.UNSIGNED_SHORT, 0, 1) was expected value: INVALID_OPERATION.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 15, gl.UNSIGNED_SHORT, 0, 1) was expected value: INVALID_OPERATION.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 18, gl.UNSIGNED_SHORT, 0, 1) was expected value: INVALID_OPERATION.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 3, gl.UNSIGNED_SHORT, 30, 1) was expected value: INVALID_OPERATION.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 0xffffffff, gl.UNSIGNED_SHORT, 0, 1) was expected value: INVALID_VALUE.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 1, gl.UNSIGNED_SHORT, 0xffffffff, 1) was expected value: INVALID_OPERATION.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 0x7fffffff, gl.UNSIGNED_SHORT, 0, 1) was expected value: INVALID_OPERATION.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 0, gl.UNSIGNED_SHORT, 0, 1) was expected value: NO_ERROR.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 6, gl.UNSIGNED_SHORT, 0, 1) was expected value: NO_ERROR.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 6, gl.UNSIGNED_SHORT, 1, 1) was expected value: INVALID_OPERATION.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 6, gl.UNSIGNED_SHORT, 2, 1) was expected value: NO_ERROR.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 6, gl.UNSIGNED_SHORT, 0, 1) was expected value: INVALID_OPERATION.
+
+Test buffer setting attrib 0 to a buffer too small and disable it.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 6, gl.UNSIGNED_SHORT, 2, 1) was expected value: NO_ERROR.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 6, gl.UNSIGNED_SHORT, 2, 1) was expected value: INVALID_OPERATION.
+
+Test with 2 instances without instanced attributes
+
+Test empty buffer
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0, 1, 2) was expected value: INVALID_OPERATION.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0, 10000, 2) was expected value: INVALID_OPERATION.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0, 10000000000000, 2) was expected value: INVALID_OPERATION.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 1, 0, 2) was expected value: NO_ERROR.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0, 0, 2) was expected value: NO_ERROR.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 100, 0, 2) was expected value: NO_ERROR.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0, -1, 2) was expected value: INVALID_VALUE.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, -1, 0, 2) was expected value: INVALID_VALUE.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 1, -1, 2) was expected value: INVALID_VALUE.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, -1, 1, 2) was expected value: INVALID_VALUE.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0, 0xffffffff, 2) was expected value: INVALID_VALUE.
+
+Test buffer with 3 float vectors
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0, 3, 2) was expected value: NO_ERROR.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 3, 2, 2) was expected value: INVALID_OPERATION.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0, 10000, 2) was expected value: INVALID_OPERATION.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0, 10000000000000, 2) was expected value: INVALID_OPERATION.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0, 0, 2) was expected value: NO_ERROR.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 100, 0, 2) was expected value: NO_ERROR.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0, -1, 2) was expected value: INVALID_VALUE.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, -1, 0, 2) was expected value: INVALID_VALUE.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 1, -1, 2) was expected value: INVALID_VALUE.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, -1, 1, 2) was expected value: INVALID_VALUE.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0, 0xffffffff, 2) was expected value: INVALID_VALUE.
+
+Test buffer with interleaved (3+2) float vectors
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0, 9, 2) was expected value: NO_ERROR.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0, -500, 2) was expected value: INVALID_VALUE.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, -200, 1, 2) was expected value: INVALID_VALUE.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, -200, -500, 2) was expected value: INVALID_VALUE.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0, 0xffffffff, 2) was expected value: INVALID_VALUE.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0xffffffff, 0xffffffff, 2) was expected value: INVALID_VALUE.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0xffffffff, 1, 2) was expected value: INVALID_VALUE.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0, 200, 2) was expected value: INVALID_OPERATION.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0, 0x7fffffff, 2) was expected value: INVALID_OPERATION.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0x7fffffff, 1, 2) was expected value: INVALID_OPERATION.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0x7fffffff, 0x7fffffff, 2) was expected value: INVALID_OPERATION.
+
+Test empty index buffer
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 0, gl.UNSIGNED_BYTE, 0, 2) was expected value: NO_ERROR.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 3, gl.UNSIGNED_BYTE, 0, 2) was expected value: INVALID_OPERATION.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 10000, gl.UNSIGNED_BYTE, 0, 2) was expected value: INVALID_OPERATION.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 10000000000000, gl.UNSIGNED_BYTE, 0, 2) was expected value: INVALID_OPERATION.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 1, gl.UNSIGNED_BYTE, 0, 2) was expected value: INVALID_OPERATION.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, -1, gl.UNSIGNED_BYTE, 0, 2) was expected value: INVALID_VALUE.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 0, gl.UNSIGNED_BYTE, -1, 2) was expected value: INVALID_VALUE.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, -1, gl.UNSIGNED_BYTE, 1, 2) was expected value: INVALID_VALUE.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 1, gl.UNSIGNED_BYTE, -1, 2) was expected value: INVALID_VALUE.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 0xffffffff, gl.UNSIGNED_BYTE, 0, 2) was expected value: INVALID_VALUE.
+
+Test buffer with 3 byte indexes
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 3, gl.UNSIGNED_BYTE, 0, 2) was expected value: NO_ERROR.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 3, gl.UNSIGNED_BYTE, 2, 2) was expected value: INVALID_OPERATION.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 10000, gl.UNSIGNED_BYTE, 0, 2) was expected value: INVALID_OPERATION.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 10000000000000, gl.UNSIGNED_BYTE, 0, 2) was expected value: INVALID_OPERATION.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, -1, gl.UNSIGNED_BYTE, 0, 2) was expected value: INVALID_VALUE.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 0, gl.UNSIGNED_BYTE, -1, 2) was expected value: INVALID_VALUE.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, -1, gl.UNSIGNED_BYTE, 1, 2) was expected value: INVALID_VALUE.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 1, gl.UNSIGNED_BYTE, -1, 2) was expected value: INVALID_VALUE.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 0xffffffff, gl.UNSIGNED_BYTE, 0, 2) was expected value: INVALID_VALUE.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 0, gl.UNSIGNED_BYTE, 4, 2) was expected value: NO_ERROR.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 0x7fffffff, gl.UNSIGNED_BYTE, 0, 2) was expected value: INVALID_OPERATION.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 0x7fffffff, gl.UNSIGNED_BYTE, 0x7fffffff, 2) was expected value: INVALID_OPERATION.
+PASS gl.bufferData(gl.ELEMENT_ARRAY_BUFFER, (new Uint8Array([ 3, 0, 1, 2 ])).subarray(1), gl.STATIC_DRAW) was expected value: NO_ERROR.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 3, gl.UNSIGNED_BYTE, 0, 2) was expected value: NO_ERROR.
+PASS gl.bufferSubData(gl.ELEMENT_ARRAY_BUFFER, 0, new Uint8Array([ 3, 0, 1])) was expected value: NO_ERROR.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 3, gl.UNSIGNED_BYTE, 0, 2) was expected value: INVALID_OPERATION.
+PASS gl.bufferSubData(gl.ELEMENT_ARRAY_BUFFER, 0, (new Uint8Array([ 3, 0, 1, 2 ])).subarray(1)) was expected value: NO_ERROR.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 3, gl.UNSIGNED_BYTE, 0, 2) was expected value: NO_ERROR.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 0, gl.UNSIGNED_BYTE, 0, 2) was expected value: NO_ERROR.
+
+Test buffer with interleaved (3+2) float vectors
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 9, gl.UNSIGNED_SHORT, 0, 2) was expected value: NO_ERROR.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 9, gl.UNSIGNED_SHORT, 1000, 2) was expected value: INVALID_OPERATION.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 12, gl.UNSIGNED_SHORT, 0, 2) was expected value: INVALID_OPERATION.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 15, gl.UNSIGNED_SHORT, 0, 2) was expected value: INVALID_OPERATION.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 18, gl.UNSIGNED_SHORT, 0, 2) was expected value: INVALID_OPERATION.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 3, gl.UNSIGNED_SHORT, 30, 2) was expected value: INVALID_OPERATION.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 0xffffffff, gl.UNSIGNED_SHORT, 0, 2) was expected value: INVALID_VALUE.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 1, gl.UNSIGNED_SHORT, 0xffffffff, 2) was expected value: INVALID_OPERATION.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 0x7fffffff, gl.UNSIGNED_SHORT, 0, 2) was expected value: INVALID_OPERATION.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 0, gl.UNSIGNED_SHORT, 0, 2) was expected value: NO_ERROR.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 6, gl.UNSIGNED_SHORT, 0, 2) was expected value: NO_ERROR.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 6, gl.UNSIGNED_SHORT, 1, 2) was expected value: INVALID_OPERATION.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 6, gl.UNSIGNED_SHORT, 2, 2) was expected value: NO_ERROR.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 6, gl.UNSIGNED_SHORT, 0, 2) was expected value: INVALID_OPERATION.
+
+Test buffer setting attrib 0 to a buffer too small and disable it.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 6, gl.UNSIGNED_SHORT, 2, 2) was expected value: NO_ERROR.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 6, gl.UNSIGNED_SHORT, 2, 2) was expected value: INVALID_OPERATION.
+
+Test out-of-range instanced attributes
+
+Test with an empty buffer for the instanced attribute
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0, 9, 0) was expected value: NO_ERROR.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0, 10000, 0) was expected value: NO_ERROR.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0, 9, 1) was expected value: INVALID_OPERATION.
+Test with a buffer with 1 float for the instanced attribute
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0, 9, 0) was expected value: NO_ERROR.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0, 10000, 0) was expected value: NO_ERROR.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0, 9, 1) was expected value: INVALID_OPERATION.
+
+Test with a buffer with 2 floats for the instanced attribute
+Divisor 1
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0, 9, 1) was expected value: NO_ERROR.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0, 9, 2) was expected value: INVALID_OPERATION.
+Divisor 3
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0, 9, 3) was expected value: NO_ERROR.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0, 9, 4) was expected value: INVALID_OPERATION.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0, 9, 10000) was expected value: INVALID_OPERATION.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0, 9, 0x7fffffff) was expected value: INVALID_OPERATION.
+
+Test with a buffer with 4 floats for the instanced attribute
+Divisor 1
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0, 9, 2) was expected value: NO_ERROR.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0, 9, 3) was expected value: INVALID_OPERATION.
+Divisor 2
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0, 9, 4) was expected value: NO_ERROR.
+PASS ext.drawArraysInstancedANGLE(gl.TRIANGLES, 0, 9, 5) was expected value: INVALID_OPERATION.
+
+Test out-of-range instanced attributes
+
+Test with an empty buffer for the instanced attribute
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 9, gl.UNSIGNED_BYTE, 0, 0) was expected value: NO_ERROR.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 10000, gl.UNSIGNED_BYTE, 0, 0) was expected value: NO_ERROR.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 9, gl.UNSIGNED_BYTE, 0, 1) was expected value: INVALID_OPERATION.
+Test with a buffer with 1 float for the instanced attribute
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 9, gl.UNSIGNED_BYTE, 0, 0) was expected value: NO_ERROR.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 10000, gl.UNSIGNED_BYTE, 0, 0) was expected value: NO_ERROR.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 9, gl.UNSIGNED_BYTE, 0, 1) was expected value: INVALID_OPERATION.
+
+Test with a buffer with 2 floats for the instanced attribute
+Divisor 1
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 9, gl.UNSIGNED_BYTE, 0, 1) was expected value: NO_ERROR.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 9, gl.UNSIGNED_BYTE, 0, 2) was expected value: INVALID_OPERATION.
+Divisor 3
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 9, gl.UNSIGNED_BYTE, 0, 3) was expected value: NO_ERROR.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 9, gl.UNSIGNED_BYTE, 0, 4) was expected value: INVALID_OPERATION.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 9, gl.UNSIGNED_BYTE, 0, 10000) was expected value: INVALID_OPERATION.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 9, gl.UNSIGNED_BYTE, 0, 0x7fffffff) was expected value: INVALID_OPERATION.
+
+Test with a buffer with 4 floats for the instanced attribute
+Divisor 1
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 9, gl.UNSIGNED_BYTE, 0, 2) was expected value: NO_ERROR.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 9, gl.UNSIGNED_BYTE, 0, 3) was expected value: INVALID_OPERATION.
+Divisor 2
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 9, gl.UNSIGNED_BYTE, 0, 4) was expected value: NO_ERROR.
+PASS ext.drawElementsInstancedANGLE(gl.TRIANGLES, 9, gl.UNSIGNED_BYTE, 0, 5) was expected value: INVALID_OPERATION.
+
+PASS successfullyParsed is true
+
+TEST COMPLETE
+
diff --git a/LayoutTests/fast/canvas/webgl/angle-instanced-arrays-out-of-bounds.html b/LayoutTests/fast/canvas/webgl/angle-instanced-arrays-out-of-bounds.html
new file mode 100644 (file)
index 0000000..1e052df
--- /dev/null
@@ -0,0 +1,80 @@
+<!--
+
+/*
+** Copyright (c) 2014 The Khronos Group Inc.
+**
+** Permission is hereby granted, free of charge, to any person obtaining a
+** copy of this software and/or associated documentation files (the
+** "Materials"), to deal in the Materials without restriction, including
+** without limitation the rights to use, copy, modify, merge, publish,
+** distribute, sublicense, and/or sell copies of the Materials, and to
+** permit persons to whom the Materials are furnished to do so, subject to
+** the following conditions:
+**
+** The above copyright notice and this permission notice shall be included
+** in all copies or substantial portions of the Materials.
+**
+** THE MATERIALS ARE PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+** EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
+** MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.
+** IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY
+** CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT,
+** TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE
+** MATERIALS OR THE USE OR OTHER DEALINGS IN THE MATERIALS.
+*/
+
+-->
+
+<!DOCTYPE html>
+<html>
+<head>
+<meta charset="utf-8">
+<link rel="stylesheet" href="../../../resources/js-test-style.css"/>
+<script src="resources/desktop-gl-constants.js" type="text/javascript"></script>
+<script src="../../../resources/js-test-pre.js"></script>
+<script src="resources/webgl-test.js"> </script>
+<script src="resources/webgl-test-utils.js"></script>
+<script src="resources/out-of-bounds-test.js"></script>
+</head>
+<body>
+<div id="description"></div>
+<canvas id="canvas" style="width: 50px; height: 50px;"></canvas>
+<div id="console"></div>
+
+<script>
+"use strict";
+description("Test of drawArraysInstancedANGLE and drawElementsInstancedANGLE with out-of-bounds parameters");
+
+var wtu = WebGLTestUtils;
+var canvas = document.getElementById("canvas");
+var gl = wtu.create3DContext(canvas);
+var ext = wtu.getExtensionWithKnownPrefixes(gl, "ANGLE_instanced_arrays");
+if (!ext) {
+    testPassed("No ANGLE_instanced_arrays support -- this is legal");
+} else {
+    testPassed("Successfully enabled ANGLE_instanced_arrays extension");
+    debug("");
+    debug("Test with 1 instance without instanced attributes");
+    debug("");
+    OutOfBoundsTest.runDrawArraysTest("ext.drawArraysInstancedANGLE(gl.TRIANGLES, $(offset), $(count), 1)", gl, wtu, ext);
+    debug("");
+    OutOfBoundsTest.runDrawElementsTest("ext.drawElementsInstancedANGLE(gl.TRIANGLES, $(count), $(type), $(offset), 1)", gl, wtu, ext);
+    debug("");
+    debug("Test with 2 instances without instanced attributes");
+    debug("");
+    OutOfBoundsTest.runDrawArraysTest("ext.drawArraysInstancedANGLE(gl.TRIANGLES, $(offset), $(count), 2)", gl, wtu, ext);
+    debug("");
+    OutOfBoundsTest.runDrawElementsTest("ext.drawElementsInstancedANGLE(gl.TRIANGLES, $(count), $(type), $(offset), 2)", gl, wtu, ext);
+    debug("");
+    OutOfBoundsTest.runDrawArraysInstancedTest("ext.drawArraysInstancedANGLE(gl.TRIANGLES, $(offset), $(count), $(primcount))", gl, wtu, ext);
+    debug("");
+    OutOfBoundsTest.runDrawElementsInstancedTest("ext.drawElementsInstancedANGLE(gl.TRIANGLES, $(count), $(type), $(offset), $(primcount))", gl, wtu, ext);
+    debug("");
+}
+
+var successfullyParsed = true;
+</script>
+
+<script src="../../../resources/js-test-post.js"></script>
+</body>
+</html>
diff --git a/LayoutTests/fast/canvas/webgl/resources/out-of-bounds-test.js b/LayoutTests/fast/canvas/webgl/resources/out-of-bounds-test.js
new file mode 100644 (file)
index 0000000..1da05eb
--- /dev/null
@@ -0,0 +1,329 @@
+/*
+** Copyright (c) 2014 The Khronos Group Inc.
+**
+** Permission is hereby granted, free of charge, to any person obtaining a
+** copy of this software and/or associated documentation files (the
+** "Materials"), to deal in the Materials without restriction, including
+** without limitation the rights to use, copy, modify, merge, publish,
+** distribute, sublicense, and/or sell copies of the Materials, and to
+** permit persons to whom the Materials are furnished to do so, subject to
+** the following conditions:
+**
+** The above copyright notice and this permission notice shall be included
+** in all copies or substantial portions of the Materials.
+**
+** THE MATERIALS ARE PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+** EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
+** MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.
+** IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY
+** CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT,
+** TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE
+** MATERIALS OR THE USE OR OTHER DEALINGS IN THE MATERIALS.
+*/
+
+'use strict';
+
+var OutOfBoundsTest = (function() {
+
+var runCommonInvalidValueTests = function(callTemplate, gl, wtu, ext) {
+    wtu.shouldGenerateGLError(gl, gl.INVALID_VALUE, wtu.replaceParams(callTemplate, {count: -1, type: 'gl.UNSIGNED_BYTE', offset: 0}));
+    wtu.shouldGenerateGLError(gl, gl.INVALID_VALUE, wtu.replaceParams(callTemplate, {count: 0, type: 'gl.UNSIGNED_BYTE', offset: -1}));
+    wtu.shouldGenerateGLError(gl, gl.INVALID_VALUE, wtu.replaceParams(callTemplate, {count: -1, type: 'gl.UNSIGNED_BYTE', offset: 1}));
+    wtu.shouldGenerateGLError(gl, gl.INVALID_VALUE, wtu.replaceParams(callTemplate, {count: 1, type: 'gl.UNSIGNED_BYTE', offset: -1}));
+    wtu.shouldGenerateGLError(gl, gl.INVALID_VALUE, wtu.replaceParams(callTemplate, {count: '0xffffffff', type: 'gl.UNSIGNED_BYTE', offset: 0}));
+};
+
+var setupProgramAndBindVertexArray = function(gl, wtu) {
+    var program = wtu.loadStandardProgram(gl);
+
+    gl.useProgram(program);
+    var vertexObject = gl.createBuffer();
+    gl.bindBuffer(gl.ARRAY_BUFFER, vertexObject);
+    gl.enableVertexAttribArray(0);
+
+    return program;
+};
+
+var setupProgram2 = function(gl, wtu) {
+    var vshader = [
+        'attribute vec3 aOne;',
+        'attribute vec2 aTwo;',
+        'void main() {',
+        '  gl_Position = vec4(aOne, 1.0) + vec4(aTwo, 0.0, 1.0);',
+        '}'
+    ].join('\n');
+
+    var fshader = [
+        'void main() {',
+        '  gl_FragColor = vec4(1.0, 0.0, 0.0, 1.0);',
+        '}'
+    ].join('\n');
+
+    var program = wtu.setupProgram(gl, [vshader, fshader], [ "aOne", "aTwo" ]);
+    if (!program) {
+        testFailed("failed to create test program");
+    }
+    return program;
+};
+
+var runDrawArraysTest = function(callTemplate, gl, wtu, ext) {
+    var program = setupProgramAndBindVertexArray(gl, wtu);
+
+    debug("Test empty buffer")
+    gl.bufferData(gl.ARRAY_BUFFER, new Float32Array([  ]), gl.STATIC_DRAW);
+    gl.vertexAttribPointer(0, 3, gl.FLOAT, false, 0, 0);
+    wtu.shouldGenerateGLError(gl, gl.INVALID_OPERATION, wtu.replaceParams(callTemplate, {offset: 0, count: 1}));
+    wtu.shouldGenerateGLError(gl, gl.INVALID_OPERATION, wtu.replaceParams(callTemplate, {offset: 0, count: 10000}));
+    wtu.shouldGenerateGLError(gl, gl.INVALID_OPERATION, wtu.replaceParams(callTemplate, {offset: 0, count: 10000000000000}));
+    wtu.shouldGenerateGLError(gl, gl.NO_ERROR, wtu.replaceParams(callTemplate, {offset: 1, count: 0}));
+    wtu.shouldGenerateGLError(gl, gl.NO_ERROR, wtu.replaceParams(callTemplate, {offset: 0, count: 0}));
+    wtu.shouldGenerateGLError(gl, gl.NO_ERROR, wtu.replaceParams(callTemplate, {offset: 100, count: 0}));
+    runCommonInvalidValueTests(callTemplate, gl, wtu, ext);
+
+    debug("")
+    debug("Test buffer with 3 float vectors")
+    gl.bufferData(gl.ARRAY_BUFFER, new Float32Array([ 0,0.5,0, -0.5,-0.5,0, 0.5,-0.5,0 ]), gl.STATIC_DRAW);
+    gl.vertexAttribPointer(0, 3, gl.FLOAT, false, 0, 0);
+    wtu.shouldGenerateGLError(gl, gl.NO_ERROR, wtu.replaceParams(callTemplate, {offset: 0, count: 3}));
+    wtu.shouldGenerateGLError(gl, gl.INVALID_OPERATION, wtu.replaceParams(callTemplate, {offset: 3, count: 2}));
+    wtu.shouldGenerateGLError(gl, gl.INVALID_OPERATION, wtu.replaceParams(callTemplate, {offset: 0, count: 10000}));
+    wtu.shouldGenerateGLError(gl, gl.INVALID_OPERATION, wtu.replaceParams(callTemplate, {offset: 0, count: 10000000000000}));
+    wtu.shouldGenerateGLError(gl, gl.NO_ERROR, wtu.replaceParams(callTemplate, {offset: 0, count: 0}));
+    wtu.shouldGenerateGLError(gl, gl.NO_ERROR, wtu.replaceParams(callTemplate, {offset: 100, count: 0}));
+    runCommonInvalidValueTests(callTemplate, gl, wtu, ext);
+
+    debug("")
+    debug("Test buffer with interleaved (3+2) float vectors")
+
+    setupProgram2(gl, wtu);
+
+    var vbo = gl.createBuffer();
+    gl.bindBuffer(gl.ARRAY_BUFFER, vbo);
+    // enough for 9 vertices, so 3 triangles
+    gl.bufferData(gl.ARRAY_BUFFER, new Float32Array(9*5), gl.STATIC_DRAW);
+
+    // bind first 3 elements, with a stride of 5 float elements
+    gl.vertexAttribPointer(0, 3, gl.FLOAT, false, 5*4, 0);
+    // bind 2 elements, starting after the first 3; same stride of 5 float elements
+    gl.vertexAttribPointer(1, 2, gl.FLOAT, false, 5*4, 3*4);
+
+    gl.enableVertexAttribArray(0);
+    gl.enableVertexAttribArray(1);
+
+    wtu.shouldGenerateGLError(gl, gl.NO_ERROR, wtu.replaceParams(callTemplate, {offset: 0, count: 9}));
+
+    // negative values must generate INVALID_VALUE; they can never be valid
+    wtu.shouldGenerateGLError(gl, gl.INVALID_VALUE, wtu.replaceParams(callTemplate, {offset: 0, count: -500}));
+    wtu.shouldGenerateGLError(gl, gl.INVALID_VALUE, wtu.replaceParams(callTemplate, {offset: -200, count: 1}));
+    wtu.shouldGenerateGLError(gl, gl.INVALID_VALUE, wtu.replaceParams(callTemplate, {offset: -200, count: -500}));
+
+    // 0xffffffff needs to convert to a 'long' IDL argument as -1, as per
+    // WebIDL 4.1.7.  JS ToInt32(0xffffffff) == -1. Thus INVALID_VALUE.
+    wtu.shouldGenerateGLError(gl, gl.INVALID_VALUE, wtu.replaceParams(callTemplate, {offset: 0, count: '0xffffffff'}));
+    wtu.shouldGenerateGLError(gl, gl.INVALID_VALUE, wtu.replaceParams(callTemplate, {offset: '0xffffffff', count: '0xffffffff'}));
+    wtu.shouldGenerateGLError(gl, gl.INVALID_VALUE, wtu.replaceParams(callTemplate, {offset: '0xffffffff', count: 1}));
+
+    // values that could otherwise be valid but aren't due to bindings generate
+    // INVALID_OPERATION
+    wtu.shouldGenerateGLError(gl, gl.INVALID_OPERATION, wtu.replaceParams(callTemplate, {offset: 0, count: 200}));
+    wtu.shouldGenerateGLError(gl, gl.INVALID_OPERATION, wtu.replaceParams(callTemplate, {offset: 0, count: '0x7fffffff'}));
+    wtu.shouldGenerateGLError(gl, gl.INVALID_OPERATION, wtu.replaceParams(callTemplate, {offset: '0x7fffffff', count: 1}));
+    wtu.shouldGenerateGLError(gl, gl.INVALID_OPERATION, wtu.replaceParams(callTemplate, {offset: '0x7fffffff', count: '0x7fffffff'}));
+};
+
+var runDrawElementsTest = function(callTemplate, gl, wtu, ext) {
+    var program = setupProgramAndBindVertexArray(gl, wtu);
+
+    gl.bufferData(gl.ARRAY_BUFFER, new Float32Array([ 0,0.5,0, -0.5,-0.5,0, 0.5,-0.5,0 ]), gl.STATIC_DRAW);
+    gl.vertexAttribPointer(0, 3, gl.FLOAT, false, 0, 0);
+
+    var indexObject = gl.createBuffer();
+
+    debug('Test empty index buffer');
+    gl.bindBuffer(gl.ELEMENT_ARRAY_BUFFER, indexObject);
+    gl.bufferData(gl.ELEMENT_ARRAY_BUFFER, new Uint8Array([  ]), gl.STATIC_DRAW);
+    wtu.shouldGenerateGLError(gl, gl.NO_ERROR, wtu.replaceParams(callTemplate, {count: 0, type: 'gl.UNSIGNED_BYTE', offset: 0}));
+    wtu.shouldGenerateGLError(gl, gl.INVALID_OPERATION, wtu.replaceParams(callTemplate, {count: 3, type: 'gl.UNSIGNED_BYTE', offset: 0}));
+    wtu.shouldGenerateGLError(gl, gl.INVALID_OPERATION, wtu.replaceParams(callTemplate, {count: 10000, type: 'gl.UNSIGNED_BYTE', offset: 0}));
+    wtu.shouldGenerateGLError(gl, gl.INVALID_OPERATION, wtu.replaceParams(callTemplate, {count: 10000000000000, type: 'gl.UNSIGNED_BYTE', offset: 0}));
+    wtu.shouldGenerateGLError(gl, gl.INVALID_OPERATION, wtu.replaceParams(callTemplate, {count: 1, type: 'gl.UNSIGNED_BYTE', offset: 0}));
+    runCommonInvalidValueTests(callTemplate, gl, wtu, ext);
+
+    debug('');
+    debug('Test buffer with 3 byte indexes');
+    gl.bindBuffer(gl.ELEMENT_ARRAY_BUFFER, indexObject);
+    gl.bufferData(gl.ELEMENT_ARRAY_BUFFER, new Uint8Array([ 0, 1, 2 ]), gl.STATIC_DRAW);
+    wtu.shouldGenerateGLError(gl, gl.NO_ERROR, wtu.replaceParams(callTemplate, {count: 3, type: 'gl.UNSIGNED_BYTE', offset: 0}));
+    wtu.shouldGenerateGLError(gl, gl.INVALID_OPERATION, wtu.replaceParams(callTemplate, {count: 3, type: 'gl.UNSIGNED_BYTE', offset: 2}));
+    wtu.shouldGenerateGLError(gl, gl.INVALID_OPERATION, wtu.replaceParams(callTemplate, {count: 10000, type: 'gl.UNSIGNED_BYTE', offset: 0}));
+    wtu.shouldGenerateGLError(gl, gl.INVALID_OPERATION, wtu.replaceParams(callTemplate, {count: 10000000000000, type: 'gl.UNSIGNED_BYTE', offset: 0}));
+    runCommonInvalidValueTests(callTemplate, gl, wtu, ext);
+    wtu.shouldGenerateGLError(gl, gl.NO_ERROR, wtu.replaceParams(callTemplate, {count: 0, type: 'gl.UNSIGNED_BYTE', offset: 4}));
+    wtu.shouldGenerateGLError(gl, gl.INVALID_OPERATION, wtu.replaceParams(callTemplate, {count: '0x7fffffff', type: 'gl.UNSIGNED_BYTE', offset: 0}));
+    wtu.shouldGenerateGLError(gl, gl.INVALID_OPERATION, wtu.replaceParams(callTemplate, {count: '0x7fffffff', type: 'gl.UNSIGNED_BYTE', offset: '0x7fffffff'}));
+
+    wtu.shouldGenerateGLError(gl, gl.NO_ERROR, 'gl.bufferData(gl.ELEMENT_ARRAY_BUFFER, (new Uint8Array([ 3, 0, 1, 2 ])).subarray(1), gl.STATIC_DRAW)');
+    wtu.shouldGenerateGLError(gl, gl.NO_ERROR, wtu.replaceParams(callTemplate, {count: 3, type: 'gl.UNSIGNED_BYTE', offset: 0}));
+    wtu.shouldGenerateGLError(gl, gl.NO_ERROR, 'gl.bufferSubData(gl.ELEMENT_ARRAY_BUFFER, 0, new Uint8Array([ 3, 0, 1]))');
+    wtu.shouldGenerateGLError(gl, gl.INVALID_OPERATION, wtu.replaceParams(callTemplate, {count: 3, type: 'gl.UNSIGNED_BYTE', offset: 0}));
+    wtu.shouldGenerateGLError(gl, gl.NO_ERROR, 'gl.bufferSubData(gl.ELEMENT_ARRAY_BUFFER, 0, (new Uint8Array([ 3, 0, 1, 2 ])).subarray(1))');
+    wtu.shouldGenerateGLError(gl, gl.NO_ERROR, wtu.replaceParams(callTemplate, {count: 3, type: 'gl.UNSIGNED_BYTE', offset: 0}));
+    wtu.shouldGenerateGLError(gl, gl.NO_ERROR, wtu.replaceParams(callTemplate, {count: 0, type: 'gl.UNSIGNED_BYTE', offset: 0}));
+
+    debug('');
+    debug('Test buffer with interleaved (3+2) float vectors');
+
+    setupProgram2(gl, wtu);
+
+    var vbo = gl.createBuffer();
+    gl.bindBuffer(gl.ARRAY_BUFFER, vbo);
+    // enough for 9 vertices, so 3 triangles
+    gl.bufferData(gl.ARRAY_BUFFER, new Float32Array(9*5), gl.STATIC_DRAW);
+
+    // bind first 3 elements, with a stride of 5 float elements
+    gl.vertexAttribPointer(0, 3, gl.FLOAT, false, 5*4, 0);
+    // bind 2 elements, starting after the first 3; same stride of 5 float elements
+    gl.vertexAttribPointer(1, 2, gl.FLOAT, false, 5*4, 3*4);
+
+    gl.enableVertexAttribArray(0);
+    gl.enableVertexAttribArray(1);
+
+    var ebo = gl.createBuffer();
+    gl.bindBuffer(gl.ELEMENT_ARRAY_BUFFER, ebo);
+    gl.bufferData(gl.ELEMENT_ARRAY_BUFFER, new Uint16Array(
+        [ 0, 1, 2,
+          1, 2, 0,
+          2, 0, 1,
+          200, 200, 200,
+          0x7fff, 0x7fff, 0x7fff,
+          0xffff, 0xffff, 0xffff ]),
+        gl.STATIC_DRAW);
+
+    wtu.shouldGenerateGLError(gl, gl.NO_ERROR, wtu.replaceParams(callTemplate, {count: 9, type: 'gl.UNSIGNED_SHORT', offset: 0}));
+
+
+    // invalid operation with indices that would be valid with correct bindings
+    wtu.shouldGenerateGLError(gl, gl.INVALID_OPERATION, wtu.replaceParams(callTemplate, {count: 9, type: 'gl.UNSIGNED_SHORT', offset: 1000}));
+    wtu.shouldGenerateGLError(gl, gl.INVALID_OPERATION, wtu.replaceParams(callTemplate, {count: 12, type: 'gl.UNSIGNED_SHORT', offset: 0}));
+    wtu.shouldGenerateGLError(gl, gl.INVALID_OPERATION, wtu.replaceParams(callTemplate, {count: 15, type: 'gl.UNSIGNED_SHORT', offset: 0}));
+    wtu.shouldGenerateGLError(gl, gl.INVALID_OPERATION, wtu.replaceParams(callTemplate, {count: 18, type: 'gl.UNSIGNED_SHORT', offset: 0}));
+    wtu.shouldGenerateGLError(gl, gl.INVALID_OPERATION, wtu.replaceParams(callTemplate, {count: 3, type: 'gl.UNSIGNED_SHORT', offset: 2*15}));
+
+    // 0xffffffff needs to convert to a 'long' IDL argument as -1, as per
+    // WebIDL 4.1.7.  JS ToInt32(0xffffffff) == -1. Thus INVALID_VALUE.
+    wtu.shouldGenerateGLError(gl, gl.INVALID_VALUE, wtu.replaceParams(callTemplate, {count: '0xffffffff', type: 'gl.UNSIGNED_SHORT', offset: 0}));
+    // offset is defined as GLintptr, which is long long in IDL (64-bit).
+    // 2^32 - 1 should not overflow, and only result in INVALID_OPERATION.
+    wtu.shouldGenerateGLError(gl, gl.INVALID_OPERATION, wtu.replaceParams(callTemplate, {count: 1, type: 'gl.UNSIGNED_SHORT', offset: '0xffffffff'}));
+
+    wtu.shouldGenerateGLError(gl, gl.INVALID_OPERATION, wtu.replaceParams(callTemplate, {count: '0x7fffffff', type: 'gl.UNSIGNED_SHORT', offset: 0}));
+
+    wtu.shouldGenerateGLError(gl, gl.NO_ERROR, wtu.replaceParams(callTemplate, {count: 0, type: 'gl.UNSIGNED_SHORT', offset: 0}));
+
+    // invalid operation with offset that's not a multiple of the type size
+    wtu.shouldGenerateGLError(gl, gl.NO_ERROR, wtu.replaceParams(callTemplate, {count: 6, type: 'gl.UNSIGNED_SHORT', offset: 0}));
+    wtu.shouldGenerateGLError(gl, gl.INVALID_OPERATION, wtu.replaceParams(callTemplate, {count: 6, type: 'gl.UNSIGNED_SHORT', offset: 1}));
+    wtu.shouldGenerateGLError(gl, gl.NO_ERROR, wtu.replaceParams(callTemplate, {count: 6, type: 'gl.UNSIGNED_SHORT', offset: 2}));
+
+    // invalid operation if no buffer is bound to ELEMENT_ARRAY_BUFFER
+    gl.bindBuffer(gl.ELEMENT_ARRAY_BUFFER, null);
+    wtu.shouldGenerateGLError(gl, gl.INVALID_OPERATION, wtu.replaceParams(callTemplate, {count: 6, type: 'gl.UNSIGNED_SHORT', offset: 0}));
+    gl.bindBuffer(gl.ELEMENT_ARRAY_BUFFER, ebo);
+
+    debug('');
+    debug('Test buffer setting attrib 0 to a buffer too small and disable it.');
+    var smallVBO = gl.createBuffer();
+    gl.bindBuffer(gl.ARRAY_BUFFER, smallVBO);
+    gl.bufferData(gl.ARRAY_BUFFER, 1, gl.STATIC_DRAW);
+    gl.vertexAttribPointer(0, 3, gl.FLOAT, false, 0, 0x10);
+    gl.disableVertexAttribArray(0);
+    wtu.shouldGenerateGLError(gl, gl.NO_ERROR, wtu.replaceParams(callTemplate, {count: 6, type: 'gl.UNSIGNED_SHORT', offset: 2}));
+    gl.bindBuffer(gl.ELEMENT_ARRAY_BUFFER, null);
+    wtu.shouldGenerateGLError(gl, gl.INVALID_OPERATION, wtu.replaceParams(callTemplate, {count: 6, type: 'gl.UNSIGNED_SHORT', offset: 2}));
+};
+
+var runInstancedTest = function(callTemplate, gl, wtu, ext) {
+    setupProgram2(gl, wtu);
+
+    // Initialize non-instanced attribute data.
+    // Enough for 9 vertices, so 3 triangles.
+    var vbo = gl.createBuffer();
+    gl.bindBuffer(gl.ARRAY_BUFFER, vbo);
+    gl.bufferData(gl.ARRAY_BUFFER, new Float32Array(9*3), gl.STATIC_DRAW);
+
+    gl.vertexAttribPointer(0, 3, gl.FLOAT, false, 0, 0);
+
+    // Setup buffer for instanced attribute data.
+    var vbo2 = gl.createBuffer();
+    gl.bindBuffer(gl.ARRAY_BUFFER, vbo2);
+    gl.vertexAttribPointer(1, 2, gl.FLOAT, false, 0, 0);
+
+    gl.enableVertexAttribArray(0);
+    gl.enableVertexAttribArray(1);
+
+    debug('Test out-of-range instanced attributes');
+    debug('');
+
+    debug('Test with an empty buffer for the instanced attribute');
+    ext.vertexAttribDivisorANGLE(1, 1);
+    wtu.shouldGenerateGLError(gl, gl.NO_ERROR, wtu.replaceParams(callTemplate, {offset: 0, count: 9, primcount: 0}));
+    wtu.shouldGenerateGLError(gl, gl.NO_ERROR, wtu.replaceParams(callTemplate, {offset: 0, count: 10000, primcount: 0}));
+    wtu.shouldGenerateGLError(gl, gl.INVALID_OPERATION, wtu.replaceParams(callTemplate, {offset: 0, count: 9, primcount: 1}));
+
+    debug('Test with a buffer with 1 float for the instanced attribute');
+    gl.bufferData(gl.ARRAY_BUFFER, new Float32Array(1), gl.STATIC_DRAW);
+    wtu.shouldGenerateGLError(gl, gl.NO_ERROR, wtu.replaceParams(callTemplate, {offset: 0, count: 9, primcount: 0}));
+    wtu.shouldGenerateGLError(gl, gl.NO_ERROR, wtu.replaceParams(callTemplate, {offset: 0, count: 10000, primcount: 0}));
+    wtu.shouldGenerateGLError(gl, gl.INVALID_OPERATION, wtu.replaceParams(callTemplate, {offset: 0, count: 9, primcount: 1}));
+
+    debug('');
+    debug('Test with a buffer with 2 floats for the instanced attribute');
+    gl.bufferData(gl.ARRAY_BUFFER, new Float32Array(2), gl.STATIC_DRAW);
+    debug('Divisor 1');
+    ext.vertexAttribDivisorANGLE(1, 1);
+    wtu.shouldGenerateGLError(gl, gl.NO_ERROR, wtu.replaceParams(callTemplate, {offset: 0, count: 9, primcount: 1}));
+    wtu.shouldGenerateGLError(gl, gl.INVALID_OPERATION, wtu.replaceParams(callTemplate, {offset: 0, count: 9, primcount: 2}));
+    debug('Divisor 3');
+    ext.vertexAttribDivisorANGLE(1, 3);
+    wtu.shouldGenerateGLError(gl, gl.NO_ERROR, wtu.replaceParams(callTemplate, {offset: 0, count: 9, primcount: 3}));
+    wtu.shouldGenerateGLError(gl, gl.INVALID_OPERATION, wtu.replaceParams(callTemplate, {offset: 0, count: 9, primcount: 4}));
+    wtu.shouldGenerateGLError(gl, gl.INVALID_OPERATION, wtu.replaceParams(callTemplate, {offset: 0, count: 9, primcount: 10000}));
+    wtu.shouldGenerateGLError(gl, gl.INVALID_OPERATION, wtu.replaceParams(callTemplate, {offset: 0, count: 9, primcount: '0x7fffffff'}));
+
+    debug('');
+    debug('Test with a buffer with 4 floats for the instanced attribute');
+    gl.bufferData(gl.ARRAY_BUFFER, new Float32Array(4), gl.STATIC_DRAW);
+    debug('Divisor 1');
+    ext.vertexAttribDivisorANGLE(1, 1);
+    wtu.shouldGenerateGLError(gl, gl.NO_ERROR, wtu.replaceParams(callTemplate, {offset: 0, count: 9, primcount: 2}));
+    wtu.shouldGenerateGLError(gl, gl.INVALID_OPERATION, wtu.replaceParams(callTemplate, {offset: 0, count: 9, primcount: 3}));
+    debug('Divisor 2');
+    ext.vertexAttribDivisorANGLE(1, 2);
+    wtu.shouldGenerateGLError(gl, gl.NO_ERROR, wtu.replaceParams(callTemplate, {offset: 0, count: 9, primcount: 4}));
+    wtu.shouldGenerateGLError(gl, gl.INVALID_OPERATION, wtu.replaceParams(callTemplate, {offset: 0, count: 9, primcount: 5}));
+};
+
+var runDrawArraysInstancedTest = function(callTemplate, gl, wtu, ext) {
+    runInstancedTest(callTemplate, gl, wtu, ext);
+};
+
+var runDrawElementsInstancedTest = function(callTemplate, gl, wtu, ext) {
+    var ebo = gl.createBuffer();
+    gl.bindBuffer(gl.ELEMENT_ARRAY_BUFFER, ebo);
+    gl.bufferData(gl.ELEMENT_ARRAY_BUFFER, new Uint8Array(
+        [ 0, 1, 2,
+          5, 4, 3,
+          6, 7, 8 ]),
+        gl.STATIC_DRAW);
+    callTemplate = wtu.replaceParams(callTemplate, {type: 'gl.UNSIGNED_BYTE', offset: '$(offset)', count: '$(count)', primcount: '$(primcount)'});
+    runInstancedTest(callTemplate, gl, wtu, ext);
+};
+
+return {
+    runDrawArraysTest: runDrawArraysTest,
+    runDrawArraysInstancedTest: runDrawArraysInstancedTest,
+    runDrawElementsTest: runDrawElementsTest,
+    runDrawElementsInstancedTest: runDrawElementsInstancedTest
+};
+
+})();
diff --git a/LayoutTests/platform/mac-mountainlion/fast/canvas/webgl/angle-instanced-arrays-out-of-bounds-expected.txt b/LayoutTests/platform/mac-mountainlion/fast/canvas/webgl/angle-instanced-arrays-out-of-bounds-expected.txt
new file mode 100644 (file)
index 0000000..025e45c
--- /dev/null
@@ -0,0 +1,9 @@
+Test of drawArraysInstancedANGLE and drawElementsInstancedANGLE with out-of-bounds parameters
+
+On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
+
+ PASS No ANGLE_instanced_arrays support -- this is legal
+PASS successfullyParsed is true
+
+TEST COMPLETE
+
index 6c12ee4..2dcca64 100644 (file)
@@ -1,3 +1,29 @@
+2014-12-10  Dean Jackson  <dino@apple.com>
+
+        InstancedArray crashes attempting to draw out of bounds
+        https://bugs.webkit.org/show_bug.cgi?id=139521
+
+        Reviewed by Simon Fraser.
+
+        We were not doing index validation correctly for instancing.
+
+        Test: fast/canvas/webgl/angle-instanced-arrays-out-of-bounds.html
+
+        * html/canvas/WebGLRenderingContext.cpp:
+        (WebCore::WebGLRenderingContext::validateVertexAttributes): We need to check
+        the number of instances drawn against the amount of instance data that has
+        been provided, taking into account the number of repeats (the divisor).
+        (WebCore::WebGLRenderingContext::drawArrays): Added some whitespace to make it more clear.
+        (WebCore::WebGLRenderingContext::validateDrawElements): This needs to take a primcount
+        parameter so that it can correctly validate the call (when used from drawElementsInstanced).
+        (WebCore::WebGLRenderingContext::drawElements): New signature to validate.
+        (WebCore::WebGLRenderingContext::drawArraysInstanced): Rearrange this a bit. The
+        primcount validation is already being done by the validateDrawArrays call. Also, there
+        was a bogus UNUSED_PARAM hanging around.
+        (WebCore::WebGLRenderingContext::drawElementsInstanced): Similar rearrangement. Use
+        the primcount parameter.
+        * html/canvas/WebGLRenderingContext.h:
+
 2014-12-11  Anders Carlsson  <andersca@apple.com>
 
         Remove unused storage header includes
index 6b8bb65..67e001b 100644 (file)
@@ -1954,10 +1954,16 @@ bool WebGLRenderingContext::validateVertexAttributes(unsigned elementCount, unsi
                 ASSERT(state.stride > 0);
                 if (bytesRemaining >= state.bytesPerElement)
                     numElements = 1 + (bytesRemaining - state.bytesPerElement) / state.stride;
-                if (!state.divisor)
+                unsigned instancesRequired = 0;
+                if (state.divisor) {
+                    instancesRequired = ceil(static_cast<float>(primitiveCount) / state.divisor);
+                    if (instancesRequired > numElements)
+                        return false;
+                } else {
                     sawNonInstancedAttrib = true;
-                if ((!state.divisor && elementCount > numElements) || (state.divisor && primitiveCount > numElements))
-                    return false;
+                    if (elementCount > numElements)
+                        return false;
+                }
             }
         }
     }
@@ -1981,7 +1987,7 @@ bool WebGLRenderingContext::validateWebGLObject(const char* functionName, WebGLO
     return true;
 }
 
-bool WebGLRenderingContext::validateDrawArrays(const char* functionName, GC3Denum mode, GC3Dint first, GC3Dsizei count, GC3Dsizei primcount)
+bool WebGLRenderingContext::validateDrawArrays(const char* functionName, GC3Denum mode, GC3Dint first, GC3Dsizei count, GC3Dsizei primitiveCount)
 {
     if (isContextLostOrPending() || !validateDrawMode(functionName, mode))
         return false;
@@ -1999,7 +2005,7 @@ bool WebGLRenderingContext::validateDrawArrays(const char* functionName, GC3Denu
         return false;
     }
 
-    if (primcount < 0) {
+    if (primitiveCount < 0) {
         synthesizeGLError(GraphicsContext3D::INVALID_VALUE, functionName, "primcount < 0");
         return false;
     }
@@ -2009,8 +2015,8 @@ bool WebGLRenderingContext::validateDrawArrays(const char* functionName, GC3Denu
         Checked<GC3Dint, RecordOverflow> checkedFirst(first);
         Checked<GC3Dint, RecordOverflow> checkedCount(count);
         Checked<GC3Dint, RecordOverflow> checkedSum = checkedFirst + checkedCount;
-        Checked<GC3Dint, RecordOverflow> checkedPrimCount(primcount);
-        if (checkedSum.hasOverflowed() || checkedPrimCount.hasOverflowed() || !validateVertexAttributes(checkedSum.unsafeGet(), checkedPrimCount.unsafeGet())) {
+        Checked<GC3Dint, RecordOverflow> checkedPrimitiveCount(primitiveCount);
+        if (checkedSum.hasOverflowed() || checkedPrimitiveCount.hasOverflowed() || !validateVertexAttributes(checkedSum.unsafeGet(), checkedPrimitiveCount.unsafeGet())) {
             synthesizeGLError(GraphicsContext3D::INVALID_OPERATION, functionName, "attempt to access out of bounds arrays");
             return false;
         }
@@ -2044,7 +2050,9 @@ void WebGLRenderingContext::drawArrays(GC3Denum mode, GC3Dint first, GC3Dsizei c
         vertexAttrib0Simulated = simulateVertexAttrib0(first + count - 1);
     if (!isGLES2NPOTStrict())
         checkTextureCompleteness("drawArrays", true);
+
     m_context->drawArrays(mode, first, count);
+
     if (!isGLES2Compliant() && vertexAttrib0Simulated)
         restoreStatesAfterVertexAttrib0Simulation();
     if (!isGLES2NPOTStrict())
@@ -2052,7 +2060,7 @@ void WebGLRenderingContext::drawArrays(GC3Denum mode, GC3Dint first, GC3Dsizei c
     markContextChanged();
 }
 
-bool WebGLRenderingContext::validateDrawElements(const char* functionName, GC3Denum mode, GC3Dsizei count, GC3Denum type, long long offset, unsigned& numElements)
+bool WebGLRenderingContext::validateDrawElements(const char* functionName, GC3Denum mode, GC3Dsizei count, GC3Denum type, long long offset, unsigned& numElements, GC3Dsizei primitiveCount)
 {
     if (isContextLostOrPending() || !validateDrawMode(functionName, mode))
         return false;
@@ -2084,6 +2092,11 @@ bool WebGLRenderingContext::validateDrawElements(const char* functionName, GC3De
         return false;
     }
 
+    if (primitiveCount < 0) {
+        synthesizeGLError(GraphicsContext3D::INVALID_VALUE, functionName, "primcount < 0");
+        return false;
+    }
+
     if (!m_boundVertexArrayObject->getElementArrayBuffer()) {
         synthesizeGLError(GraphicsContext3D::INVALID_OPERATION, functionName, "no ELEMENT_ARRAY_BUFFER bound");
         return false;
@@ -2097,8 +2110,16 @@ bool WebGLRenderingContext::validateDrawElements(const char* functionName, GC3De
         }
         if (!count)
             return false;
-        if (!validateIndexArrayConservative(type, numElements) || !validateVertexAttributes(numElements)) {
-            if (!validateIndexArrayPrecise(count, type, static_cast<GC3Dintptr>(offset), numElements) || !validateVertexAttributes(numElements)) {
+
+        Checked<GC3Dint, RecordOverflow> checkedCount(count);
+        Checked<GC3Dint, RecordOverflow> checkedPrimitiveCount(primitiveCount);
+        if (checkedCount.hasOverflowed() || checkedPrimitiveCount.hasOverflowed()) {
+            synthesizeGLError(GraphicsContext3D::INVALID_OPERATION, functionName, "attempt to access out of bounds arrays");
+            return false;
+        }
+
+        if (!validateIndexArrayConservative(type, numElements) || !validateVertexAttributes(numElements, checkedPrimitiveCount.unsafeGet())) {
+            if (!validateIndexArrayPrecise(checkedCount.unsafeGet(), type, static_cast<GC3Dintptr>(offset), numElements) || !validateVertexAttributes(numElements, checkedPrimitiveCount.unsafeGet())) {
                 synthesizeGLError(GraphicsContext3D::INVALID_OPERATION, functionName, "attempt to access out of bounds arrays");
                 return false;
             }
@@ -2124,7 +2145,7 @@ void WebGLRenderingContext::drawElements(GC3Denum mode, GC3Dsizei count, GC3Denu
     UNUSED_PARAM(ec);
 
     unsigned numElements = 0;
-    if (!validateDrawElements("drawElements", mode, count, type, offset, numElements))
+    if (!validateDrawElements("drawElements", mode, count, type, offset, numElements, 0))
         return;
 
     clearIfComposited();
@@ -2137,7 +2158,9 @@ void WebGLRenderingContext::drawElements(GC3Denum mode, GC3Dsizei count, GC3Denu
     }
     if (!isGLES2NPOTStrict())
         checkTextureCompleteness("drawElements", true);
+
     m_context->drawElements(mode, count, type, static_cast<GC3Dintptr>(offset));
+
     if (!isGLES2Compliant() && vertexAttrib0Simulated)
         restoreStatesAfterVertexAttrib0Simulation();
     if (!isGLES2NPOTStrict())
@@ -6211,19 +6234,14 @@ bool WebGLRenderingContext::supportsDrawBuffers()
 
 void WebGLRenderingContext::drawArraysInstanced(GC3Denum mode, GC3Dint first, GC3Dsizei count, GC3Dsizei primcount)
 {
-    if (!validateDrawArrays("drawArraysInstanced", mode, first, count, primcount))
-        return;
-
-    if (primcount < 0) {
-        synthesizeGLError(GraphicsContext3D::INVALID_VALUE, "drawArraysInstanced", "primcount < 0");
-        return;
-    }
-
     if (!primcount) {
         markContextChanged();
         return;
     }
 
+    if (!validateDrawArrays("drawArraysInstanced", mode, first, count, primcount))
+        return;
+
     clearIfComposited();
 
     bool vertexAttrib0Simulated = false;
@@ -6231,8 +6249,9 @@ void WebGLRenderingContext::drawArraysInstanced(GC3Denum mode, GC3Dint first, GC
         vertexAttrib0Simulated = simulateVertexAttrib0(first + count - 1);
     if (!isGLES2NPOTStrict())
         checkTextureCompleteness("drawArraysInstanced", true);
-    UNUSED_PARAM(primcount);
+
     m_context->drawArraysInstanced(mode, first, count, primcount);
+
     if (!isGLES2Compliant() && vertexAttrib0Simulated)
         restoreStatesAfterVertexAttrib0Simulation();
     if (!isGLES2NPOTStrict())
@@ -6242,20 +6261,15 @@ void WebGLRenderingContext::drawArraysInstanced(GC3Denum mode, GC3Dint first, GC
 
 void WebGLRenderingContext::drawElementsInstanced(GC3Denum mode, GC3Dsizei count, GC3Denum type, long long offset, GC3Dsizei primcount)
 {
-    unsigned numElements = 0;
-    if (!validateDrawElements("drawElementsInstanced", mode, count, type, offset, numElements))
-        return;
-
-    if (primcount < 0) {
-        synthesizeGLError(GraphicsContext3D::INVALID_VALUE, "drawElementsInstanced", "primcount < 0");
-        return;
-    }
-
     if (!primcount) {
         markContextChanged();
         return;
     }
 
+    unsigned numElements = 0;
+    if (!validateDrawElements("drawElementsInstanced", mode, count, type, offset, numElements, primcount))
+        return;
+
     clearIfComposited();
 
     bool vertexAttrib0Simulated = false;
@@ -6266,7 +6280,9 @@ void WebGLRenderingContext::drawElementsInstanced(GC3Denum mode, GC3Dsizei count
     }
     if (!isGLES2NPOTStrict())
         checkTextureCompleteness("drawElementsInstanced", true);
+
     m_context->drawElementsInstanced(mode, count, type, static_cast<GC3Dintptr>(offset), primcount);
+
     if (!isGLES2Compliant() && vertexAttrib0Simulated)
         restoreStatesAfterVertexAttrib0Simulation();
     if (!isGLES2NPOTStrict())
index 89bfe39..251b576 100644 (file)
@@ -389,7 +389,7 @@ private:
     bool validateWebGLObject(const char*, WebGLObject*);
 
     bool validateDrawArrays(const char* functionName, GC3Denum mode, GC3Dint first, GC3Dsizei count, GC3Dsizei primcount);
-    bool validateDrawElements(const char* functionName, GC3Denum mode, GC3Dsizei count, GC3Denum type, long long offset, unsigned& numElements);
+    bool validateDrawElements(const char* functionName, GC3Denum mode, GC3Dsizei count, GC3Denum type, long long offset, unsigned& numElements, GC3Dsizei primcount);
 
     // Adds a compressed texture format.
     void addCompressedTextureFormat(GC3Denum);