Mojo: Don't short-circuit when failing to deserialize arrays of native types.

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

 // Copyright 2016 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.
 
 #ifndef MOJO_PUBLIC_CPP_BINDINGS_LIB_ARRAY_SERIALIZATION_TRAITS_H_
 #define MOJO_PUBLIC_CPP_BINDINGS_LIB_ARRAY_SERIALIZATION_TRAITS_H_
 
 #include <stddef.h>
 #include <string.h>  // For |memcpy()|.
 
@@ skipping 380 matching lines @@
     static_assert(
         std::is_same<F, Array_Data<uint8_t>*>::value,
         "Native-only type array must deserialize from array of byte arrays.");
     DCHECK(input);
 
     ArrayType result(input->size());
     bool success = true;
     for (size_t i = 0; i < input->size(); ++i) {
       // We don't short-circuit on failure since we can't know what the native
       // type's ParamTraits' expectations are.
-      success =
-          success && DeserializeNative_(input->at(i), &result[i], context);
+      if (!DeserializeNative_(input->at(i), &result[i], context))
+        success = false;
     }
     output->Swap(&result);
     return success;
   }
 };
 
 template <typename ArrayType>
 struct ArraySerializationImpl {
   using Strategy = ArraySerializationStrategy<ArrayType>;
 
@@ skipping 34 matching lines @@
     *output = nullptr;
     return true;
   }
 };
 
 }  // namespace internal
 
 }  // namespace mojo
 
 #endif  // MOJO_PUBLIC_CPP_BINDINGS_LIB_ARRAY_SERIALIZATION_TRAITS_H_