| commit | 0b43a803bfb9fe4c426c7dbd7b618f61841dc47c | [log] [tgz] |
|---|---|---|
| author | Yunchao He <yunchao.he@intel.com> | Fri Jul 10 23:30:56 2020 +0000 |
| committer | Commit Bot service account <commit-bot@chromium.org> | Fri Jul 10 23:30:56 2020 +0000 |
| tree | 1c05e0e1c18d169873c8c13092e811db0c761109 | |
| parent | 7695afc902eba46927cb2d4d1ef97476c8413af1 [diff] |
Remove a unnecessary validation test in bind group binding index >= kMaxBindingsPerGroup is actually valid. Bug: none Change-Id: Idfba743d50013b7115f125b6441fd49732dffc96 Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/24700 Commit-Queue: Kai Ninomiya <kainino@chromium.org> Reviewed-by: Austin Eng <enga@chromium.org> Reviewed-by: Kai Ninomiya <kainino@chromium.org>
diff --git a/src/tests/unittests/validation/BindGroupValidationTests.cpp b/src/tests/unittests/validation/BindGroupValidationTests.cpp index 9b3aeb5..7187be3 100644 --- a/src/tests/unittests/validation/BindGroupValidationTests.cpp +++ b/src/tests/unittests/validation/BindGroupValidationTests.cpp
@@ -109,9 +109,6 @@ // Check that binding must be present in the layout ASSERT_DEVICE_ERROR(utils::MakeBindGroup(device, layout, {{1, mSampler}})); - - // Check that binding >= kMaxBindingsPerGroup fails. - ASSERT_DEVICE_ERROR(utils::MakeBindGroup(device, layout, {{kMaxBindingsPerGroup, mSampler}})); } // Check that the same binding cannot be set twice