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

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

Index: mojo/public/cpp/bindings/tests/map_unittest.cc
diff --git a/mojo/public/cpp/bindings/tests/map_unittest.cc b/mojo/public/cpp/bindings/tests/map_unittest.cc
index 9a1525326ada98b2fb5112bead2df90012ea7af9..499a67b68df0363c4cf1e2f37d1a344c1359835b 100644
--- a/mojo/public/cpp/bindings/tests/map_unittest.cc
+++ b/mojo/public/cpp/bindings/tests/map_unittest.cc
@@ -246,7 +246,8 @@ TEST_F(MapTest, ArrayOfMap) {
     Array_Data<Map_Data<int32_t, int8_t>*>* data;
     ArrayValidateParams validate_params(
         0, false, new ArrayValidateParams(0, false, nullptr));
-    SerializeArray_(&array, &buf, &data, &validate_params);
+    EXPECT_EQ(internal::VALIDATION_ERROR_NONE,

[viettrungluu, 2015/10/09 01:35:20]

(Like you do here!)

[vardhan, 2015/10/09 22:31:29]

Because there's no conflict with test::internal here :-)

+              SerializeArray_(&array, &buf, &data, &validate_params));
 
     Array<Map<int32_t, int8_t>> deserialized_array;
     Deserialize_(data, &deserialized_array);
@@ -269,7 +270,8 @@ TEST_F(MapTest, ArrayOfMap) {
     ArrayValidateParams validate_params(
         0, false, new ArrayValidateParams(
                       0, false, new ArrayValidateParams(0, false, nullptr)));
-    SerializeArray_(&array, &buf, &data, &validate_params);
+    EXPECT_EQ(internal::VALIDATION_ERROR_NONE,
+              SerializeArray_(&array, &buf, &data, &validate_params));
 
     Array<Map<String, Array<bool>>> deserialized_array;
     Deserialize_(data, &deserialized_array);