gpu: Compositor management of GpuMemoryBuffer instances.

content/common/child_process_host_impl.cc

Index: content/common/child_process_host_impl.cc
diff --git a/content/common/child_process_host_impl.cc b/content/common/child_process_host_impl.cc
index c4d209e08773d5b103922d3809f43556e85210dc..4bf59d2dccdc1132e6998df3375c88dd3fd235df 100644
--- a/content/common/child_process_host_impl.cc
+++ b/content/common/child_process_host_impl.cc
@@ -253,8 +253,6 @@ bool ChildProcessHostImpl::OnMessageReceived(const IPC::Message& msg) {
                           OnShutdownRequest)
       IPC_MESSAGE_HANDLER(ChildProcessHostMsg_SyncAllocateSharedMemory,
                           OnAllocateSharedMemory)
-      IPC_MESSAGE_HANDLER(ChildProcessHostMsg_SyncAllocateGpuMemoryBuffer,
-                          OnAllocateGpuMemoryBuffer)

[piman, 2014/10/09 21:33:02]

For consistency, if ChildProcessHostMsg_SyncAllocateGpuMemoryBuffer is only
available for renderers, should the ChildGpuMemoryBufferManager only be
available on RenderThreadImpl rather than ChildThread?

[reveman, 2014/10/10 02:20:16]

I added a working implementation for this to latest patch instead. It's limited
to shared memory backed GpuMemoryBuffers for now but that's better than nothing.
I also added a TODO as we should be adding support for other types asap.

       IPC_MESSAGE_UNHANDLED(handled = false)
     IPC_END_MESSAGE_MAP()
 
@@ -307,17 +305,4 @@ void ChildProcessHostImpl::OnShutdownRequest() {
     Send(new ChildProcessMsg_Shutdown());
 }
 
-void ChildProcessHostImpl::OnAllocateGpuMemoryBuffer(
-    uint32 width,
-    uint32 height,
-    uint32 internalformat,
-    uint32 usage,
-    gfx::GpuMemoryBufferHandle* handle) {
-  handle->type = gfx::SHARED_MEMORY_BUFFER;
-  AllocateSharedMemory(
-      width * height * GpuMemoryBufferImpl::BytesPerPixel(internalformat),
-      peer_handle_,
-      &handle->handle);
-}
-
 }  // namespace content