Log states of JobController in Net MemoryDumpProvider

net/http/http_stream_factory.h

Index: net/http/http_stream_factory.h
diff --git a/net/http/http_stream_factory.h b/net/http/http_stream_factory.h
index c4a95798cc5726ee0374e5c9fbf73cb3447b3f6c..8424e8ea841908ef761c28519a6327dfb9a81f0a 100644
--- a/net/http/http_stream_factory.h
+++ b/net/http/http_stream_factory.h
@@ -22,6 +22,12 @@
 // introduce any link dependency to net/websockets.
 #include "net/websockets/websocket_handshake_stream_base.h"
 
+namespace base {
+namespace trace_event {
+class ProcessMemoryDump;
+}
+}
+
 namespace net {
 
 class AuthCredentials;
@@ -235,6 +241,11 @@ class NET_EXPORT HttpStreamFactory {
 
   virtual const HostMappingRules* GetHostMappingRules() const = 0;
 
+  // Dumps memory allocation stats. |parent_dump_absolute_name| is the name
+  // used by the parent MemoryAllocatorDump in the memory dump hierarchy.
+  virtual void DumpMemoryStats(base::trace_event::ProcessMemoryDump* pmd,
+                               const std::string& parent_absolute_name) const {}

[ssid, 2017/01/18 23:51:52]

Can we make it pure virtual, so that some other implementations in future does
not miss this? Just an extra definition in the unittest I guess?

[xunjieli, 2017/01/19 14:50:41]

Done.

+
  protected:
   HttpStreamFactory();