Make RenderThreadImpl a client of memory coordinator

third_party/WebKit/Source/platform/MemoryCoordinator.cpp

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "platform/MemoryCoordinator.h"
 
 #include "platform/fonts/FontCache.h"
 #include "platform/graphics/ImageDecodingStore.h"
 #include "platform/tracing/TraceEvent.h"
 #include "wtf/allocator/Partitions.h"
@@ skipping 24 matching lines @@
 void MemoryCoordinator::prepareToSuspend() {
   for (auto& client : m_clients)
     client->prepareToSuspend();
   WTF::Partitions::decommitFreeableMemory();
 }
 
 void MemoryCoordinator::onMemoryPressure(WebMemoryPressureLevel level) {
   TRACE_EVENT0("blink", "MemoryCoordinator::onMemoryPressure");
   for (auto& client : m_clients)
     client->onMemoryPressure(level);
-  if (level == WebMemoryPressureLevelCritical) {
+  clearMemory(level == WebMemoryPressureLevelCritical);
+}
+
+void MemoryCoordinator::onMemoryStateChange(MemoryState state) {
+  for (auto& client : m_clients)
+    client->onMemoryStateChange(state);
+  clearMemory(state == MemoryState::SUSPENDED);
+}
+
+void MemoryCoordinator::clearMemory(bool critical) {

[haraken, 2016/10/07 11:45:30]

I'd avoid introducing a boolean flag. See my below comment.

[hajimehoshi, 2016/10/11 07:05:32]

Done.

+  if (critical) {
     // Clear the image cache.
     // TODO(tasak|bashi): Make ImageDecodingStore and FontCache be
     // MemoryCoordinatorClients rather than clearing caches here.
     ImageDecodingStore::instance().clear();
     FontCache::fontCache()->invalidate();
   }
   WTF::Partitions::decommitFreeableMemory();
 }
 
 DEFINE_TRACE(MemoryCoordinator) {
   visitor->trace(m_clients);
 }
 
 }  // namespace blink