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

Side by Side Diff: net/spdy/spdy_framer.h

Issue 267063006: Allocate HpackDecoder/Encoder only for SpdyFramer > SPDY3. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 7 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 | Annotate | Revision Log
« no previous file with comments | « no previous file | net/spdy/spdy_framer.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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_SPDY_FRAMER_H_ 5 #ifndef NET_SPDY_SPDY_FRAMER_H_
6 #define NET_SPDY_SPDY_FRAMER_H_ 6 #define NET_SPDY_SPDY_FRAMER_H_
7 7
8 #include <list> 8 #include <list>
9 #include <map> 9 #include <map>
10 #include <string> 10 #include <string>
(...skipping 683 matching lines...) Expand 10 before | Expand all | Expand 10 after
694 // Scratch space for handling SETTINGS frames. 694 // Scratch space for handling SETTINGS frames.
695 // TODO(hkhalil): Unify memory for this scratch space with 695 // TODO(hkhalil): Unify memory for this scratch space with
696 // current_frame_buffer_. 696 // current_frame_buffer_.
697 SpdySettingsScratch settings_scratch_; 697 SpdySettingsScratch settings_scratch_;
698 698
699 bool enable_compression_; // Controls all compression 699 bool enable_compression_; // Controls all compression
700 // SPDY header compressors. 700 // SPDY header compressors.
701 scoped_ptr<z_stream> header_compressor_; 701 scoped_ptr<z_stream> header_compressor_;
702 scoped_ptr<z_stream> header_decompressor_; 702 scoped_ptr<z_stream> header_decompressor_;
703 703
704 HpackEncoder hpack_encoder_; 704 scoped_ptr<HpackEncoder> hpack_encoder_;
705 HpackDecoder hpack_decoder_; 705 scoped_ptr<HpackDecoder> hpack_decoder_;
706 706
707 SpdyFramerVisitorInterface* visitor_; 707 SpdyFramerVisitorInterface* visitor_;
708 SpdyFramerDebugVisitorInterface* debug_visitor_; 708 SpdyFramerDebugVisitorInterface* debug_visitor_;
709 709
710 std::string display_protocol_; 710 std::string display_protocol_;
711 711
712 // The major SPDY version to be spoken/understood by this framer. 712 // The major SPDY version to be spoken/understood by this framer.
713 const SpdyMajorVersion spdy_version_; 713 const SpdyMajorVersion spdy_version_;
714 714
715 // Tracks if we've ever gotten far enough in framing to see a control frame of 715 // Tracks if we've ever gotten far enough in framing to see a control frame of
(...skipping 20 matching lines...) Expand all
736 // If a HEADERS frame is followed by a CONTINUATION frame, the FIN/END_STREAM 736 // If a HEADERS frame is followed by a CONTINUATION frame, the FIN/END_STREAM
737 // flag is still carried in the HEADERS frame. If it's set, flip this so that 737 // flag is still carried in the HEADERS frame. If it's set, flip this so that
738 // we know to terminate the stream when the entire header block has been 738 // we know to terminate the stream when the entire header block has been
739 // processed. 739 // processed.
740 bool end_stream_when_done_; 740 bool end_stream_when_done_;
741 }; 741 };
742 742
743 } // namespace net 743 } // namespace net
744 744
745 #endif // NET_SPDY_SPDY_FRAMER_H_ 745 #endif // NET_SPDY_SPDY_FRAMER_H_
OLDNEW
« no previous file with comments | « no previous file | net/spdy/spdy_framer.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698