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

Side by Side Diff: net/quic/quic_client_session.h

Issue 393953009: Moving the work currently done in the QuicSession constructor to (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 5 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/quic/quic_client_session.cc » ('j') | net/quic/quic_client_session.cc » ('J')
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 // A client specific QuicSession subclass. This class owns the underlying 5 // A client specific QuicSession subclass. This class owns the underlying
6 // QuicConnection and QuicConnectionHelper objects. The connection stores 6 // QuicConnection and QuicConnectionHelper objects. The connection stores
7 // a non-owning pointer to the helper so this session needs to ensure that 7 // a non-owning pointer to the helper so this session needs to ensure that
8 // the helper outlives the connection. 8 // the helper outlives the connection.
9 9
10 #ifndef NET_QUIC_QUIC_CLIENT_SESSION_H_ 10 #ifndef NET_QUIC_QUIC_CLIENT_SESSION_H_
(...skipping 76 matching lines...) Expand 10 before | Expand all | Expand 10 after
87 DISALLOW_COPY_AND_ASSIGN(StreamRequest); 87 DISALLOW_COPY_AND_ASSIGN(StreamRequest);
88 }; 88 };
89 89
90 // Constructs a new session connected to |server_id| which will own 90 // Constructs a new session connected to |server_id| which will own
91 // |connection|, but not |stream_factory|, which must outlive this session. 91 // |connection|, but not |stream_factory|, which must outlive this session.
92 // TODO(rch): decouple the factory from the session via a Delegate interface. 92 // TODO(rch): decouple the factory from the session via a Delegate interface.
93 QuicClientSession(QuicConnection* connection, 93 QuicClientSession(QuicConnection* connection,
94 scoped_ptr<DatagramClientSocket> socket, 94 scoped_ptr<DatagramClientSocket> socket,
95 scoped_ptr<QuicDefaultPacketWriter> writer, 95 scoped_ptr<QuicDefaultPacketWriter> writer,
96 QuicStreamFactory* stream_factory, 96 QuicStreamFactory* stream_factory,
97 QuicCryptoClientStreamFactory* crypto_client_stream_factory,
98 scoped_ptr<QuicServerInfo> server_info, 97 scoped_ptr<QuicServerInfo> server_info,
99 const QuicServerId& server_id,
100 const QuicConfig& config, 98 const QuicConfig& config,
101 QuicCryptoClientConfig* crypto_config,
102 base::TaskRunner* task_runner, 99 base::TaskRunner* task_runner,
103 NetLog* net_log); 100 NetLog* net_log);
104 virtual ~QuicClientSession(); 101 virtual ~QuicClientSession();
105 102
103 void InitializeSession(
104 const QuicServerId& server_id,
105 QuicCryptoClientConfig* config,
106 QuicCryptoClientStreamFactory* crypto_client_stream_factory);
107
106 void AddObserver(Observer* observer); 108 void AddObserver(Observer* observer);
107 void RemoveObserver(Observer* observer); 109 void RemoveObserver(Observer* observer);
108 110
109 // Attempts to create a new stream. If the stream can be 111 // Attempts to create a new stream. If the stream can be
110 // created immediately, returns OK. If the open stream limit 112 // created immediately, returns OK. If the open stream limit
111 // has been reached, returns ERR_IO_PENDING, and |request| 113 // has been reached, returns ERR_IO_PENDING, and |request|
112 // will be added to the stream requets queue and will 114 // will be added to the stream requets queue and will
113 // be completed asynchronously. 115 // be completed asynchronously.
114 // TODO(rch): remove |stream| from this and use setter on |request| 116 // TODO(rch): remove |stream| from this and use setter on |request|
115 // and fix in spdy too. 117 // and fix in spdy too.
(...skipping 96 matching lines...) Expand 10 before | Expand all | Expand 10 after
212 214
213 // Posts a task to notify the factory that this session has been closed. 215 // Posts a task to notify the factory that this session has been closed.
214 void NotifyFactoryOfSessionClosedLater(); 216 void NotifyFactoryOfSessionClosedLater();
215 217
216 // Notifies the factory that this session has been closed which will 218 // Notifies the factory that this session has been closed which will
217 // delete |this|. 219 // delete |this|.
218 void NotifyFactoryOfSessionClosed(); 220 void NotifyFactoryOfSessionClosed();
219 221
220 void OnConnectTimeout(); 222 void OnConnectTimeout();
221 223
222 const HostPortPair server_host_port_; 224 scoped_ptr<HostPortPair> server_host_port_;
223 bool require_confirmation_; 225 bool require_confirmation_;
224 scoped_ptr<QuicCryptoClientStream> crypto_stream_; 226 scoped_ptr<QuicCryptoClientStream> crypto_stream_;
225 QuicStreamFactory* stream_factory_; 227 QuicStreamFactory* stream_factory_;
226 scoped_ptr<DatagramClientSocket> socket_; 228 scoped_ptr<DatagramClientSocket> socket_;
227 scoped_ptr<QuicDefaultPacketWriter> writer_; 229 scoped_ptr<QuicDefaultPacketWriter> writer_;
228 scoped_refptr<IOBufferWithSize> read_buffer_; 230 scoped_refptr<IOBufferWithSize> read_buffer_;
229 scoped_ptr<QuicServerInfo> server_info_; 231 scoped_ptr<QuicServerInfo> server_info_;
230 scoped_ptr<CertVerifyResult> cert_verify_result_; 232 scoped_ptr<CertVerifyResult> cert_verify_result_;
231 ObserverSet observers_; 233 ObserverSet observers_;
232 StreamRequestQueue stream_requests_; 234 StreamRequestQueue stream_requests_;
(...skipping 10 matching lines...) Expand all
243 // on this session. Existing stream will continue to be processed. 245 // on this session. Existing stream will continue to be processed.
244 bool going_away_; 246 bool going_away_;
245 base::WeakPtrFactory<QuicClientSession> weak_factory_; 247 base::WeakPtrFactory<QuicClientSession> weak_factory_;
246 248
247 DISALLOW_COPY_AND_ASSIGN(QuicClientSession); 249 DISALLOW_COPY_AND_ASSIGN(QuicClientSession);
248 }; 250 };
249 251
250 } // namespace net 252 } // namespace net
251 253
252 #endif // NET_QUIC_QUIC_CLIENT_SESSION_H_ 254 #endif // NET_QUIC_QUIC_CLIENT_SESSION_H_
OLDNEW
« no previous file with comments | « no previous file | net/quic/quic_client_session.cc » ('j') | net/quic/quic_client_session.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698