Improve memory estimate of SpdySessionPool in net/ MemoryDumpProvider.

net/spdy/hpack/hpack_encoder.cc

 // Copyright 2014 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/hpack/hpack_encoder.h"
 
 #include <algorithm>
 #include <limits>
 
 #include "base/logging.h"
@@ skipping 114 matching lines @@
     return;
   }
   if (size_setting < header_table_.settings_size_bound()) {
     min_table_size_setting_received_ =
         std::min(size_setting, min_table_size_setting_received_);
   }
   header_table_.SetSettingsHeaderTableSize(size_setting);
   should_emit_table_size_ = true;
 }
 
+size_t HpackEncoder::EstimateMemoryUsage() const {
+  // |huffman_table_| is a singleton. It's accounted for in spdy_session_pool.cc
+  return header_table_.size() + output_stream_.size();

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

They should both be EMUs - header_table_.size() it doesn't include overhead from
deque / unordered_map, and output_stream_.size() returns buffer's size, not
capacity.

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

Done.

+}
+
 void HpackEncoder::EncodeRepresentations(RepresentationIterator* iter,
                                          string* output) {
   MaybeEmitTableSize();
   while (iter->HasNext()) {
     const auto header = iter->Next();
     listener_(header.first, header.second);
     if (enable_compression_) {
       const HpackEntry* entry =
           header_table_.GetByNameAndValue(header.first, header.second);
       if (entry != nullptr) {
@@ skipping 212 matching lines @@
   has_next_ = encoder_->output_stream_.size() > max_encoded_bytes;
   encoder_->output_stream_.BoundedTakeString(max_encoded_bytes, output);
 }
 
 std::unique_ptr<HpackEncoder::ProgressiveEncoder> HpackEncoder::EncodeHeaderSet(
     const SpdyHeaderBlock& header_set) {
   return base::MakeUnique<Encoderator>(header_set, this);
 }
 
 }  // namespace net