Chromium Code Reviews| Index: cc/raster/staging_buffer_pool.cc |
| diff --git a/cc/raster/staging_buffer_pool.cc b/cc/raster/staging_buffer_pool.cc |
| index ade400d9906ad3c07a5b92280775f3c91644e6ab..e52fc46b2275f9a94c7ec4fac639c6064a76d4c6 100644 |
| --- a/cc/raster/staging_buffer_pool.cc |
| +++ b/cc/raster/staging_buffer_pool.cc |
| @@ -13,6 +13,7 @@ |
| #include "cc/base/container_util.h" |
| #include "cc/debug/traced_value.h" |
| #include "cc/resources/scoped_resource.h" |
| +#include "components/memory_coordinator/common/client_registry.h" |
| #include "gpu/command_buffer/client/gles2_interface.h" |
| namespace cc { |
| @@ -141,11 +142,22 @@ StagingBufferPool::StagingBufferPool(base::SequencedTaskRunner* task_runner, |
| this, "cc::StagingBufferPool", base::ThreadTaskRunnerHandle::Get()); |
| reduce_memory_usage_callback_ = base::Bind( |
| &StagingBufferPool::ReduceMemoryUsage, weak_ptr_factory_.GetWeakPtr()); |
| + |
| + // Register this component with memory_coordinator::ClientRegistry. |
| + memory_coordinator::ClientRegistry* registry = |
| + memory_coordinator::ClientRegistry::GetInstance(); |
| + if (registry) |
| + registry->RegisterClient(this); |
|
tasak
2016/09/01 09:39:32
Is it better to implement RegisterClientIfAvailabl
|
| } |
| StagingBufferPool::~StagingBufferPool() { |
| base::trace_event::MemoryDumpManager::GetInstance()->UnregisterDumpProvider( |
| this); |
| + // Unregister this component with memory_coordinator::ClientRegistry. |
| + memory_coordinator::ClientRegistry* registry = |
| + memory_coordinator::ClientRegistry::GetInstance(); |
| + if (registry) |
| + registry->UnregisterClient(this); |
| } |
| void StagingBufferPool::Shutdown() { |
| @@ -409,4 +421,24 @@ void StagingBufferPool::ReleaseBuffersNotUsedSince(base::TimeTicks time) { |
| } |
| } |
| +void StagingBufferPool::OnMemoryStateChange( |
| + memory_coordinator::mojom::MemoryState state) { |
| + switch (state) { |
| + case memory_coordinator::mojom::MemoryState::NORMAL: |
| + // TODO(tasak): go back to normal state. |
| + break; |
| + case memory_coordinator::mojom::MemoryState::THROTTLED: |
| + // TODO(tasak): make the limits of this component's caches smaller to |
| + // save memory usage. |
| + break; |
| + case memory_coordinator::mojom::MemoryState::SUSPENDED: |
| + // TODO(tasak): free this component's caches as much as possible before |
| + // suspending renderer. |
| + break; |
| + default: |
| + // NOT_REACHED. |
| + break; |
| + } |
| +} |
| + |
| } // namespace cc |