Add a ctor for InterfaceRequest<I> that takes a ScopedMessagePipeHandle, and remove MakeRequest<I>(…

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

 // 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.
 
 // TODO(vardhan):  Currently, the logic for serializing a mojom type exists in
 // two places: the C++ code generator template, and here.  However, most types
 // are serialized the same way within Arrays or outside, with the exception of
 // |bool|.  Consider defining serialization/deserialization traits for each
 // serializable type and call those traits from here.  This should help us
 // remove most of the ArraySerializer<> specializations here.
 
 #ifndef MOJO_PUBLIC_CPP_BINDINGS_LIB_ARRAY_SERIALIZATION_H_
 #define MOJO_PUBLIC_CPP_BINDINGS_LIB_ARRAY_SERIALIZATION_H_
 
 #include <string.h>  // For |memcpy()|.
 #include <type_traits>
 #include <vector>
 
 #include "mojo/public/c/system/macros.h"
 #include "mojo/public/cpp/bindings/lib/array_internal.h"
 #include "mojo/public/cpp/bindings/lib/bindings_internal.h"
 #include "mojo/public/cpp/bindings/lib/iterator_util.h"
 #include "mojo/public/cpp/bindings/lib/map_data_internal.h"
 #include "mojo/public/cpp/bindings/lib/map_serialization_forward.h"
 #include "mojo/public/cpp/bindings/lib/string_serialization.h"
 #include "mojo/public/cpp/bindings/lib/validation_errors.h"
 
 namespace mojo {
 
-template <typename Interface>
-InterfaceRequest<Interface> MakeRequest(ScopedMessagePipeHandle handle);
-
 namespace internal {
 
 // The ArraySerializer template contains static methods for serializing |Array|s
 // of various types.  These methods include:
 //   * size_t GetSerializedSize(..)
 //       Computes the size of the serialized version of the |Array|.
 //   * void SerializeElements(..)
 //       Takes an |Iterator| and a size and serializes it.
 //   * void DeserializeElements(..)
 //       Takes a pointer to an |Array_Data| and deserializes it into a given
@@ skipping 185 matching lines @@
     }
 
     return ValidationError::NONE;
   }
 
   static void DeserializeElements(Array_Data<MessagePipeHandle>* input,
                                   Array<InterfaceRequest<I>>* output) {
     auto result = Array<InterfaceRequest<I>>::New(input->size());
     for (size_t i = 0; i < input->size(); ++i)
       result.at(i) =
-          MakeRequest<I>(MakeScopedHandle(FetchAndReset(&input->at(i)))).Pass();
+          InterfaceRequest<I>(MakeScopedHandle(FetchAndReset(&input->at(i))))
+              .Pass();
     output->Swap(&result);
   }
 };
 
 // Serializes and deserializes arrays of interfaces (interface handles).
 template <typename Interface>
 struct ArraySerializer<InterfaceHandle<Interface>, Interface_Data, false> {
   static size_t GetSerializedSize(
       const Array<InterfaceHandle<Interface>>& input) {
     return sizeof(Array_Data<Interface_Data>) +
@@ skipping 283 matching lines @@
   if (input) {
     internal::ArraySerializer<E, F>::DeserializeElements(input, output);
   } else {
     output->reset();
   }
 }
 
 }  // namespace mojo
 
 #endif  // MOJO_PUBLIC_CPP_BINDINGS_LIB_ARRAY_SERIALIZATION_H_