Mojo: Report bindings validation errors via MojoNotifyBadMessage

mojo/public/cpp/bindings/lib/control_message_handler.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/lib/control_message_handler.h"
 
 #include <stddef.h>
 #include <stdint.h>
 #include <utility>
 
@@ skipping 11 matching lines @@
          message->header()->name == kRunOrClosePipeMessageId;
 }
 
 ControlMessageHandler::ControlMessageHandler(uint32_t interface_version)
     : interface_version_(interface_version) {
 }
 
 ControlMessageHandler::~ControlMessageHandler() {
 }
 
-bool ControlMessageHandler::Accept(Message* message) {
+bool ControlMessageHandler::Accept(Message* message, Error* error) {
   if (message->header()->name == kRunOrClosePipeMessageId)
     return RunOrClosePipe(message);
 
   NOTREACHED();
   return false;
 }
 
 bool ControlMessageHandler::AcceptWithResponder(
     Message* message,
-    MessageReceiverWithStatus* responder) {
+    MessageReceiverWithStatus* responder,
+    Error* error) {
   if (message->header()->name == kRunMessageId)
     return Run(message, responder);
 
   NOTREACHED();
   return false;
 }
 
 bool ControlMessageHandler::Run(Message* message,
                                 MessageReceiverWithStatus* responder) {
   RunResponseMessageParamsPtr response_params_ptr(
       RunResponseMessageParams::New());
   response_params_ptr->reserved0 = 16u;
   response_params_ptr->reserved1 = 0u;
   response_params_ptr->query_version_result = QueryVersionResult::New();
   response_params_ptr->query_version_result->version = interface_version_;
 
   size_t size = PrepareToSerialize<RunResponseMessageParamsPtr>(
       response_params_ptr, &context_);
   ResponseMessageBuilder builder(kRunMessageId, size, message->request_id());
 
   RunResponseMessageParams_Data* response_params = nullptr;
   Serialize<RunResponseMessageParamsPtr>(response_params_ptr, builder.buffer(),
                                          &response_params, &context_);
   response_params->EncodePointers();
-  bool ok = responder->Accept(builder.message());
+
+  Error send_error;
+  bool ok = responder->Accept(builder.message(), &send_error);
   ALLOW_UNUSED_LOCAL(ok);
   delete responder;
 
   return true;
 }
 
 bool ControlMessageHandler::RunOrClosePipe(Message* message) {
   RunOrClosePipeMessageParams_Data* params =
       reinterpret_cast<RunOrClosePipeMessageParams_Data*>(
           message->mutable_payload());
   params->DecodePointers();
 
   RunOrClosePipeMessageParamsPtr params_ptr;
   Deserialize<RunOrClosePipeMessageParamsPtr>(params, &params_ptr, &context_);
 
   return interface_version_ >= params_ptr->require_version->version;
 }
 
 }  // namespace internal
 }  // namespace mojo