mojo::Serialize*_() calls now propogate/return validation errors.

mojo/public/cpp/bindings/tests/serialization_warning_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.
 
 // Serialization warnings are only recorded in debug build.
 #ifndef NDEBUG
 
 #include "mojo/public/cpp/bindings/array.h"
 #include "mojo/public/cpp/bindings/lib/array_internal.h"
 #include "mojo/public/cpp/bindings/lib/array_serialization.h"
@@ skipping 39 matching lines @@
 
   template <typename T>
   void TestWarning(InlinedStructPtr<T> obj,
                    mojo::internal::ValidationError expected_warning) {
     TestStructWarningImpl<T>(obj.Pass(), expected_warning);
   }
 
   template <typename T, typename TPtr>
   void TestStructWarningImpl(TPtr obj,
                              mojo::internal::ValidationError expected_warning) {
-    warning_observer_.set_last_warning(mojo::internal::VALIDATION_ERROR_NONE);
-
     mojo::internal::FixedBufferForTesting buf(GetSerializedSize_(*obj));
     typename T::Data_* data;
-    Serialize_(obj.get(), &buf, &data);
-
-    EXPECT_EQ(expected_warning, warning_observer_.last_warning());
+    EXPECT_EQ(expected_warning, Serialize_(obj.get(), &buf, &data));
   }
 
   template <typename T>
   void TestArrayWarning(T obj,
                         mojo::internal::ValidationError expected_warning,
                         const ArrayValidateParams* validate_params) {
-    warning_observer_.set_last_warning(mojo::internal::VALIDATION_ERROR_NONE);
-
     mojo::internal::FixedBufferForTesting buf(GetSerializedSize_(obj));
     typename T::Data_* data;
-    SerializeArray_(&obj, &buf, &data, validate_params);
-
-    EXPECT_EQ(expected_warning, warning_observer_.last_warning());
+    EXPECT_EQ(expected_warning,
+              SerializeArray_(&obj, &buf, &data, validate_params));
   }
 
-  mojo::internal::SerializationWarningObserverForTesting warning_observer_;
   Environment env_;
 };
 
 TEST_F(SerializationWarningTest, HandleInStruct) {
   Struct2Ptr test_struct(Struct2::New());
   EXPECT_FALSE(test_struct->hdl.is_valid());
 
   TestWarning(test_struct.Pass(),
               mojo::internal::VALIDATION_ERROR_UNEXPECTED_INVALID_HANDLE);
 
@@ skipping 127 matching lines @@
   TestArrayWarning(test_array.Pass(),
                    mojo::internal::VALIDATION_ERROR_UNEXPECTED_ARRAY_HEADER,
                    &validate_params_2);
 }
 
 }  // namespace
 }  // namespace test
 }  // namespace mojo
 
 #endif