HTML MessagePort as mojo::MessagePipeHandle

third_party/WebKit/Source/platform/CrossThreadCopier.h

Index: third_party/WebKit/Source/platform/CrossThreadCopier.h
diff --git a/third_party/WebKit/Source/platform/CrossThreadCopier.h b/third_party/WebKit/Source/platform/CrossThreadCopier.h
index a46f9d606f4a430841ceaa3c084234b076f11c17..5f836d64c4c128f3ff5697304817d586f88a151b 100644
--- a/third_party/WebKit/Source/platform/CrossThreadCopier.h
+++ b/third_party/WebKit/Source/platform/CrossThreadCopier.h
@@ -149,6 +149,16 @@ struct CrossThreadCopier<std::unique_ptr<T, Deleter>> {
   }
 };
 
+template <typename T, size_t inlineCapacity, typename Allocator>
+struct CrossThreadCopier<Vector<T, inlineCapacity, Allocator>> {
+  STATIC_ONLY(CrossThreadCopier);
+  using Type = Vector<T, inlineCapacity, Allocator>;
+  static Vector<T, inlineCapacity, Allocator> copy(
+      Vector<T, inlineCapacity, Allocator> pointer) {
+    return pointer;  // This is in fact a move.
+  }
+};

[kinuko, 2017/02/08 09:45:06]

Hmm, I think having CrossThreadCoppier for generic Vector is dangerous, if T
needs non-trivial copier support we need to recursively call
CrossThreadCopier::copy() on each element.  Also the comment doesn't look right
as this either moves or copies the given vector depending on what the caller
passes, as Vector supports both copy and move semantics.

Could we... maybe just define CrossThreadCopier for Vector<std::unique_ptr<T>>?

[darin (slow to review), 2017/02/08 17:09:18]

Great suggestion. That works perfectly.

+
 template <typename T>
 struct CrossThreadCopier<CrossThreadPersistent<T>>
     : public CrossThreadCopierPassThrough<CrossThreadPersistent<T>> {