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

Unified Diff: content/common/child_process_host_impl.cc

Issue 685983005: gpu: Associate all GpuMemoryBuffers with unique IDs. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 1 month 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/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 6bf0d7ec58817b144f4f440c0537051dec3176da..23654958ebe7a8483c90572c3aa9c39b0168e982 100644
--- a/content/common/child_process_host_impl.cc
+++ b/content/common/child_process_host_impl.cc
@@ -79,6 +79,9 @@ base::FilePath TransformPathForFeature(const base::FilePath& path,
// Global atomic to generate child process unique IDs.
base::StaticAtomicSequenceNumber g_unique_id;
+// Global atomic to generate gpu memory buffer unique IDs.
+base::StaticAtomicSequenceNumber g_next_gpu_memory_buffer_id;
+
} // namespace
namespace content {
@@ -334,6 +337,7 @@ void ChildProcessHostImpl::OnAllocateGpuMemoryBuffer(
// Note: It is safe to use base::Unretained here as the shared memory
// implementation of AllocateForChildProcess() calls this synchronously.
GpuMemoryBufferImplSharedMemory::AllocateForChildProcess(
+ g_next_gpu_memory_buffer_id.GetNext(),
gfx::Size(width, height),
format,
peer_handle_,
@@ -344,7 +348,7 @@ void ChildProcessHostImpl::OnAllocateGpuMemoryBuffer(
void ChildProcessHostImpl::OnDeletedGpuMemoryBuffer(
gfx::GpuMemoryBufferType type,
- const gfx::GpuMemoryBufferId& id) {
+ gfx::GpuMemoryBufferId id) {
// Note: Nothing to do here as ownership of shared memory backed
// GpuMemoryBuffers is passed with IPC.
}

Powered by Google App Engine
This is Rietveld 408576698