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

Side by Side Diff: net/http/http_stream_factory_impl.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_proxy_client_socket_pool.cc ('k') | net/http/http_stream_factory_impl.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_H_ 5 #ifndef NET_HTTP_HTTP_STREAM_FACTORY_IMPL_H_
6 #define NET_HTTP_HTTP_STREAM_FACTORY_IMPL_H_ 6 #define NET_HTTP_HTTP_STREAM_FACTORY_IMPL_H_
7 7
8 #include <map> 8 #include <map>
9 #include <set> 9 #include <set>
10 #include <vector> 10 #include <vector>
(...skipping 76 matching lines...) Expand 10 before | Expand all | Expand 10 after
87 PortAlternateProtocolPair GetAlternateProtocolRequestFor( 87 PortAlternateProtocolPair GetAlternateProtocolRequestFor(
88 const GURL& original_url, 88 const GURL& original_url,
89 GURL* alternate_url) const; 89 GURL* alternate_url) const;
90 90
91 // Detaches |job| from |request|. 91 // Detaches |job| from |request|.
92 void OrphanJob(Job* job, const Request* request); 92 void OrphanJob(Job* job, const Request* request);
93 93
94 // Called when a SpdySession is ready. It will find appropriate Requests and 94 // Called when a SpdySession is ready. It will find appropriate Requests and
95 // fulfill them. |direct| indicates whether or not |spdy_session| uses a 95 // fulfill them. |direct| indicates whether or not |spdy_session| uses a
96 // proxy. 96 // proxy.
97 void OnNewSpdySessionReady(scoped_refptr<SpdySession> spdy_session, 97 void OnNewSpdySessionReady(const base::WeakPtr<SpdySession>& spdy_session,
98 bool direct, 98 bool direct,
99 const SSLConfig& used_ssl_config, 99 const SSLConfig& used_ssl_config,
100 const ProxyInfo& used_proxy_info, 100 const ProxyInfo& used_proxy_info,
101 bool was_npn_negotiated, 101 bool was_npn_negotiated,
102 NextProto protocol_negotiated, 102 NextProto protocol_negotiated,
103 bool using_spdy, 103 bool using_spdy,
104 const BoundNetLog& net_log); 104 const BoundNetLog& net_log);
105 105
106 // Called when the Job detects that the endpoint indicated by the 106 // Called when the Job detects that the endpoint indicated by the
107 // Alternate-Protocol does not work. Lets the factory update 107 // Alternate-Protocol does not work. Lets the factory update
(...skipping 38 matching lines...) Expand 10 before | Expand all | Expand 10 after
146 // deleted when the factory is destroyed. 146 // deleted when the factory is destroyed.
147 std::set<const Job*> preconnect_job_set_; 147 std::set<const Job*> preconnect_job_set_;
148 148
149 const bool for_websockets_; 149 const bool for_websockets_;
150 DISALLOW_COPY_AND_ASSIGN(HttpStreamFactoryImpl); 150 DISALLOW_COPY_AND_ASSIGN(HttpStreamFactoryImpl);
151 }; 151 };
152 152
153 } // namespace net 153 } // namespace net
154 154
155 #endif // NET_HTTP_HTTP_STREAM_FACTORY_IMPL_H_ 155 #endif // NET_HTTP_HTTP_STREAM_FACTORY_IMPL_H_
OLDNEW
« no previous file with comments | « net/http/http_proxy_client_socket_pool.cc ('k') | net/http/http_stream_factory_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698