Mojo: Make PlatformChannelRecvmsg() append handles to a deque instead.

mojo/embedder/platform_channel_utils_posix.h

Index: mojo/embedder/platform_channel_utils_posix.h
diff --git a/mojo/embedder/platform_channel_utils_posix.h b/mojo/embedder/platform_channel_utils_posix.h
index 044199f84e96cf939829dd9b0c103be68819169f..3cb36b031ab1c47bdb24eacf34bf9abcb820a92d 100644
--- a/mojo/embedder/platform_channel_utils_posix.h
+++ b/mojo/embedder/platform_channel_utils_posix.h
@@ -8,11 +8,10 @@
 #include <stddef.h>
 #include <sys/types.h>  // For |ssize_t|.
 
-#include <vector>
+#include <deque>
 
 #include "base/memory/scoped_ptr.h"
 #include "mojo/embedder/platform_handle.h"
-#include "mojo/embedder/platform_handle_vector.h"
 #include "mojo/system/system_impl_export.h"
 
 struct iovec;  // Declared in <sys/uio.h>.
@@ -48,14 +47,13 @@ MOJO_SYSTEM_IMPL_EXPORT bool PlatformChannelSendHandles(PlatformHandle h,
                                                         size_t num_handles);
 
 // Wrapper around |recvmsg()|, which will extract any attached file descriptors
-// (in the control message) to |PlatformHandle|s. (This also handles |EINTR|.)
-// If |*handles| is null and handles are received, a vector will be allocated;
-// otherwise, any handles received will be appended to the existing vector.
+// (in the control message) to |PlatformHandle|s (and appended to

[yzshen1, 2014/05/20 21:09:46]

nit: appended -> append them?

[viettrungluu, 2014/05/20 21:17:36]

Done.

+// |platform_handles|). (This also handles |EINTR|.)
 MOJO_SYSTEM_IMPL_EXPORT ssize_t PlatformChannelRecvmsg(
     PlatformHandle h,
     void* buf,
     size_t num_bytes,
-    ScopedPlatformHandleVectorPtr* handles);
+    std::deque<PlatformHandle>* platform_handles);
 
 }  // namespace embedder
 }  // namespace mojo