Mojo cpp bindings: remove redundant validation in Decode*().

mojo/public/cpp/bindings/lib/array_internal.cc

 // Copyright 2013 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.
 
 #include "mojo/public/cpp/bindings/lib/array_internal.h"
 
 namespace mojo {
 namespace internal {
 
 ArrayDataTraits<bool>::BitRef::~BitRef() {
@@ skipping 26 matching lines @@
 // static
 void ArraySerializationHelper<Handle, true>::EncodePointersAndHandles(
     const ArrayHeader* header,
     ElementType* elements,
     std::vector<Handle>* handles) {
   for (uint32_t i = 0; i < header->num_elements; ++i)
     EncodeHandle(&elements[i], handles);
 }
 
 // static
-bool ArraySerializationHelper<Handle, true>::DecodePointersAndHandles(
+void ArraySerializationHelper<Handle, true>::DecodePointersAndHandles(
     const ArrayHeader* header,
     ElementType* elements,
-    Message* message) {
-  for (uint32_t i = 0; i < header->num_elements; ++i) {
-    if (!DecodeHandle(&elements[i], message->mutable_handles()))
-      return false;
-  }
-  return true;
+    std::vector<Handle>* handles) {
+  for (uint32_t i = 0; i < header->num_elements; ++i)
+    DecodeHandle(&elements[i], handles);
 }
 
 // static
 bool ArraySerializationHelper<Handle, true>::ValidateElements(
     const ArrayHeader* header,
     const ElementType* elements,
     BoundsChecker* bounds_checker) {
   for (uint32_t i = 0; i < header->num_elements; ++i) {
     if (!bounds_checker->ClaimHandle(elements[i]))
       return false;
   }
   return true;
 }
 
 }  // namespace internal
 }  // namespace mojo