Improve memory estimate of SpdySessionPool in net/ MemoryDumpProvider.

net/spdy/http2_priority_dependencies.cc

 // Copyright 2016 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 "net/spdy/http2_priority_dependencies.h"
+#include "base/trace_event/memory_usage_estimator.h"
 
 namespace net {
 
 Http2PriorityDependencies::Http2PriorityDependencies() {}
 
 Http2PriorityDependencies::~Http2PriorityDependencies() {}
 
 void Http2PriorityDependencies::OnStreamCreation(
     SpdyStreamId id,
     SpdyPriority priority,
@@ skipping 134 matching lines @@
 
 void Http2PriorityDependencies::OnStreamDestruction(SpdyStreamId id) {
   EntryMap::iterator emit = entry_by_stream_id_.find(id);
   DCHECK(emit != entry_by_stream_id_.end());
 
   IdList::iterator it = emit->second;
   id_priority_lists_[it->second].erase(it);
   entry_by_stream_id_.erase(emit);
 }
 
+size_t Http2PriorityDependencies::EstimateMemoryUsage() const {
+  return base::trace_event::EstimateMemoryUsage(id_priority_lists_);
+  // TODO(xunjieli): https://crbug.com/669108. Include |entry_by_stream_id_|
+  // when memory_usage_estimator.h supports std::list::iterator.

[DmitrySkiba, 2017/02/06 18:02:34]

Nice catch! I'll implement EMUs for iterators (I would expect iterators to have
trivial dtors, but I guess some STL implementations have debug checks in them).

[xunjieli, 2017/02/08 15:42:50]

Acknowledged. Thanks! I will file 690015 to keep track of it. Updated the
comment as well.

+}
+
 }  // namespace net