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

Unified Diff: content/browser/gpu/gpu_memory_buffer_factory_host_impl.cc

Issue 654223006: Cleanup GpuMemoryBuffer (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: compile 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/browser/gpu/gpu_memory_buffer_factory_host_impl.cc
diff --git a/content/browser/gpu/gpu_memory_buffer_factory_host_impl.cc b/content/browser/gpu/gpu_memory_buffer_factory_host_impl.cc
index 10112cbf013ed22c7228c66e656245e5bbed8932..0fc448e36569b59b4d57fd79746911f94d66f3df 100644
--- a/content/browser/gpu/gpu_memory_buffer_factory_host_impl.cc
+++ b/content/browser/gpu/gpu_memory_buffer_factory_host_impl.cc
@@ -24,6 +24,19 @@ void GpuMemoryBufferFactoryHostImpl::CreateGpuMemoryBuffer(
gfx::GpuMemoryBuffer::Format format,
gfx::GpuMemoryBuffer::Usage usage,
const CreateGpuMemoryBufferCallback& callback) {
+ BrowserThread::PostTask(
+ BrowserThread::IO, FROM_HERE,
+ base::Bind(&GpuMemoryBufferFactoryHostImpl::CreateGpuMemoryBufferOnIO,
+ base::Unretained(this), handle, size, format, usage,
+ callback));
reveman 2014/11/03 22:26:12 nit: cl format?
+}
+
+void GpuMemoryBufferFactoryHostImpl::CreateGpuMemoryBufferOnIO(
+ const gfx::GpuMemoryBufferHandle& handle,
+ const gfx::Size& size,
+ gfx::GpuMemoryBuffer::Format format,
+ gfx::GpuMemoryBuffer::Usage usage,
+ const CreateGpuMemoryBufferCallback& callback) {
DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
GpuProcessHost* host = GpuProcessHost::FromID(gpu_host_id_);
@@ -48,6 +61,15 @@ void GpuMemoryBufferFactoryHostImpl::CreateGpuMemoryBuffer(
void GpuMemoryBufferFactoryHostImpl::DestroyGpuMemoryBuffer(
const gfx::GpuMemoryBufferHandle& handle,
int32 sync_point) {
+ BrowserThread::PostTask(
+ BrowserThread::IO, FROM_HERE,
+ base::Bind(&GpuMemoryBufferFactoryHostImpl::DestroyGpuMemoryBufferOnIO,
+ base::Unretained(this), handle, sync_point));
reveman 2014/11/03 22:26:12 nit: cl format?
alexst (slow to review) 2014/11/03 22:47:42 I was as surprised as you, this is the result of '
+}
+
+void GpuMemoryBufferFactoryHostImpl::DestroyGpuMemoryBufferOnIO(
+ const gfx::GpuMemoryBufferHandle& handle,
+ int32 sync_point) {
DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
GpuProcessHost* host = GpuProcessHost::FromID(gpu_host_id_);

Powered by Google App Engine
This is Rietveld 408576698