Chromium Code Reviews| Index: media/video/gpu_memory_buffer_video_frame_pool.cc |
| diff --git a/media/video/gpu_memory_buffer_video_frame_pool.cc b/media/video/gpu_memory_buffer_video_frame_pool.cc |
| index d75f7aaa690b4b5a2442be1d1797cb642a86eb92..8743d434f594fe496146f20ba0dd5a67742c6d3c 100644 |
| --- a/media/video/gpu_memory_buffer_video_frame_pool.cc |
| +++ b/media/video/gpu_memory_buffer_video_frame_pool.cc |
| @@ -472,9 +472,9 @@ bool GpuMemoryBufferVideoFramePool::PoolImpl::OnMemoryDump( |
| dump->AddScalar("free_size", |
| base::trace_event::MemoryAllocatorDump::kUnitsBytes, |
| frame_resources->IsInUse() ? 0 : buffer_size_in_bytes); |
| - base::trace_event::MemoryAllocatorDumpGuid shared_buffer_guid = |
| - gfx::GetGpuMemoryBufferGUIDForTracing(tracing_process_id, |
| - buffer_id); |
| + const auto& shared_buffer_guid = |
|
DaleCurtis
2017/04/14 18:27:32
is const& really what you want here?
hajimehoshi
2017/04/17 06:18:07
Actually not. Done.
|
| + plane_resource.gpu_memory_buffer->GetHandle().GetGUIDForTracing( |
| + tracing_process_id); |
| pmd->CreateSharedGlobalAllocatorDump(shared_buffer_guid); |
| pmd->AddOwnershipEdge(dump->guid(), shared_buffer_guid, kImportance); |
| } |