Check for invalid dimensions in glTexStorage*

Check for invalid dimensions in glTexStorage*

width, height, or depth of 0 are invalid for glTexStorage, but valid for
most other texture entry points, so this case wasn't covered by
TextureManager::ValidForTarget.

BUG=700270
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;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/2751553002
Cr-Commit-Position: refs/heads/master@{#456615}
Committed: https://chromium.googlesource.com/chromium/src/+/87b070a02a9413d0b48fd7ebd0d1c271c2f657c5

[Kai Ninomiya, 2017-03-13 21:50:36]

Description was changed from

==========
Check for invalid dimensions in glTexStorage*

width, height, or depth of 0 are invalid for glTexStorage, but valid for
most other texture entry points, so this case wasn't covered by
TextureManager::ValidForTarget.

BUG=700270
==========

to

==========
Check for invalid dimensions in glTexStorage*

width, height, or depth of 0 are invalid for glTexStorage, but valid for
most other texture entry points, so this case wasn't covered by
TextureManager::ValidForTarget.

BUG=700270
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;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
==========

[Kai Ninomiya, 2017-03-13 21:57:37]

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

[commit-bot: I haz the power, 2017-03-13 21:58:25]

Dry run: 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-03-13 22:54:26]

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

[commit-bot: I haz the power, 2017-03-13 22:54:27]

Dry run: This issue passed the CQ dry run.

[Kai Ninomiya, 2017-03-13 22:58:48]

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

[Kai Ninomiya, 2017-03-13 22:58:48]

kbr, PTAL.

[Ken Russell (switch to Gerrit), 2017-03-14 03:00:18]

Great. Good unit test. LGTM

[Kai Ninomiya, 2017-03-14 03:02:19]

The CQ bit was checked by kainino@chromium.org

[commit-bot: I haz the power, 2017-03-14 03:02:43]

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-03-14 03:28:25]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1489460539312760,
"parent_rev": "cd3dad73b453f8a89731bc2a2cd72f3bd5def991", "commit_rev":
"87b070a02a9413d0b48fd7ebd0d1c271c2f657c5"}

[commit-bot: I haz the power, 2017-03-14 03:29:08]

Description was changed from

==========
Check for invalid dimensions in glTexStorage*

width, height, or depth of 0 are invalid for glTexStorage, but valid for
most other texture entry points, so this case wasn't covered by
TextureManager::ValidForTarget.

BUG=700270
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;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

==========
Check for invalid dimensions in glTexStorage*

width, height, or depth of 0 are invalid for glTexStorage, but valid for
most other texture entry points, so this case wasn't covered by
TextureManager::ValidForTarget.

BUG=700270
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;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/2751553002
Cr-Commit-Position: refs/heads/master@{#456615}
Committed:
https://chromium.googlesource.com/chromium/src/+/87b070a02a9413d0b48fd7ebd0d1...
==========

[commit-bot: I haz the power, 2017-03-14 03:29:12]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/87b070a02a9413d0b48fd7ebd0d1...