content: Pass IOSurface references using Mach IPC.

content/child/child_io_surface_manager_mac.cc

Index: content/child/child_io_surface_manager_mac.cc
diff --git a/content/child/child_io_surface_manager_mac.cc b/content/child/child_io_surface_manager_mac.cc
new file mode 100644
index 0000000000000000000000000000000000000000..8d9403d239348dc3f78f633aac61ff4a2aab1e22
--- /dev/null
+++ b/content/child/child_io_surface_manager_mac.cc
@@ -0,0 +1,136 @@
+// Copyright 2015 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 "content/child/child_io_surface_manager_mac.h"
+
+#include "base/mac/mach_logging.h"
+#include "content/common/mac/io_surface_manager_messages.h"
+
+namespace content {
+
+// static
+ChildIOSurfaceManager* ChildIOSurfaceManager::GetInstance() {
+  return Singleton<ChildIOSurfaceManager,
+                   LeakySingletonTraits<ChildIOSurfaceManager>>::get();
+}
+
+bool ChildIOSurfaceManager::RegisterIOSurface(int io_surface_id,
+                                              int client_id,
+                                              IOSurfaceRef io_surface) {
+  mach_port_t reply_port;
+  kern_return_t kr = mach_port_allocate(mach_task_self(),
+                                        MACH_PORT_RIGHT_RECEIVE, &reply_port);
+  if (kr != KERN_SUCCESS) {
+    MACH_LOG(ERROR, kr) << "mach_port_allocate";
+    return false;
+  }
+  base::mac::ScopedMachReceiveRight scoped_receive_right(reply_port);
+
+  // Deallocate the right after sending a copy to the parent.
+  base::mac::ScopedMachSendRight scoped_io_surface_right(
+      IOSurfaceCreateMachPort(io_surface));
+
+  union {
+    IOSurfaceManagerHostMsg_RegisterIOSurface request;
+    struct {
+      IOSurfaceManagerMsg_RegisterIOSurfaceReply msg;
+      mach_msg_trailer_t trailer;
+    } reply;
+  } data = {{{0}}};

[Robert Sesek, 2015/05/19 23:26:23]

Here, I'd declare two separate variables for the request and reply.

[reveman, 2015/05/27 05:03:18]

Did not change this as discussed.

+  data.request.header.msgh_bits =
+      MACH_MSGH_BITS(MACH_MSG_TYPE_COPY_SEND, MACH_MSG_TYPE_MAKE_SEND_ONCE) |
+      MACH_MSGH_BITS_COMPLEX;
+  DCHECK(service_port_ != MACH_PORT_NULL);
+  data.request.header.msgh_remote_port = service_port_;
+  data.request.header.msgh_local_port = reply_port;
+  data.request.header.msgh_size = sizeof(data.request);
+  data.request.header.msgh_id = IOSurfaceManagerHostMsg_RegisterIOSurface::ID;
+  data.request.body.msgh_descriptor_count = 1;
+  data.request.io_surface_port.name = scoped_io_surface_right;
+  data.request.io_surface_port.disposition = MACH_MSG_TYPE_COPY_SEND;
+  data.request.io_surface_port.type = MACH_MSG_PORT_DESCRIPTOR;
+  data.request.io_surface_id = io_surface_id;
+  data.request.client_id = client_id;
+  DCHECK(!mailbox_.IsZero());
+  memcpy(data.request.mailbox_name, mailbox_.name, sizeof(mailbox_.name));
+
+  kr = mach_msg(&data.request.header, MACH_SEND_MSG | MACH_RCV_MSG,
+                sizeof(data.request), sizeof(data.reply), reply_port,
+                MACH_MSG_TIMEOUT_NONE, MACH_PORT_NULL);
+  if (kr != KERN_SUCCESS) {
+    MACH_LOG(ERROR, kr) << "mach_msg";
+    return false;
+  }
+
+  return data.reply.msg.result;
+}
+
+void ChildIOSurfaceManager::UnregisterIOSurface(int io_surface_id,
+                                                int client_id) {
+  IOSurfaceManagerHostMsg_UnregisterIOSurface request = {{0}};
+  request.header.msgh_bits = MACH_MSGH_BITS(MACH_MSG_TYPE_COPY_SEND, 0);
+  DCHECK(service_port_ != MACH_PORT_NULL);
+  request.header.msgh_remote_port = service_port_;
+  request.header.msgh_local_port = MACH_PORT_NULL;
+  request.header.msgh_size = sizeof(request);
+  request.header.msgh_id = IOSurfaceManagerHostMsg_UnregisterIOSurface::ID;
+  request.io_surface_id = io_surface_id;
+  request.client_id = client_id;
+  DCHECK(!mailbox_.IsZero());
+  memcpy(request.mailbox_name, mailbox_.name, sizeof(mailbox_.name));
+
+  kern_return_t kr =
+      mach_msg(&request.header, MACH_SEND_MSG, sizeof(request), 0,
+               MACH_PORT_NULL, MACH_MSG_TIMEOUT_NONE, MACH_PORT_NULL);
+  if (kr != KERN_SUCCESS) {
+    MACH_LOG(ERROR, kr) << "mach_msg";
+  }
+}
+
+IOSurfaceRef ChildIOSurfaceManager::AcquireIOSurface(int io_surface_id) {
+  mach_port_t reply_port;
+  kern_return_t kr = mach_port_allocate(mach_task_self(),
+                                        MACH_PORT_RIGHT_RECEIVE, &reply_port);
+  if (kr != KERN_SUCCESS) {
+    MACH_LOG(ERROR, kr) << "mach_port_allocate";
+    return nullptr;
+  }
+  base::mac::ScopedMachReceiveRight scoped_receive_right(reply_port);
+
+  union {
+    IOSurfaceManagerHostMsg_AcquireIOSurface request;
+    struct {
+      IOSurfaceManagerMsg_AcquireIOSurfaceReply msg;
+      mach_msg_trailer_t trailer;
+    } reply;
+  } data = {{{0}}};
+  data.request.header.msgh_bits =
+      MACH_MSGH_BITS(MACH_MSG_TYPE_COPY_SEND, MACH_MSG_TYPE_MAKE_SEND_ONCE);
+  DCHECK(service_port_ != MACH_PORT_NULL);
+  data.request.header.msgh_remote_port = service_port_;
+  data.request.header.msgh_local_port = reply_port;
+  data.request.header.msgh_size = sizeof(data.request);
+  data.request.header.msgh_id = IOSurfaceManagerHostMsg_AcquireIOSurface::ID;
+  data.request.io_surface_id = io_surface_id;
+  DCHECK(!mailbox_.IsZero());
+  memcpy(data.request.mailbox_name, mailbox_.name, sizeof(mailbox_.name));
+
+  kr = mach_msg(&data.request.header, MACH_SEND_MSG | MACH_RCV_MSG,
+                sizeof(data.request), sizeof(data.reply), reply_port,
+                MACH_MSG_TIMEOUT_NONE, MACH_PORT_NULL);
+  if (kr != KERN_SUCCESS) {
+    MACH_LOG(ERROR, kr) << "mach_msg";
+    return nullptr;
+  }
+
+  return IOSurfaceLookupFromMachPort(data.reply.msg.io_surface_port.name);

[Robert Sesek, 2015/05/19 23:26:23]

Does this function consume the port right?

[reveman, 2015/05/27 05:03:18]

Good call. It does not based on the documentation:
https://developer.apple.com/library/mac/documentation/Miscellaneous/Reference...

Latest patch frees the port after creating an IOSurface reference. 

+}
+
+ChildIOSurfaceManager::ChildIOSurfaceManager() : service_port_(MACH_PORT_NULL) {
+}
+
+ChildIOSurfaceManager::~ChildIOSurfaceManager() {
+}
+
+}  // namespace content