Improve memory estimate of SpdySessionPool in net/ MemoryDumpProvider.

net/spdy/buffered_spdy_framer.cc

 // Copyright (c) 2012 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/buffered_spdy_framer.h"
 
 #include <algorithm>
 #include <utility>
 
 #include "base/logging.h"
 #include "base/strings/string_util.h"
+#include "base/trace_event/memory_usage_estimator.h"
 
 namespace net {
 
 namespace {
 
 // GOAWAY frame debug data is only buffered up to this many bytes.
 size_t kGoAwayDebugDataMaxSize = 1024;
 
 }  // namespace
 
@@ skipping 305 matching lines @@
     int weight,
     bool exclusive) const {
   SpdyPriorityIR priority_ir(stream_id, dependency_id, weight, exclusive);
   return new SpdySerializedFrame(spdy_framer_.SerializePriority(priority_ir));
 }
 
 SpdyPriority BufferedSpdyFramer::GetHighestPriority() const {
   return spdy_framer_.GetHighestPriority();
 }
 
+size_t BufferedSpdyFramer::EstimateMemoryUsage() const {
+  return base::trace_event::EstimateMemoryUsage(spdy_framer_) +
+         base::trace_event::EstimateMemoryUsage(header_buffer_) +
+         base::trace_event::EstimateMemoryUsage(coalescer_);

[DmitrySkiba, 2017/02/02 18:16:27]

I would also add control_frame_fields_ and  goaway_fields_.

[xunjieli, 2017/02/03 22:25:09]

Done for goaway_fields_. ControlFrameFields do not have members that can
allocate, so I omitted it.

+}
+
 void BufferedSpdyFramer::InitHeaderStreaming(SpdyStreamId stream_id) {
   header_buffer_.clear();
   header_buffer_valid_ = true;
   header_stream_id_ = stream_id;
   DCHECK_NE(header_stream_id_, SpdyFramer::kInvalidStream);
 }
 
 }  // namespace net