Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1239)

Unified Diff: content/child/child_io_surface_manager_mac.cc

Issue 1137453002: content: Pass IOSurface references using Mach IPC. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rsesek's review Created 5 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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..7966c92f2cb30b6af147daedddcc65a9b87fe82a
--- /dev/null
+++ b/content/child/child_io_surface_manager_mac.cc
@@ -0,0 +1,140 @@
+// 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}}};
+ 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_.is_valid());
Robert Sesek 2015/06/01 21:58:06 Move this DCHECK to the top of the function.
reveman 2015/06/02 22:48:07 Done. Moved all DCHECKs in this file to the top of
+ 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(!token_.IsZero());
+ memcpy(data.request.token_name, token_.name, sizeof(token_.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_.is_valid());
+ 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(!token_.IsZero());
+ memcpy(request.token_name, token_.name, sizeof(token_.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_.is_valid());
+ 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(!token_.IsZero());
+ memcpy(data.request.token_name, token_.name, sizeof(token_.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;
+ }
+
+ // Deallocate the right after creating an IOSurface reference.
+ base::mac::ScopedMachSendRight scoped_io_surface_right(
+ data.reply.msg.io_surface_port.name);
+
+ return IOSurfaceLookupFromMachPort(scoped_io_surface_right);
+}
+
+ChildIOSurfaceManager::ChildIOSurfaceManager() {
+}
+
+ChildIOSurfaceManager::~ChildIOSurfaceManager() {
+}
+
+} // namespace content

Powered by Google App Engine
This is Rietveld 408576698