From 0b43a803bfb9fe4c426c7dbd7b618f61841dc47c Mon Sep 17 00:00:00 2001
From: Yunchao He <yunchao.he@intel.com>
Date: Fri, 10 Jul 2020 23:30:56 +0000
Subject: [PATCH] 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>
---
 src/tests/unittests/validation/BindGroupValidationTests.cpp | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/src/tests/unittests/validation/BindGroupValidationTests.cpp b/src/tests/unittests/validation/BindGroupValidationTests.cpp
index 9b3aeb5567..7187be3148 100644
--- a/src/tests/unittests/validation/BindGroupValidationTests.cpp
+++ b/src/tests/unittests/validation/BindGroupValidationTests.cpp
@@ -109,9 +109,6 @@ TEST_F(BindGroupValidationTest, WrongBindings) {
 
     // 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