From 6be3b97ff55d419be666fc172a3f01b3d0814cd4 Mon Sep 17 00:00:00 2001 From: Corentin Wallez Date: Mon, 11 Feb 2019 23:34:52 +0000 Subject: [PATCH] End2EndTests: Fix -w runs without ASSERTs enabled. The flushing of command buffers was in an ASSERT which made them skipped on release builds. BUG= Change-Id: I07bf558d1689f4bea0047a80ab3c8b04938f04a0 Reviewed-on: https://dawn-review.googlesource.com/c/4501 Commit-Queue: Kai Ninomiya Reviewed-by: Austin Eng Reviewed-by: Kai Ninomiya --- src/tests/DawnTest.cpp | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/tests/DawnTest.cpp b/src/tests/DawnTest.cpp index c643047555..357c0904a4 100644 --- a/src/tests/DawnTest.cpp +++ b/src/tests/DawnTest.cpp @@ -332,8 +332,10 @@ void DawnTest::SwapBuffersForCapture() { void DawnTest::FlushWire() { if (gTestUsesWire) { - ASSERT(mC2sBuf->Flush()); - ASSERT(mS2cBuf->Flush()); + bool C2SFlushed = mC2sBuf->Flush(); + bool S2CFlushed = mS2cBuf->Flush(); + ASSERT(C2SFlushed); + ASSERT(S2CFlushed); } }