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

Unified Diff: content/browser/browser_io_surface_manager_mac.h

Issue 1137453002: content: Pass IOSurface references using Mach IPC. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase, remove extra DCHECKs and fix reply port typo 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/browser/browser_io_surface_manager_mac.h
diff --git a/content/browser/browser_io_surface_manager_mac.h b/content/browser/browser_io_surface_manager_mac.h
new file mode 100644
index 0000000000000000000000000000000000000000..cccad55efee856529dcd77a4a99b79f76caafd21
--- /dev/null
+++ b/content/browser/browser_io_surface_manager_mac.h
@@ -0,0 +1,114 @@
+// 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.
+
+#ifndef CONTENT_BROWSER_BROWSER_IO_SURFACE_MANAGER_MAC_H_
+#define CONTENT_BROWSER_BROWSER_IO_SURFACE_MANAGER_MAC_H_
+
+#include <mach/mach.h>
+
+#include <map>
+#include <utility>
+
+#include "base/containers/scoped_ptr_hash_map.h"
+#include "base/mac/dispatch_source_mach.h"
+#include "base/mac/scoped_mach_port.h"
+#include "base/macros.h"
+#include "base/memory/scoped_ptr.h"
+#include "base/memory/singleton.h"
+#include "base/synchronization/lock.h"
+#include "content/common/mac/io_surface_manager.h"
+#include "content/common/mac/io_surface_manager_messages.h"
+#include "gpu/command_buffer/common/mailbox.h"
+
+namespace content {
+
+// Implementation of IOSurfaceManager that provides a mechanism for child
+// processes to register and acquire IOSurfaces through a Mach service.
+class CONTENT_EXPORT BrowserIOSurfaceManager : public IOSurfaceManager {
+ public:
+ // Returns the global BrowserIOSurfaceManager.
+ static BrowserIOSurfaceManager* GetInstance();
+
+ // Look up the IOSurfaceManager service port that's been registered with
+ // the bootstrap server. |pid| is the process ID of the service.
+ static mach_port_t LookupServicePort(pid_t pid);
Robert Sesek 2015/05/19 23:26:23 This should return in a scoper to ensure proper li
+
+ // Overridden from IOSurfaceManager:
+ bool RegisterIOSurface(int io_surface_id,
+ int client_id,
+ IOSurfaceRef io_surface) override;
+ void UnregisterIOSurface(int io_surface_id, int client_id) override;
+ IOSurfaceRef AcquireIOSurface(int io_surface_id) override;
+
+ // Performs any necessary setup that cannot happen in the constructor.
+ void EnsureRunning();
+
+ // Get the unique unguessable mailbox name that the GPU process can
+ // use to register/unregister IOSurface for use by clients.
+ gpu::Mailbox GetGpuProcessMailbox() const;
+
+ // Generate a unique unguessable mailbox name that the child process
+ // associated |child_process_id| can use to acquire IOSurface references.
+ gpu::Mailbox GenerateChildProcessMailbox(int child_process_id);
+
+ // Invalidate a previously generated mailbox name.
+ void InvalidateChildProcessMailbox(const gpu::Mailbox& mailbox);
+
+ private:
+ friend class BrowserIOSurfaceManagerTest;
+ friend struct DefaultSingletonTraits<BrowserIOSurfaceManager>;
+
+ BrowserIOSurfaceManager();
+ ~BrowserIOSurfaceManager() override;
+
+ // Performs any initialization work.
+ bool Initialize();
+
+ // Message handler that is invoked on |dispatch_source_| when an
+ // incoming message needs to be received.
+ void HandleRequest();
+
+ // Message handlers that are invoked from HandleRequest.
+ bool HandleRegisterIOSurfaceRequest(
+ const IOSurfaceManagerHostMsg_RegisterIOSurface& request,
+ IOSurfaceManagerMsg_RegisterIOSurfaceReply* reply);
+ bool HandleUnregisterIOSurfaceRequest(
+ const IOSurfaceManagerHostMsg_UnregisterIOSurface& request);
+ bool HandleAcquireIOSurfaceRequest(
+ const IOSurfaceManagerHostMsg_AcquireIOSurface& request,
+ IOSurfaceManagerMsg_AcquireIOSurfaceReply* reply);
+
+ // Whether or not the class has been initialized.
+ bool initialized_;
+
+ // The Mach port on which the server listens.
+ base::mac::ScopedMachReceiveRight server_port_;
+
+ // The dispatch source and queue on which Mach messages will be received.
+ scoped_ptr<base::DispatchSourceMach> dispatch_source_;
+
+ // Stores the IOSurfces for all GPU clients.
+ using IOSurfaceMapKey = std::pair<int, int>;
+ using IOSurfaceMap =
+ base::ScopedPtrHashMap<IOSurfaceMapKey,
+ scoped_ptr<base::mac::ScopedMachSendRight>>;
+ IOSurfaceMap io_surfaces_;
+
+ // Stores the Child process unique id (RenderProcessHost ID) for every
+ // mailbox.
+ using ChildProcessIdMap = std::map<gpu::Mailbox, int>;
+ ChildProcessIdMap child_process_ids_;
+
+ // Stores the GPU process mailbox.
+ const gpu::Mailbox gpu_process_mailbox_;
+
+ // Mutex that guards |io_surfaces_| and |child_process_ids_|.
+ mutable base::Lock lock_;
+
+ DISALLOW_COPY_AND_ASSIGN(BrowserIOSurfaceManager);
+};
+
+} // namespace content
+
+#endif // CONTENT_BROWSER_BROWSER_IO_SURFACE_MANAGER_MAC_H_

Powered by Google App Engine
This is Rietveld 408576698