From fc6b1e3912ec63772108d41d00f563299f6856ac Mon Sep 17 00:00:00 2001 From: Austin Eng Date: Wed, 18 Dec 2019 00:35:36 +0000 Subject: [PATCH] fuzzing: WireServer -> CommandHandler in WireServerTraceLayer This is a cleanup patch which changes the WireServer interface to a CommandHandler interface in the DawnTest tracing layer. It also fixes the opening mode flags on the output stream which was missing an output usage. Bug: dawn:295 Change-Id: I5b154e5201f8215b4dafc6389ebf1ec977a09c05 Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/14620 Reviewed-by: Corentin Wallez Reviewed-by: Kai Ninomiya Commit-Queue: Austin Eng --- src/tests/DawnTest.cpp | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/src/tests/DawnTest.cpp b/src/tests/DawnTest.cpp index 17d839f5e1..86235baee1 100644 --- a/src/tests/DawnTest.cpp +++ b/src/tests/DawnTest.cpp @@ -310,18 +310,18 @@ void DawnTestEnvironment::DiscoverOpenGLAdapter() { class WireServerTraceLayer : public dawn_wire::CommandHandler { public: - WireServerTraceLayer(const char* file, dawn_wire::WireServer* server) - : dawn_wire::CommandHandler(), mServer(server) { - mFile.open(file, std::ios_base::app | std::ios_base::binary | std::ios_base::trunc); + WireServerTraceLayer(const char* file, dawn_wire::CommandHandler* handler) + : dawn_wire::CommandHandler(), mHandler(handler) { + mFile.open(file, std::ios_base::out | std::ios_base::binary | std::ios_base::trunc); } const volatile char* HandleCommands(const volatile char* commands, size_t size) override { mFile.write(const_cast(commands), size); - return mServer->HandleCommands(commands, size); + return mHandler->HandleCommands(commands, size); } private: - dawn_wire::WireServer* mServer; + dawn_wire::CommandHandler* mHandler; std::ofstream mFile; };