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

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

Issue 2964603002: JobController: do not create alternative job for those AlternativeServiceInfo (Closed)
Patch Set: Always initialize local variable to fix windows compile issue Created 3 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
OLDNEW
1 // Copyright (c) 2016 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2016 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_CONTROLLER_H_ 5 #ifndef NET_HTTP_HTTP_STREAM_FACTORY_IMPL_JOB_CONTROLLER_H_
6 #define NET_HTTP_HTTP_STREAM_FACTORY_IMPL_JOB_CONTROLLER_H_ 6 #define NET_HTTP_HTTP_STREAM_FACTORY_IMPL_JOB_CONTROLLER_H_
7 7
8 #include <memory> 8 #include <memory>
9 9
10 #include "net/base/host_port_pair.h" 10 #include "net/base/host_port_pair.h"
(...skipping 259 matching lines...) Expand 10 before | Expand all | Expand 10 after
270 AlternativeServiceInfo GetAlternativeServiceInfoFor( 270 AlternativeServiceInfo GetAlternativeServiceInfoFor(
271 const HttpRequestInfo& request_info, 271 const HttpRequestInfo& request_info,
272 HttpStreamRequest::Delegate* delegate, 272 HttpStreamRequest::Delegate* delegate,
273 HttpStreamRequest::StreamType stream_type); 273 HttpStreamRequest::StreamType stream_type);
274 274
275 AlternativeServiceInfo GetAlternativeServiceInfoInternal( 275 AlternativeServiceInfo GetAlternativeServiceInfoInternal(
276 const HttpRequestInfo& request_info, 276 const HttpRequestInfo& request_info,
277 HttpStreamRequest::Delegate* delegate, 277 HttpStreamRequest::Delegate* delegate,
278 HttpStreamRequest::StreamType stream_type); 278 HttpStreamRequest::StreamType stream_type);
279 279
280 // Returns a QuicVersion that has been advertised in |advertised_versions|
281 // and is supported. If more than one QuicVersions are supported, the first
282 // matched in the supported versions will be returned. If no mutually
283 // supported version is found, QUIC_VERSION_UNSUPPORTED_VERSION will be
284 // returned.
285 QuicVersion SelectQuicVersion(const QuicVersionVector& advertised_versions);
286
280 // Remove session from the SpdySessionRequestMap. 287 // Remove session from the SpdySessionRequestMap.
281 void RemoveRequestFromSpdySessionRequestMap(); 288 void RemoveRequestFromSpdySessionRequestMap();
282 289
283 // Returns true if the |request_| can be fetched via an alternative 290 // Returns true if the |request_| can be fetched via an alternative
284 // proxy server, and sets |alternative_proxy_server| to the available 291 // proxy server, and sets |alternative_proxy_server| to the available
285 // alternative proxy server. |alternative_proxy_server| should not be null, 292 // alternative proxy server. |alternative_proxy_server| should not be null,
286 // and is owned by the caller. 293 // and is owned by the caller.
287 bool ShouldCreateAlternativeProxyServerJob( 294 bool ShouldCreateAlternativeProxyServerJob(
288 const ProxyInfo& proxy_info_, 295 const ProxyInfo& proxy_info_,
289 const GURL& url, 296 const GURL& url,
(...skipping 79 matching lines...) Expand 10 before | Expand all | Expand 10 after
369 HttpStreamRequest::StreamType stream_type_; 376 HttpStreamRequest::StreamType stream_type_;
370 RequestPriority priority_; 377 RequestPriority priority_;
371 const NetLogWithSource net_log_; 378 const NetLogWithSource net_log_;
372 379
373 base::WeakPtrFactory<JobController> ptr_factory_; 380 base::WeakPtrFactory<JobController> ptr_factory_;
374 }; 381 };
375 382
376 } // namespace net 383 } // namespace net
377 384
378 #endif // NET_HTTP_HTTP_STREAM_FACTORY_IMPL_JOB_CONTROLLER_H_ 385 #endif // NET_HTTP_HTTP_STREAM_FACTORY_IMPL_JOB_CONTROLLER_H_
OLDNEW
« no previous file with comments | « components/grpc_support/test/get_stream_engine.cc ('k') | net/http/http_stream_factory_impl_job_controller.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698