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

Side by Side Diff: net/quic/chromium/quic_http_stream.h

Issue 2844493002: QuicChromiumClientSession::StreamRequest to be created by the session (Closed)
Patch Set: Rebase Created 3 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
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_QUIC_CHROMIUM_QUIC_HTTP_STREAM_H_ 5 #ifndef NET_QUIC_CHROMIUM_QUIC_HTTP_STREAM_H_
6 #define NET_QUIC_CHROMIUM_QUIC_HTTP_STREAM_H_ 6 #define NET_QUIC_CHROMIUM_QUIC_HTTP_STREAM_H_
7 7
8 #include <stddef.h> 8 #include <stddef.h>
9 #include <stdint.h> 9 #include <stdint.h>
10 10
(...skipping 141 matching lines...) Expand 10 before | Expand all | Expand 10 after
152 State next_state_; 152 State next_state_;
153 153
154 base::WeakPtr<QuicChromiumClientSession> session_; 154 base::WeakPtr<QuicChromiumClientSession> session_;
155 const QuicServerId server_id_; // The ID of the QUIC server for this stream. 155 const QuicServerId server_id_; // The ID of the QUIC server for this stream.
156 156
157 HttpServerProperties* http_server_properties_; // Unowned. 157 HttpServerProperties* http_server_properties_; // Unowned.
158 158
159 QuicVersion quic_version_; 159 QuicVersion quic_version_;
160 int session_error_; // Error code from the connection shutdown. 160 int session_error_; // Error code from the connection shutdown.
161 bool was_handshake_confirmed_; // True if the crypto handshake succeeded. 161 bool was_handshake_confirmed_; // True if the crypto handshake succeeded.
162 QuicChromiumClientSession::StreamRequest stream_request_; 162 std::unique_ptr<QuicChromiumClientSession::StreamRequest> stream_request_;
163 QuicChromiumClientStream* stream_; // Non-owning. 163 QuicChromiumClientStream* stream_; // Non-owning.
164 164
165 // The following three fields are all owned by the caller and must 165 // The following three fields are all owned by the caller and must
166 // outlive this object, according to the HttpStream contract. 166 // outlive this object, according to the HttpStream contract.
167 167
168 // The request to send. 168 // The request to send.
169 // Only valid before the response body is read. 169 // Only valid before the response body is read.
170 const HttpRequestInfo* request_info_; 170 const HttpRequestInfo* request_info_;
171 171
172 // The request body to send, if any, owned by the caller. 172 // The request body to send, if any, owned by the caller.
(...skipping 65 matching lines...) Expand 10 before | Expand all | Expand 10 after
238 LoadTimingInfo::ConnectTiming connect_timing_; 238 LoadTimingInfo::ConnectTiming connect_timing_;
239 239
240 base::WeakPtrFactory<QuicHttpStream> weak_factory_; 240 base::WeakPtrFactory<QuicHttpStream> weak_factory_;
241 241
242 DISALLOW_COPY_AND_ASSIGN(QuicHttpStream); 242 DISALLOW_COPY_AND_ASSIGN(QuicHttpStream);
243 }; 243 };
244 244
245 } // namespace net 245 } // namespace net
246 246
247 #endif // NET_QUIC_CHROMIUM_QUIC_HTTP_STREAM_H_ 247 #endif // NET_QUIC_CHROMIUM_QUIC_HTTP_STREAM_H_
OLDNEW
« no previous file with comments | « net/quic/chromium/quic_chromium_client_session_test.cc ('k') | net/quic/chromium/quic_http_stream.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698