Make URLRequestContext a MemoryDumpProvider (Abandoned)

net/ssl/ssl_client_session_cache.h

Index: net/ssl/ssl_client_session_cache.h
diff --git a/net/ssl/ssl_client_session_cache.h b/net/ssl/ssl_client_session_cache.h
index cd668a2614def6cd59d343ca1bf2f16b38bf23bb..fd825ef05580a78f5fe074740150286c3ef9eb30 100644
--- a/net/ssl/ssl_client_session_cache.h
+++ b/net/ssl/ssl_client_session_cache.h
@@ -24,6 +24,9 @@
 
 namespace base {
 class Clock;
+namespace trace_event {
+class ProcessMemoryDump;
+}
 }
 
 namespace net {
@@ -56,6 +59,8 @@ class NET_EXPORT SSLClientSessionCache : public base::MemoryCoordinatorClient {
 
   void SetClockForTesting(std::unique_ptr<base::Clock> clock);
 
+  void DumpMemoryStats(base::trace_event::ProcessMemoryDump* pmd);

[ssid, 2016/12/02 21:36:35]

I wonder if it is better to add a MemoryDumpProvider implementation to this
class instead of calling this multiple times and dumping only the first time.
This class already seems to implement MemoryCoordinatorClient, so it should be
possible to implement MemoryDumpProvider.

[xunjieli, 2016/12/02 22:01:23]

There is an effort underway to tie it with a URLRequestContext (such that
SSLClientSessionCache won't be a singleton). I will keep it this way and not
make it into a MDP.

+
  private:
   // base::MemoryCoordinatorClient implementation:
   void OnMemoryStateChange(base::MemoryState state) override;