SurfaceTests: Clarify a suppression.
Bug: dawn:2116
Change-Id: Ibfeb283eccc2aaac54d91ebdffe9d91541606366
Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/205036
Auto-Submit: Corentin Wallez <cwallez@chromium.org>
Reviewed-by: Loko Kung <lokokung@google.com>
Commit-Queue: Corentin Wallez <cwallez@chromium.org>
diff --git a/src/dawn/tests/end2end/SurfaceTests.cpp b/src/dawn/tests/end2end/SurfaceTests.cpp
index d636569..6b96b53 100644
--- a/src/dawn/tests/end2end/SurfaceTests.cpp
+++ b/src/dawn/tests/end2end/SurfaceTests.cpp
@@ -396,8 +396,11 @@
// Test switching devices on the same adapter.
TEST_P(SurfaceTests, SwitchingDevice) {
- // TODO(https://crbug.com/dawn/2116): Disabled due to new Validation Layer failures.
- DAWN_SUPPRESS_TEST_IF(IsVulkan());
+ // TODO(https://crbug.com/dawn/2116): VVLs crash because oldSwapchain is from a different
+ // device. The spec says it is ok if it is only the same instance but clarifications yet to be
+ // published by Khronos are that the spec is wrong and should require the swapchain to be from
+ // the same device.
+ DAWN_SUPPRESS_TEST_IF(IsVulkan() && IsBackendValidationEnabled());
wgpu::Device device2 = CreateDevice();