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

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

Issue 2300683002: Increase maximum size of the HPACK decoder dynamic table to 64 kB. (Closed)
Patch Set: Update Cronet Android tests. Created 4 years, 3 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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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_DECODER_H_ 5 #ifndef NET_SPDY_HPACK_HPACK_DECODER_H_
6 #define NET_SPDY_HPACK_HPACK_DECODER_H_ 6 #define NET_SPDY_HPACK_HPACK_DECODER_H_
7 7
8 #include <stddef.h> 8 #include <stddef.h>
9 #include <stdint.h> 9 #include <stdint.h>
10 10
(...skipping 20 matching lines...) Expand all
31 class HpackDecoderPeer; 31 class HpackDecoderPeer;
32 } // namespace test 32 } // namespace test
33 33
34 class NET_EXPORT_PRIVATE HpackDecoder : public HpackDecoderInterface { 34 class NET_EXPORT_PRIVATE HpackDecoder : public HpackDecoderInterface {
35 public: 35 public:
36 friend class test::HpackDecoderPeer; 36 friend class test::HpackDecoderPeer;
37 37
38 HpackDecoder(); 38 HpackDecoder();
39 ~HpackDecoder() override; 39 ~HpackDecoder() override;
40 40
41 // Called upon acknowledgement of SETTINGS_HEADER_TABLE_SIZE. 41 // Called upon sending a SETTINGS_HEADER_TABLE_SIZE value.
42 void ApplyHeaderTableSizeSetting(size_t size_setting) override; 42 void ApplyHeaderTableSizeSetting(size_t size_setting) override;
43 43
44 // If a SpdyHeadersHandlerInterface is provided, HpackDecoder will emit 44 // If a SpdyHeadersHandlerInterface is provided, HpackDecoder will emit
45 // headers to it rather than accumulating them in a SpdyHeaderBlock. 45 // headers to it rather than accumulating them in a SpdyHeaderBlock.
46 void HandleControlFrameHeadersStart( 46 void HandleControlFrameHeadersStart(
47 SpdyHeadersHandlerInterface* handler) override; 47 SpdyHeadersHandlerInterface* handler) override;
48 48
49 // Called as headers data arrives. Returns false if an error occurred. 49 // Called as headers data arrives. Returns false if an error occurred.
50 // TODO(jgraettinger): A future version of this method will incrementally 50 // TODO(jgraettinger): A future version of this method will incrementally
51 // parse and deliver headers via SpdyHeadersHandlerInterface. For now, 51 // parse and deliver headers via SpdyHeadersHandlerInterface. For now,
(...skipping 93 matching lines...) Expand 10 before | Expand all | Expand 10 after
145 145
146 // Saved value of --gfe2_reloadable_flag_add_hpack_incremental_decode. 146 // Saved value of --gfe2_reloadable_flag_add_hpack_incremental_decode.
147 bool incremental_decode_; 147 bool incremental_decode_;
148 148
149 DISALLOW_COPY_AND_ASSIGN(HpackDecoder); 149 DISALLOW_COPY_AND_ASSIGN(HpackDecoder);
150 }; 150 };
151 151
152 } // namespace net 152 } // namespace net
153 153
154 #endif // NET_SPDY_HPACK_HPACK_DECODER_H_ 154 #endif // NET_SPDY_HPACK_HPACK_DECODER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698