gpu: Compositor management of GpuMemoryBuffer instances.

content/common/child_process_host_impl.cc

 // Copyright (c) 2012 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/common/child_process_host_impl.h"
 
 #include <limits>
 
 #include "base/atomic_sequence_num.h"
 #include "base/command_line.h"
@@ skipping 235 matching lines @@
     }
   }
 
   if (!handled) {
     handled = true;
     IPC_BEGIN_MESSAGE_MAP(ChildProcessHostImpl, msg)
       IPC_MESSAGE_HANDLER(ChildProcessHostMsg_ShutdownRequest,
                           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()
 
     if (!handled)
       handled = delegate_->OnMessageReceived(msg);
   }
 
 #ifdef IPC_MESSAGE_LOG_ENABLED
   if (logger->Enabled())
     logger->OnPostDispatchMessage(msg, channel_id_);
@@ skipping 32 matching lines @@
     uint32 buffer_size,
     base::SharedMemoryHandle* handle) {
   AllocateSharedMemory(buffer_size, peer_handle_, handle);
 }
 
 void ChildProcessHostImpl::OnShutdownRequest() {
   if (delegate_->CanShutdown())
     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