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

mojo/public/cpp/bindings/tests/serialization_warning_unittest.cc

Index: mojo/public/cpp/bindings/tests/serialization_warning_unittest.cc
diff --git a/mojo/public/cpp/bindings/tests/serialization_warning_unittest.cc b/mojo/public/cpp/bindings/tests/serialization_warning_unittest.cc
index b61d3bc57e41482d50bbcbef8eb50b4683d3f132..85b0924a7befda9b6440b30ad50fbd4cd1e83e2a 100644
--- a/mojo/public/cpp/bindings/tests/serialization_warning_unittest.cc
+++ b/mojo/public/cpp/bindings/tests/serialization_warning_unittest.cc
@@ -57,29 +57,21 @@ class SerializationWarningTest : public testing::Test {
   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_;
 };