Mojo: Report bindings validation errors via MojoNotifyBadMessage

mojo/public/cpp/bindings/tests/router_test_util.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "mojo/public/cpp/bindings/tests/router_test_util.h"
 
 #include <stddef.h>
 #include <stdint.h>
 #include <string.h>
 
@@ skipping 22 matching lines @@
 
   builder.message()->MoveTo(message);
 }
 
 MessageAccumulator::MessageAccumulator(MessageQueue* queue,
                                        const base::Closure& closure)
     : queue_(queue), closure_(closure) {}
 
 MessageAccumulator::~MessageAccumulator() {}
 
-bool MessageAccumulator::Accept(Message* message) {
+MessageReceiver::Result MessageAccumulator::Accept(Message* message) {
   queue_->Push(message);
   if (!closure_.is_null()) {
     closure_.Run();
     closure_.Reset();
   }
-  return true;
+  return Result::ForSuccess();
 }
 
 ResponseGenerator::ResponseGenerator() {}
 
-bool ResponseGenerator::Accept(Message* message) {
-  return false;
+MessageReceiver::Result ResponseGenerator::Accept(Message* message) {
+  return Result::ForUnknownError();
 }
 
-bool ResponseGenerator::AcceptWithResponder(
+MessageReceiver::Result ResponseGenerator::AcceptWithResponder(
     Message* message,
     MessageReceiverWithStatus* responder) {
   EXPECT_TRUE(message->has_flag(internal::kMessageExpectsResponse));
 
   bool result = SendResponse(message->name(), message->request_id(),
                              reinterpret_cast<const char*>(message->payload()),
                              responder);
   EXPECT_TRUE(responder->IsValid());
   delete responder;
-  return result;
+  if (result)
+    return Result::ForSuccess();
+  return Result::ForUnknownError();
 }
 
 bool ResponseGenerator::SendResponse(uint32_t name,
                                      uint64_t request_id,
                                      const char* request_string,
                                      MessageReceiver* responder) {
   Message response;
   std::string response_string(request_string);
   response_string += " world!";
   AllocResponseMessage(name, response_string.c_str(), request_id, &response);
 
-  return responder->Accept(&response);
+  return responder->Accept(&response).Succeeded();
+
 }
 
 LazyResponseGenerator::LazyResponseGenerator(const base::Closure& closure)
     : responder_(nullptr), name_(0), request_id_(0), closure_(closure) {}
 
 LazyResponseGenerator::~LazyResponseGenerator() {
   delete responder_;
 }
 
-bool LazyResponseGenerator::AcceptWithResponder(
+MessageReceiver::Result LazyResponseGenerator::AcceptWithResponder(
     Message* message,
     MessageReceiverWithStatus* responder) {
   name_ = message->name();
   request_id_ = message->request_id();
   request_string_ =
       std::string(reinterpret_cast<const char*>(message->payload()));
   responder_ = responder;
   if (!closure_.is_null()) {
     closure_.Run();
     closure_.Reset();
   }
-  return true;
+  return Result::ForSuccess();
 }
 
 void LazyResponseGenerator::Complete(bool send_response) {
   if (send_response) {
     SendResponse(name_, request_id_, request_string_.c_str(), responder_);
   }
   delete responder_;
   responder_ = nullptr;
 }
 
 }  // namespace test
 }  // namespace mojo