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

Side by Side Diff: net/spdy/hpack/hpack_decoder2.h

Issue 2665283003: Improve memory estimate of SpdySessionPool in net/ MemoryDumpProvider. (Closed)
Patch Set: address bnc comments 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 unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef NET_SPDY_HPACK_HPACK_DECODER2_H_ 5 #ifndef NET_SPDY_HPACK_HPACK_DECODER2_H_
6 #define NET_SPDY_HPACK_HPACK_DECODER2_H_ 6 #define NET_SPDY_HPACK_HPACK_DECODER2_H_
7 7
8 // HpackDecoder2 8 // HpackDecoder2
9 9
10 // An HpackDecoder decodes header sets as outlined in 10 // An HpackDecoder decodes header sets as outlined in
(...skipping 38 matching lines...) Expand 10 before | Expand all | Expand 10 after
49 SpdyHeadersHandlerInterface* handler) override; 49 SpdyHeadersHandlerInterface* handler) override;
50 bool HandleControlFrameHeadersData(const char* headers_data, 50 bool HandleControlFrameHeadersData(const char* headers_data,
51 size_t headers_data_length) override; 51 size_t headers_data_length) override;
52 bool HandleControlFrameHeadersComplete(size_t* compressed_len) override; 52 bool HandleControlFrameHeadersComplete(size_t* compressed_len) override;
53 const SpdyHeaderBlock& decoded_block() const override; 53 const SpdyHeaderBlock& decoded_block() const override;
54 void SetHeaderTableDebugVisitor( 54 void SetHeaderTableDebugVisitor(
55 std::unique_ptr<HpackHeaderTable::DebugVisitorInterface> visitor) 55 std::unique_ptr<HpackHeaderTable::DebugVisitorInterface> visitor)
56 override; 56 override;
57 void set_max_decode_buffer_size_bytes( 57 void set_max_decode_buffer_size_bytes(
58 size_t max_decode_buffer_size_bytes) override; 58 size_t max_decode_buffer_size_bytes) override;
59 size_t EstimateMemoryUsage() const override;
59 60
60 protected: 61 protected:
61 // Override the HpackEntryDecoderListener methods: 62 // Override the HpackEntryDecoderListener methods:
62 63
63 void OnIndexedHeader(size_t index) override; 64 void OnIndexedHeader(size_t index) override;
64 void OnStartLiteralHeader(HpackEntryType entry_type, 65 void OnStartLiteralHeader(HpackEntryType entry_type,
65 size_t maybe_name_index) override; 66 size_t maybe_name_index) override;
66 void OnNameStart(bool huffman_encoded, size_t len) override; 67 void OnNameStart(bool huffman_encoded, size_t len) override;
67 void OnNameData(const char* data, size_t len) override; 68 void OnNameData(const char* data, size_t len) override;
68 void OnNameEnd() override; 69 void OnNameEnd() override;
(...skipping 68 matching lines...) Expand 10 before | Expand all | Expand 10 after
137 // is not being provided by the caller. 138 // is not being provided by the caller.
138 // TODO(jamessynge): Consider collapsing several of these bools into a single 139 // TODO(jamessynge): Consider collapsing several of these bools into a single
139 // enum representing the state of the decoding process. 140 // enum representing the state of the decoding process.
140 bool header_block_started_; 141 bool header_block_started_;
141 142
142 DISALLOW_COPY_AND_ASSIGN(HpackDecoder2); 143 DISALLOW_COPY_AND_ASSIGN(HpackDecoder2);
143 }; 144 };
144 145
145 } // namespace net 146 } // namespace net
146 #endif // NET_SPDY_HPACK_HPACK_DECODER2_H_ 147 #endif // NET_SPDY_HPACK_HPACK_DECODER2_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698