Mojo: Report bindings validation errors via MojoNotifyBadMessage

mojo/public/cpp/bindings/tests/validation_unittest.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 <stddef.h>
 #include <stdint.h>
 #include <stdio.h>
 #include <algorithm>
 #include <string>
 #include <utility>
@@ skipping 158 matching lines @@
 
   for (size_t i = 0; i < tests.size(); ++i) {
     Message message;
     std::string expected;
     ASSERT_TRUE(ReadTestCase(tests[i], &message, &expected));
 
     std::string result;
     base::RunLoop run_loop;
     mojo::internal::ValidationErrorObserverForTesting observer(
         run_loop.QuitClosure());
-    ignore_result(test_message_receiver->Accept(&message));
+    Error error;
+    ignore_result(test_message_receiver->Accept(&message, &error));
     if (expected != "PASS")  // Observer only gets called on errors.
       run_loop.Run();
     if (observer.last_error() == mojo::internal::VALIDATION_ERROR_NONE)
       result = "PASS";
     else
       result = mojo::internal::ValidationErrorToString(observer.last_error());
 
     EXPECT_EQ(expected, result) << "failed test: " << tests[i];
   }
 }
 
 class DummyMessageReceiver : public MessageReceiver {
  public:
-  bool Accept(Message* message) override {
+  bool Accept(Message* message, Error* error) override {
     return true;  // Any message is OK.
   }
 };
 
 class ValidationTest : public testing::Test {
  public:
   ValidationTest() {}
 
  protected:
   base::MessageLoop loop_;
@@ skipping 34 matching lines @@
     TestMessageReceiver(ValidationIntegrationTest* owner,
                         ScopedMessagePipeHandle handle)
         : owner_(owner),
           connector_(std::move(handle),
                      mojo::internal::Connector::SINGLE_THREADED_SEND,
                      base::ThreadTaskRunnerHandle::Get()) {
       connector_.set_enforce_errors_from_incoming_receiver(false);
     }
     ~TestMessageReceiver() override {}
 
-    bool Accept(Message* message) override {
-      return connector_.Accept(message);
+    bool Accept(Message* message, Error* error) override {
+      return connector_.Accept(message, error);
     }
 
    public:
     ValidationIntegrationTest* owner_;
     mojo::internal::Connector connector_;
   };
 
   void PumpMessages() { loop_.RunUntilIdle(); }
 
   TestMessageReceiver* test_message_receiver_;
@@ skipping 230 matching lines @@
   EXPECT_TRUE(IsKnownEnumValue(static_cast<StructWithEnum::EnumWithin>(0)));
   EXPECT_TRUE(IsKnownEnumValue(static_cast<StructWithEnum::EnumWithin>(1)));
   EXPECT_TRUE(IsKnownEnumValue(static_cast<StructWithEnum::EnumWithin>(2)));
   EXPECT_TRUE(IsKnownEnumValue(static_cast<StructWithEnum::EnumWithin>(3)));
   EXPECT_FALSE(IsKnownEnumValue(static_cast<StructWithEnum::EnumWithin>(4)));
 }
 
 }  // namespace
 }  // namespace test
 }  // namespace mojo