Mojo: Report bindings validation errors via MojoNotifyBadMessage

mojo/public/cpp/bindings/lib/message_header_validator.cc

 // Copyright 2014 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/message_header_validator.h"
 
 #include "mojo/public/cpp/bindings/lib/bounds_checker.h"
 #include "mojo/public/cpp/bindings/lib/validation_errors.h"
 #include "mojo/public/cpp/bindings/lib/validation_util.h"
 
@@ skipping 38 matching lines @@
     ReportValidationError(VALIDATION_ERROR_MESSAGE_HEADER_INVALID_FLAGS);
     return false;
   }
 
   return true;
 }
 
 }  // namespace
 
 MessageHeaderValidator::MessageHeaderValidator(MessageReceiver* sink)
-    : MessageFilter(sink) {
-}
+    : MessageHeaderValidator(sink, "Unknown header validator") {}
 
-bool MessageHeaderValidator::Accept(Message* message) {
+MessageHeaderValidator::MessageHeaderValidator(MessageReceiver* sink,
+                                               const std::string& debug_info)
+    : MessageFilter(sink), debug_info_(debug_info) {}
+
+bool MessageHeaderValidator::Accept(Message* message, Error* error) {
   // Pass 0 as number of handles because we don't expect any in the header, even
   // if |message| contains handles.
   BoundsChecker bounds_checker(message->data(), message->data_num_bytes(), 0);
 
-  if (!ValidateStructHeaderAndClaimMemory(message->data(), &bounds_checker))
+  if (!ValidateStructHeaderAndClaimMemory(message->data(), &bounds_checker)) {
+    *error = Error::ForBadMessage(debug_info_, message);
     return false;
+  }
 
-  if (!IsValidMessageHeader(message->header()))
+  if (!IsValidMessageHeader(message->header())) {
+    *error = Error::ForBadMessage(debug_info_, message);
     return false;
+  }
 
-  return sink_->Accept(message);
+  return sink_->Accept(message, error);
 }
 
 }  // namespace internal
 }  // namespace mojo