From bfe9f91975ff6e41d75c29bb6195c6c3dd212da7 Mon Sep 17 00:00:00 2001
From: Loko Kung <lokokung@google.com>
Date: Mon, 9 May 2022 23:10:54 +0000
Subject: [PATCH] Remove NiceMock on device lost callback.

- Unexpected device lost should result in an error.

Fixes: dawn:1412
Change-Id: I6fe0b1ccc3e4b51cc85b9a058f8eac65c7622d89
Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/89500
Auto-Submit: Loko Kung <lokokung@google.com>
Reviewed-by: Austin Eng <enga@chromium.org>
Commit-Queue: Austin Eng <enga@chromium.org>
---
 src/dawn/tests/DawnTest.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/dawn/tests/DawnTest.h b/src/dawn/tests/DawnTest.h
index 8bc57a2d34..4ce69a3143 100644
--- a/src/dawn/tests/DawnTest.h
+++ b/src/dawn/tests/DawnTest.h
@@ -354,7 +354,7 @@ class DawnTestBase {
     // Mock callbacks tracking errors and destruction. Device lost is a nice mock since tests that
     // do not care about device destruction can ignore the callback entirely.
     testing::MockCallback<WGPUErrorCallback> mDeviceErrorCallback;
-    testing::NiceMock<testing::MockCallback<WGPUDeviceLostCallback>> mDeviceLostCallback;
+    testing::MockCallback<WGPUDeviceLostCallback> mDeviceLostCallback;
 
     // Helper methods to implement the EXPECT_ macros
     std::ostringstream& AddBufferExpectation(const char* file,