Make DOMStorageContextWrapper a client of memory coordinator

content/browser/dom_storage/dom_storage_context_wrapper.h

Index: content/browser/dom_storage/dom_storage_context_wrapper.h
diff --git a/content/browser/dom_storage/dom_storage_context_wrapper.h b/content/browser/dom_storage/dom_storage_context_wrapper.h
index 701324850accb5570bacdd80feabc529dc074dbb..8c663099fbacccf0f364103aae345eeb993e55f3 100644
--- a/content/browser/dom_storage/dom_storage_context_wrapper.h
+++ b/content/browser/dom_storage/dom_storage_context_wrapper.h
@@ -11,6 +11,7 @@
 #include "base/macros.h"
 #include "base/memory/memory_pressure_listener.h"
 #include "base/memory/ref_counted.h"
+#include "components/memory_coordinator/common/memory_coordinator_client.h"
 #include "content/common/content_export.h"
 #include "content/common/storage_partition_service.mojom.h"
 #include "content/public/browser/dom_storage_context.h"
@@ -37,7 +38,8 @@ class LevelDBWrapperImpl;
 // state.
 class CONTENT_EXPORT DOMStorageContextWrapper :
     NON_EXPORTED_BASE(public DOMStorageContext),
-    public base::RefCountedThreadSafe<DOMStorageContextWrapper> {
+    public base::RefCountedThreadSafe<DOMStorageContextWrapper>,
+    public memory_coordinator::MemoryCoordinatorClient {
  public:
   // If |data_path| is empty, nothing will be saved to disk.
   DOMStorageContextWrapper(
@@ -78,6 +80,9 @@ class CONTENT_EXPORT DOMStorageContextWrapper :
   void OnMemoryPressure(
       base::MemoryPressureListener::MemoryPressureLevel memory_pressure_level);
 
+  // memory_coordinator::MemoryCoordinatorClient implementation:

[michaeln, 2016/09/14 01:34:41]

while your here, can u make all these OnMemory related methods private?

[hajimehoshi, 2016/09/14 11:49:48]

Done.

+  void OnMemoryStateChange(memory_coordinator::MemoryState state) override;
+
  private:
   friend class DOMStorageMessageFilter;  // for access to context()
   friend class SessionStorageNamespaceImpl;  // ditto
@@ -86,6 +91,8 @@ class CONTENT_EXPORT DOMStorageContextWrapper :
   ~DOMStorageContextWrapper() override;
   DOMStorageContextImpl* context() const { return context_.get(); }
 
+  void PurgeMemory(bool aggressively);
+
   // An inner class to keep all mojo-ish details together and not bleed them
   // through the public interface.
   class MojoState;