Improve memory estimate of SpdySessionPool in net/ MemoryDumpProvider.

net/http2/hpack/decoder/http2_hpack_decoder.cc

 // Copyright 2017 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/http2/hpack/decoder/http2_hpack_decoder.h"
 
 #include "base/logging.h"
 #include "net/http2/decoder/decode_status.h"
 
 using base::StringPiece;
@@ skipping 98 matching lines @@
       DVLOG(2) << "Http2HpackDecoder::error_detected in entry_buffer_";
       error_detected_ = true;
     } else if (decoder_state_.error_detected()) {
       DVLOG(2) << "Http2HpackDecoder::error_detected in decoder_state_";
       error_detected_ = true;
     }
   }
   return error_detected_;
 }
 
+size_t Http2HpackDecoder::EstimateMemoryUsage() const {
+  return entry_buffer_.EstimateMemoryUsage();

[Bence, 2017/02/05 19:11:23]

I thought EstimateMemoryUsage(entry_buffer_) is preferred per
https://codereview.chromium.org/2665283003/#msg16 comment 3.

[xunjieli, 2017/02/06 14:54:56]

Done. Oops sorry. Good catch! I missed this one and the ones below.

+}
+
 void Http2HpackDecoder::ReportError(StringPiece error_message) {
   DVLOG(3) << "Http2HpackDecoder::ReportError is new="
            << (!error_detected_ ? "true" : "false")
            << ", error_message: " << error_message;
   if (!error_detected_) {
     error_detected_ = true;
     listener()->OnHeaderErrorDetected(error_message);
   }
 }
 
 }  // namespace net