Added overloads of uniform[1234]uiv taking Uint32Array to match spec.

Added overloads of uniform[1234]uiv taking Uint32Array to match spec.

Also fixed typo in validation for uniform1uiv causing crashes in test.

BUG=295792
Committed: https://crrev.com/ed4c93ce00a78fc6625573adb86c85e93aff193b
Cr-Commit-Position: refs/heads/master@{#352159}

[Ken Russell (switch to Gerrit), 2015-09-30 21:43:37]

kbr@chromium.org changed reviewers:
+ bajones@chromium.org, zmo@chromium.org

[Ken Russell (switch to Gerrit), 2015-09-30 21:43:38]

Please review.

Tested with deqp/functional/gles3/uniformapi.html test. It makes more cases pass
but the overall test still doesn't pass yet.

https://chromiumcodereview.appspot.com/1379863003/diff/1/third_party/WebKit/S...
File third_party/WebKit/Source/modules/webgl/WebGLRenderingContextBase.cpp
(left):

https://chromiumcodereview.appspot.com/1379863003/diff/1/third_party/WebKit/S...
third_party/WebKit/Source/modules/webgl/WebGLRenderingContextBase.cpp:6085: bool
WebGLRenderingContextBase::validateUniformParameters(const char* functionName,
const WebGLUniformLocation* location, const
TypedFlexibleArrayBufferView<WTFTypedArray>& v, GLsizei requiredMinSize)
Had to be moved to the header in order to let WebGL2RenderingContextBase define
new instantiations of this template.

https://chromiumcodereview.appspot.com/1379863003/diff/1/third_party/WebKit/S...
File third_party/WebKit/Source/modules/webgl/WebGLRenderingContextBase.h (left):

https://chromiumcodereview.appspot.com/1379863003/diff/1/third_party/WebKit/S...
third_party/WebKit/Source/modules/webgl/WebGLRenderingContextBase.h:597:
ImageBuffer* imageBuffer(const IntSize& size);
This change was required to pass the presubmit check.

https://chromiumcodereview.appspot.com/1379863003/diff/1/third_party/WebKit/S...
third_party/WebKit/Source/modules/webgl/WebGLRenderingContextBase.h:1046: void
bufferSubDataImpl(GLenum target, long long offset, GLsizeiptr size, const void*
data);
This one too.

[Zhenyao Mo, 2015-09-30 21:54:19]

LGTM

[bajones, 2015-09-30 22:23:30]

LGTM too.

[Ken Russell (switch to Gerrit), 2015-09-30 22:29:42]

kbr@chromium.org changed reviewers:
+ jochen@chromium.org

[Ken Russell (switch to Gerrit), 2015-09-30 22:29:43]

+jochen for FlexibleArrayBufferView change

[jochen (gone - plz use gerrit), 2015-10-02 13:24:06]

lgtm

[Ken Russell (switch to Gerrit), 2015-10-02 21:00:53]

The CQ bit was checked by kbr@chromium.org

[Ken Russell (switch to Gerrit), 2015-10-02 21:00:53]

The patchset sent to the CQ was uploaded after l-g-t-m from zmo@chromium.org,
bajones@chromium.org
Link to the patchset: https://codereview.chromium.org/1379863003/#ps20001
(title: "Fixed typo causing crashes in uniform API tests.")

[commit-bot: I haz the power, 2015-10-02 21:02:10]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1379863003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1379863003/20001

[commit-bot: I haz the power, 2015-10-02 22:47:20]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-10-02 22:48:47]

Patchset 2 (id:??) landed as
https://crrev.com/ed4c93ce00a78fc6625573adb86c85e93aff193b
Cr-Commit-Position: refs/heads/master@{#352159}