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

Side by Side Diff: net/quic/core/quic_spdy_session.h

Issue 2446893003: Make Quic client more memory efficient. Protected by --quic_headers_stream_release_sequencer_buffer. (Closed)
Patch Set: Remove NOLINT Created 4 years, 1 month 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 (c) 2015 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2015 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_QUIC_QUIC_SPDY_SESSION_H_ 5 #ifndef NET_QUIC_QUIC_SPDY_SESSION_H_
6 #define NET_QUIC_QUIC_SPDY_SESSION_H_ 6 #define NET_QUIC_QUIC_SPDY_SESSION_H_
7 7
8 #include <stddef.h> 8 #include <stddef.h>
9 9
10 #include <memory> 10 #include <memory>
(...skipping 98 matching lines...) Expand 10 before | Expand all | Expand 10 after
109 bool fin); 109 bool fin);
110 110
111 bool force_hol_blocking() const { return force_hol_blocking_; } 111 bool force_hol_blocking() const { return force_hol_blocking_; }
112 112
113 bool server_push_enabled() const { return server_push_enabled_; } 113 bool server_push_enabled() const { return server_push_enabled_; }
114 114
115 // Called by |QuicHeadersStream::UpdateEnableServerPush()| with 115 // Called by |QuicHeadersStream::UpdateEnableServerPush()| with
116 // value from SETTINGS_ENABLE_PUSH. 116 // value from SETTINGS_ENABLE_PUSH.
117 void set_server_push_enabled(bool enable) { server_push_enabled_ = enable; } 117 void set_server_push_enabled(bool enable) { server_push_enabled_ = enable; }
118 118
119 // Return true if this session wants to release headers stream's buffer
120 // aggressively.
121 virtual bool ShouldReleaseHeadersStreamSequencerBuffer();
122
119 protected: 123 protected:
120 // Override CreateIncomingDynamicStream() and CreateOutgoingDynamicStream() 124 // Override CreateIncomingDynamicStream() and CreateOutgoingDynamicStream()
121 // with QuicSpdyStream return type to make sure that all data streams are 125 // with QuicSpdyStream return type to make sure that all data streams are
122 // QuicSpdyStreams. 126 // QuicSpdyStreams.
123 QuicSpdyStream* CreateIncomingDynamicStream(QuicStreamId id) override = 0; 127 QuicSpdyStream* CreateIncomingDynamicStream(QuicStreamId id) override = 0;
124 QuicSpdyStream* CreateOutgoingDynamicStream(SpdyPriority priority) override = 128 QuicSpdyStream* CreateOutgoingDynamicStream(SpdyPriority priority) override =
125 0; 129 0;
126 130
127 QuicSpdyStream* GetSpdyDataStream(const QuicStreamId stream_id); 131 QuicSpdyStream* GetSpdyDataStream(const QuicStreamId stream_id);
128 132
(...skipping 16 matching lines...) Expand all
145 // Set during handshake. If true, resources in x-associated-content and link 149 // Set during handshake. If true, resources in x-associated-content and link
146 // headers will be pushed. 150 // headers will be pushed.
147 bool server_push_enabled_; 151 bool server_push_enabled_;
148 152
149 DISALLOW_COPY_AND_ASSIGN(QuicSpdySession); 153 DISALLOW_COPY_AND_ASSIGN(QuicSpdySession);
150 }; 154 };
151 155
152 } // namespace net 156 } // namespace net
153 157
154 #endif // NET_QUIC_QUIC_SPDY_SESSION_H_ 158 #endif // NET_QUIC_QUIC_SPDY_SESSION_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698