Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(547)

Unified Diff: net/spdy/hpack/hpack_decoder2.cc

Issue 2665283003: Improve memory estimate of SpdySessionPool in net/ MemoryDumpProvider. (Closed)
Patch Set: rebased Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: net/spdy/hpack/hpack_decoder2.cc
diff --git a/net/spdy/hpack/hpack_decoder2.cc b/net/spdy/hpack/hpack_decoder2.cc
index 03b1c2ef229415eb0322a972f1a5cbc40b96d1ae..db1d5e5f7c2e0e05e77db946cf966344740b0f92 100644
--- a/net/spdy/hpack/hpack_decoder2.cc
+++ b/net/spdy/hpack/hpack_decoder2.cc
@@ -9,6 +9,7 @@
#include "base/logging.h"
#include "base/strings/string_piece.h"
+#include "base/trace_event/memory_usage_estimator.h"
#include "net/http2/decoder/decode_buffer.h"
#include "net/http2/decoder/decode_status.h"
#include "net/spdy/hpack/hpack_entry.h"
@@ -159,6 +160,13 @@ void HpackDecoder2::set_max_decode_buffer_size_bytes(
max_decode_buffer_size_bytes_ = max_decode_buffer_size_bytes;
}
+size_t HpackDecoder2::EstimateMemoryUsage() const {
+ return base::trace_event::EstimateMemoryUsage(header_table_) +
+ base::trace_event::EstimateMemoryUsage(decoded_block_) +
+ base::trace_event::EstimateMemoryUsage(name_) +
+ base::trace_event::EstimateMemoryUsage(value_);
+}
+
void HpackDecoder2::OnIndexedHeader(size_t index) {
DVLOG(2) << "HpackDecoder2::OnIndexedHeader: index=" << index;
DCHECK(!error_detected_);

Powered by Google App Engine
This is Rietveld 408576698