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

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

Issue 2799113003: Remove |auth_controllers_| a dead field in HttpStreamFactoryImpl::Job (Closed)
Patch Set: rebased Created 3 years, 8 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
« no previous file with comments | « no previous file | no next file » | 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 <memory> 8 #include <memory>
9 #include <utility> 9 #include <utility>
10 10
(...skipping 431 matching lines...) Expand 10 before | Expand all | Expand 10 after
442 // True if this network transaction is using SPDY instead of HTTP. 442 // True if this network transaction is using SPDY instead of HTTP.
443 bool using_spdy_; 443 bool using_spdy_;
444 444
445 // True if this network transaction is using QUIC instead of HTTP. 445 // True if this network transaction is using QUIC instead of HTTP.
446 bool using_quic_; 446 bool using_quic_;
447 QuicStreamRequest quic_request_; 447 QuicStreamRequest quic_request_;
448 448
449 // True if this job used an existing QUIC session. 449 // True if this job used an existing QUIC session.
450 bool using_existing_quic_session_; 450 bool using_existing_quic_session_;
451 451
452 scoped_refptr<HttpAuthController>
453 auth_controllers_[HttpAuth::AUTH_NUM_TARGETS];
454
455 // True when the tunnel is in the process of being established - we can't 452 // True when the tunnel is in the process of being established - we can't
456 // read from the socket until the tunnel is done. 453 // read from the socket until the tunnel is done.
457 bool establishing_tunnel_; 454 bool establishing_tunnel_;
458 455
459 std::unique_ptr<HttpStream> stream_; 456 std::unique_ptr<HttpStream> stream_;
460 std::unique_ptr<WebSocketHandshakeStreamBase> websocket_stream_; 457 std::unique_ptr<WebSocketHandshakeStreamBase> websocket_stream_;
461 std::unique_ptr<BidirectionalStreamImpl> bidirectional_stream_impl_; 458 std::unique_ptr<BidirectionalStreamImpl> bidirectional_stream_impl_;
462 459
463 // True if we negotiated ALPN. 460 // True if we negotiated ALPN.
464 bool was_alpn_negotiated_; 461 bool was_alpn_negotiated_;
(...skipping 70 matching lines...) Expand 10 before | Expand all | Expand 10 after
535 const SSLConfig& proxy_ssl_config, 532 const SSLConfig& proxy_ssl_config,
536 HostPortPair destination, 533 HostPortPair destination,
537 GURL origin_url, 534 GURL origin_url,
538 bool enable_ip_based_pooling, 535 bool enable_ip_based_pooling,
539 NetLog* net_log) = 0; 536 NetLog* net_log) = 0;
540 }; 537 };
541 538
542 } // namespace net 539 } // namespace net
543 540
544 #endif // NET_HTTP_HTTP_STREAM_FACTORY_IMPL_JOB_H_ 541 #endif // NET_HTTP_HTTP_STREAM_FACTORY_IMPL_JOB_H_
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698