Explicitly validate sizes before allocating vectors.

Explicitly validate sizes before allocating vectors.

GLsizei validation is not auto-generated in the passthrough command
decoder and is left up to ANGLE's validation but some entry points need
to make copies of the data before making GL calls.  Make sure the
GLsizei is non-negative in these cases.

BUG=602737
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

Review-Url: https://codereview.chromium.org/2687393003
Cr-Commit-Position: refs/heads/master@{#449864}
Committed: https://chromium.googlesource.com/chromium/src/+/3dbb71ef483227725d0167e2fc602a64b07d02f8

[Geoff Lang, 2017-02-10 20:36:53]

Description was changed from

==========
Explicitly validate sizes before allocating vectors.

GLsizei validation is not auto-generated in the passthrough command
decoder and is left up to ANGLE's validation but some entry points need
to make copies of the data before making GL calls.  Make sure the
GLsizei is non-negative in these cases.

BUG=602737
==========

to

==========
Explicitly validate sizes before allocating vectors.

GLsizei validation is not auto-generated in the passthrough command
decoder and is left up to ANGLE's validation but some entry points need
to make copies of the data before making GL calls.  Make sure the
GLsizei is non-negative in these cases.

BUG=602737
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[Geoff Lang, 2017-02-10 20:42:32]

Description was changed from

==========
Explicitly validate sizes before allocating vectors.

GLsizei validation is not auto-generated in the passthrough command
decoder and is left up to ANGLE's validation but some entry points need
to make copies of the data before making GL calls.  Make sure the
GLsizei is non-negative in these cases.

BUG=602737
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
Explicitly validate sizes before allocating vectors.

GLsizei validation is not auto-generated in the passthrough command
decoder and is left up to ANGLE's validation but some entry points need
to make copies of the data before making GL calls.  Make sure the
GLsizei is non-negative in these cases.

BUG=602737
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[Geoff Lang, 2017-02-10 20:42:32]

geofflang@chromium.org changed reviewers:
+ zmo@chromium.org

[Geoff Lang, 2017-02-10 20:42:48]

PTAL

[Geoff Lang, 2017-02-10 20:42:52]

The CQ bit was checked by geofflang@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-02-10 20:43:56]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Zhenyao Mo, 2017-02-10 20:56:08]

On 2017/02/10 20:42:48, Geoff Lang wrote:
> PTAL

lgtm

[commit-bot: I haz the power, 2017-02-10 22:07:26]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-02-10 22:07:27]

Dry run: This issue passed the CQ dry run.

[Geoff Lang, 2017-02-11 17:00:26]

The CQ bit was checked by geofflang@chromium.org

[commit-bot: I haz the power, 2017-02-11 17:00:36]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-11 17:04:13]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1486832426717810, "parent_rev":
"559c4b2fba5e962a26396a952b79bf448de3160c", "commit_rev":
"3dbb71ef483227725d0167e2fc602a64b07d02f8"}

[commit-bot: I haz the power, 2017-02-11 17:04:48]

Description was changed from

==========
Explicitly validate sizes before allocating vectors.

GLsizei validation is not auto-generated in the passthrough command
decoder and is left up to ANGLE's validation but some entry points need
to make copies of the data before making GL calls.  Make sure the
GLsizei is non-negative in these cases.

BUG=602737
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
Explicitly validate sizes before allocating vectors.

GLsizei validation is not auto-generated in the passthrough command
decoder and is left up to ANGLE's validation but some entry points need
to make copies of the data before making GL calls.  Make sure the
GLsizei is non-negative in these cases.

BUG=602737
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

Review-Url: https://codereview.chromium.org/2687393003
Cr-Commit-Position: refs/heads/master@{#449864}
Committed:
https://chromium.googlesource.com/chromium/src/+/3dbb71ef483227725d0167e2fc60...
==========

[commit-bot: I haz the power, 2017-02-11 17:04:49]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/3dbb71ef483227725d0167e2fc60...