Make cc::SoftwareImageDecodeController, cc::ResourcePool, and cc::StagingBufferPoo…

cc/raster/staging_buffer_pool.cc

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);
 }
 
 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) {

[ericrk, 2016/09/01 16:42:23]

Thanks for the thorough stubs! Out of curiosity, what thread is
OnMemoryStateChanged called on?

[haraken, 2016/09/01 17:42:05]

RegisterClient/UnregisterClient/OnMemoryStateChange must be called on the main
thread of the renderer process.

If StagingBufferPool's constructor/destructor can be called on a non-main
thread, we need to post a task to do the registration/unregistration on the main
thread.

+  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