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

Side by Side Diff: net/http/http_stream_factory_impl_job.h

Issue 18546008: [SPDY] Use WeakPtr<SpdySession> everywhere but SpdySessionPool (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix test, other minor formatting/comment changes Created 7 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 | « net/http/http_stream_factory_impl.cc ('k') | net/http/http_stream_factory_impl_job.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_HTTP_HTTP_STREAM_FACTORY_IMPL_JOB_H_ 5 #ifndef NET_HTTP_HTTP_STREAM_FACTORY_IMPL_JOB_H_
6 #define NET_HTTP_HTTP_STREAM_FACTORY_IMPL_JOB_H_ 6 #define NET_HTTP_HTTP_STREAM_FACTORY_IMPL_JOB_H_
7 7
8 #include "base/memory/ref_counted.h" 8 #include "base/memory/ref_counted.h"
9 #include "base/memory/scoped_ptr.h" 9 #include "base/memory/scoped_ptr.h"
10 #include "base/memory/weak_ptr.h" 10 #include "base/memory/weak_ptr.h"
(...skipping 289 matching lines...) Expand 10 before | Expand all | Expand 10 after
300 bool was_npn_negotiated_; 300 bool was_npn_negotiated_;
301 301
302 // Protocol negotiated with the server. 302 // Protocol negotiated with the server.
303 NextProto protocol_negotiated_; 303 NextProto protocol_negotiated_;
304 304
305 // 0 if we're not preconnecting. Otherwise, the number of streams to 305 // 0 if we're not preconnecting. Otherwise, the number of streams to
306 // preconnect. 306 // preconnect.
307 int num_streams_; 307 int num_streams_;
308 308
309 // Initialized when we create a new SpdySession. 309 // Initialized when we create a new SpdySession.
310 scoped_refptr<SpdySession> new_spdy_session_; 310 base::WeakPtr<SpdySession> new_spdy_session_;
311 311
312 // Initialized when we have an existing SpdySession. 312 // Initialized when we have an existing SpdySession.
313 scoped_refptr<SpdySession> existing_spdy_session_; 313 base::WeakPtr<SpdySession> existing_spdy_session_;
314 314
315 // Only used if |new_spdy_session_| is non-NULL. 315 // Only used if |new_spdy_session_| is non-NULL.
316 bool spdy_session_direct_; 316 bool spdy_session_direct_;
317 317
318 // Key used to identify the HttpPipelinedHost for |request_|. 318 // Key used to identify the HttpPipelinedHost for |request_|.
319 scoped_ptr<HttpPipelinedHost::Key> http_pipelining_key_; 319 scoped_ptr<HttpPipelinedHost::Key> http_pipelining_key_;
320 320
321 // True if an existing pipeline can handle this job's request. 321 // True if an existing pipeline can handle this job's request.
322 bool existing_available_pipeline_; 322 bool existing_available_pipeline_;
323 323
324 base::WeakPtrFactory<Job> ptr_factory_; 324 base::WeakPtrFactory<Job> ptr_factory_;
325 325
326 DISALLOW_COPY_AND_ASSIGN(Job); 326 DISALLOW_COPY_AND_ASSIGN(Job);
327 }; 327 };
328 328
329 } // namespace net 329 } // namespace net
330 330
331 #endif // NET_HTTP_HTTP_STREAM_FACTORY_IMPL_JOB_H_ 331 #endif // NET_HTTP_HTTP_STREAM_FACTORY_IMPL_JOB_H_
OLDNEW
« no previous file with comments | « net/http/http_stream_factory_impl.cc ('k') | net/http/http_stream_factory_impl_job.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698