Move generated Mojo C++ error reporting code out of line

mojo/public/cpp/bindings/lib/validation_errors.h

Index: mojo/public/cpp/bindings/lib/validation_errors.h
diff --git a/mojo/public/cpp/bindings/lib/validation_errors.h b/mojo/public/cpp/bindings/lib/validation_errors.h
index ec0aa2798c28c026ff611e6e37de3886133bb353..cfc3a79e8fa1d1059c77aace575ca5ba31f05f2d 100644
--- a/mojo/public/cpp/bindings/lib/validation_errors.h
+++ b/mojo/public/cpp/bindings/lib/validation_errors.h
@@ -11,6 +11,9 @@
 #include "mojo/public/cpp/bindings/lib/validation_context.h"
 
 namespace mojo {
+
+class Message;
+
 namespace internal {
 
 enum ValidationError {
@@ -75,6 +78,11 @@ void ReportValidationError(ValidationContext* context,
                            ValidationError error,
                            const char* description = nullptr);
 
+void ReportValidationErrorForMessage(
+    mojo::Message* message,
+    ValidationError error,
+    const base::StringPiece& description = "");

[yzshen1, 2016/07/31 07:16:32]

nit: does it make sense to use the same type for |description| as
ReportValidationError()?

[tibell, 2016/07/31 23:50:29]

Done.

+
 // Only used by validation tests and when there is only one thread doing message
 // validation.
 class ValidationErrorObserverForTesting {