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>
This commit is contained in:
Austin Eng 2019-12-18 00:35:36 +00:00 committed by Commit Bot service account
parent 5e98e38744
commit fc6b1e3912

View File

@ -310,18 +310,18 @@ void DawnTestEnvironment::DiscoverOpenGLAdapter() {
class WireServerTraceLayer : public dawn_wire::CommandHandler { class WireServerTraceLayer : public dawn_wire::CommandHandler {
public: public:
WireServerTraceLayer(const char* file, dawn_wire::WireServer* server) WireServerTraceLayer(const char* file, dawn_wire::CommandHandler* handler)
: dawn_wire::CommandHandler(), mServer(server) { : dawn_wire::CommandHandler(), mHandler(handler) {
mFile.open(file, std::ios_base::app | std::ios_base::binary | std::ios_base::trunc); 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 { const volatile char* HandleCommands(const volatile char* commands, size_t size) override {
mFile.write(const_cast<const char*>(commands), size); mFile.write(const_cast<const char*>(commands), size);
return mServer->HandleCommands(commands, size); return mHandler->HandleCommands(commands, size);
} }
private: private:
dawn_wire::WireServer* mServer; dawn_wire::CommandHandler* mHandler;
std::ofstream mFile; std::ofstream mFile;
}; };