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

Unified Diff: content/browser/renderer_host/render_message_filter.cc

Issue 703463005: Re-land: content: Cleanup GpuMemoryBuffers when child process is removed. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@gpu-memory-buffer-id-refactor
Patch Set: handle allocation failure correctly 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/renderer_host/render_message_filter.cc
diff --git a/content/browser/renderer_host/render_message_filter.cc b/content/browser/renderer_host/render_message_filter.cc
index b218da885612c4fe06e6e7d5e07c8ebdfa9f0ad1..27b9cd54f5f9bd8c904fae355ee41fd6e6cbbc06 100644
--- a/content/browser/renderer_host/render_message_filter.cc
+++ b/content/browser/renderer_host/render_message_filter.cc
@@ -326,7 +326,8 @@ RenderMessageFilter::~RenderMessageFilter() {
DCHECK_CURRENTLY_ON(BrowserThread::IO);
DCHECK(plugin_host_clients_.empty());
HostSharedBitmapManager::current()->ProcessRemoved(PeerHandle());
- BrowserGpuMemoryBufferManager::current()->ProcessRemoved(PeerHandle());
+ BrowserGpuMemoryBufferManager::current()->ProcessRemoved(
+ PeerHandle(), render_process_id_);
HostDiscardableSharedMemoryManager::current()->ProcessRemoved(PeerHandle());
}
@@ -1271,11 +1272,10 @@ void RenderMessageFilter::GpuMemoryBufferAllocated(
}
void RenderMessageFilter::OnDeletedGpuMemoryBuffer(
- gfx::GpuMemoryBufferType type,
gfx::GpuMemoryBufferId id,
uint32 sync_point) {
BrowserGpuMemoryBufferManager::current()->ChildProcessDeletedGpuMemoryBuffer(
- type, id, PeerHandle(), render_process_id_, sync_point);
+ id, PeerHandle(), render_process_id_, sync_point);
}
} // namespace content

Powered by Google App Engine
This is Rietveld 408576698