Mojo: Report bindings validation errors via MojoNotifyBadMessage

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

Index: mojo/public/cpp/bindings/lib/pipe_control_message_proxy.cc
diff --git a/mojo/public/cpp/bindings/lib/pipe_control_message_proxy.cc b/mojo/public/cpp/bindings/lib/pipe_control_message_proxy.cc
index a7f340fb701d4deeba0dcd4344be7899409012cf..acbf5a2ee1b6d98e6e26f19856014675d836f606 100644
--- a/mojo/public/cpp/bindings/lib/pipe_control_message_proxy.cc
+++ b/mojo/public/cpp/bindings/lib/pipe_control_message_proxy.cc
@@ -9,6 +9,7 @@
 
 #include "base/compiler_specific.h"
 #include "base/logging.h"
+#include "base/macros.h"
 #include "mojo/public/cpp/bindings/lib/message_builder.h"
 #include "mojo/public/cpp/bindings/lib/serialization.h"
 #include "mojo/public/cpp/bindings/message.h"
@@ -35,10 +36,9 @@ void SendRunOrClosePipeMessage(MessageReceiver* receiver,
       params_ptr, builder.buffer(), &params, context);
   params->EncodePointers();
   builder.message()->set_interface_id(kInvalidInterfaceId);
-  bool ok = receiver->Accept(builder.message());
-  // This return value may be ignored as !ok implies the underlying message pipe
-  // has encountered an error, which will be visible through other means.
-  ALLOW_UNUSED_LOCAL(ok);
+  // This return value may be ignored as failure implies the underlying message
+  // pipe has encountered an error, which will be visible through other means.
+  ignore_result(receiver->Accept(builder.message()));
 }
 
 }  // namespace