commit fc6b1e3912ec63772108d41d00f563299f6856ac
author Austin Eng <enga@chromium.org> Wed Dec 18 00:35:36 2019 +0000
committer Commit Bot service account <commit-bot@chromium.org> Wed Dec 18 00:35:36 2019 +0000
tree e3a0209d9a6c8b8fc7409d858aa2dc115a6a5990
parent 5e98e387444aea9600430165c83749e567965d55

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 <cwallez@chromium.org>
Reviewed-by: Kai Ninomiya <kainino@chromium.org>
Commit-Queue: Austin Eng <enga@chromium.org>

src/tests/DawnTest.cpp

diff --git a/src/tests/DawnTest.cpp b/src/tests/DawnTest.cpp
index 17d839f..86235ba 100644
--- a/src/tests/DawnTest.cpp
+++ b/src/tests/DawnTest.cpp
@@ -310,18 +310,18 @@
 
 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<const char*>(commands), size);
-        return mServer->HandleCommands(commands, size);
+        return mHandler->HandleCommands(commands, size);
     }
 
   private:
-    dawn_wire::WireServer* mServer;
+    dawn_wire::CommandHandler* mHandler;
     std::ofstream mFile;
 };