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

Side by Side Diff: net/http/http_proxy_client_socket_pool_unittest.cc

Issue 353713005: Implements new, more robust design for communicating between SSLConnectJobs. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Updated SSLClientSocket tests & fixed bug in SSLSessionCacheOpenSSL Created 6 years, 4 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) 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 #include "net/http/http_proxy_client_socket_pool.h" 5 #include "net/http/http_proxy_client_socket_pool.h"
6 6
7 #include "base/callback.h" 7 #include "base/callback.h"
8 #include "base/compiler_specific.h" 8 #include "base/compiler_specific.h"
9 #include "base/strings/string_util.h" 9 #include "base/strings/string_util.h"
10 #include "base/strings/utf_string_conversions.h" 10 #include "base/strings/utf_string_conversions.h"
(...skipping 66 matching lines...) Expand 10 before | Expand all | Expand 10 after
77 session_deps_.cert_verifier.get(), 77 session_deps_.cert_verifier.get(),
78 NULL /* server_bound_cert_store */, 78 NULL /* server_bound_cert_store */,
79 NULL /* transport_security_state */, 79 NULL /* transport_security_state */,
80 NULL /* cert_transparency_verifier */, 80 NULL /* cert_transparency_verifier */,
81 std::string() /* ssl_session_cache_shard */, 81 std::string() /* ssl_session_cache_shard */,
82 session_deps_.deterministic_socket_factory.get(), 82 session_deps_.deterministic_socket_factory.get(),
83 &transport_socket_pool_, 83 &transport_socket_pool_,
84 NULL, 84 NULL,
85 NULL, 85 NULL,
86 session_deps_.ssl_config_service.get(), 86 session_deps_.ssl_config_service.get(),
87 false,
87 BoundNetLog().net_log()), 88 BoundNetLog().net_log()),
88 session_(CreateNetworkSession()), 89 session_(CreateNetworkSession()),
89 http_proxy_histograms_("HttpProxyUnitTest"), 90 http_proxy_histograms_("HttpProxyUnitTest"),
90 spdy_util_(GetParam().protocol), 91 spdy_util_(GetParam().protocol),
91 pool_(kMaxSockets, 92 pool_(kMaxSockets,
92 kMaxSocketsPerGroup, 93 kMaxSocketsPerGroup,
93 &http_proxy_histograms_, 94 &http_proxy_histograms_,
94 NULL, 95 NULL,
95 &transport_socket_pool_, 96 &transport_socket_pool_,
96 &ssl_socket_pool_, 97 &ssl_socket_pool_,
(...skipping 625 matching lines...) Expand 10 before | Expand all | Expand 10 after
722 EXPECT_TRUE(headers->IsRedirect(&location)); 723 EXPECT_TRUE(headers->IsRedirect(&location));
723 EXPECT_EQ(location, redirectTarget); 724 EXPECT_EQ(location, redirectTarget);
724 } 725 }
725 } 726 }
726 727
727 // It would be nice to also test the timeouts in HttpProxyClientSocketPool. 728 // It would be nice to also test the timeouts in HttpProxyClientSocketPool.
728 729
729 } // namespace 730 } // namespace
730 731
731 } // namespace net 732 } // namespace net
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698