Log states of JobController in Net MemoryDumpProvider

net/http/http_network_session.cc

Index: net/http/http_network_session.cc
diff --git a/net/http/http_network_session.cc b/net/http/http_network_session.cc
index c64755a707212620dda5f4963330f8dd9980bd27..ce15b0742923b63c9c38284042ceffcf7fd60807 100644
--- a/net/http/http_network_session.cc
+++ b/net/http/http_network_session.cc
@@ -17,6 +17,7 @@
 #include "base/strings/string_util.h"
 #include "base/strings/stringprintf.h"
 #include "base/trace_event/memory_allocator_dump.h"
+#include "base/trace_event/memory_dump_request_args.h"
 #include "base/trace_event/process_memory_dump.h"
 #include "base/values.h"
 #include "net/base/network_throttle_manager_impl.h"
@@ -418,6 +419,7 @@ void HttpNetworkSession::GetSSLConfig(const HttpRequestInfo& request,
 
 void HttpNetworkSession::DumpMemoryStats(
     base::trace_event::ProcessMemoryDump* pmd,
+    const base::trace_event::MemoryDumpArgs& args,
     const std::string& parent_absolute_name) const {
   std::string name = base::StringPrintf("net/http_network_session_%p", this);
   base::trace_event::MemoryAllocatorDump* http_network_session_dump =
@@ -428,6 +430,12 @@ void HttpNetworkSession::DumpMemoryStats(
         pmd, http_network_session_dump->absolute_name());
     spdy_session_pool_.DumpMemoryStats(
         pmd, http_network_session_dump->absolute_name());
+    if (args.level_of_detail ==

[ssid, 2017/01/13 22:23:58]

I think your should call this always. Yes it adds an additional row. But this is
going to account for memory usage better.
Else we will miss tracking the size of stream factory in background mode.

[xunjieli, 2017/01/17 17:51:23]

Done.

+            base::trace_event::MemoryDumpLevelOfDetail::DETAILED &&
+        http_stream_factory_) {
+      http_stream_factory_->DumpMemoryStats(
+          pmd, http_network_session_dump->absolute_name());
+    }
   }
   // Create an empty row under parent's dump so size can be attributed correctly
   // if |this| is shared between URLRequestContexts.