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

Unified Diff: net/socket/ssl_client_socket_pool.h

Issue 1580903002: Convert ignore_limits from a SocketParam to a socket request argument. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Missed one Created 4 years, 11 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 side-by-side diff with in-line comments
Download patch
Index: net/socket/ssl_client_socket_pool.h
diff --git a/net/socket/ssl_client_socket_pool.h b/net/socket/ssl_client_socket_pool.h
index b015baeb7976ca3d6baae20ec824b69c2e104922..a5cead55797477f374766d472a967770cad1cba0 100644
--- a/net/socket/ssl_client_socket_pool.h
+++ b/net/socket/ssl_client_socket_pool.h
@@ -72,7 +72,6 @@ class NET_EXPORT_PRIVATE SSLSocketParams
PrivacyMode privacy_mode() const { return privacy_mode_; }
int load_flags() const { return load_flags_; }
bool expect_spdy() const { return expect_spdy_; }
- bool ignore_limits() const { return ignore_limits_; }
private:
friend class base::RefCounted<SSLSocketParams>;
@@ -86,7 +85,6 @@ class NET_EXPORT_PRIVATE SSLSocketParams
const PrivacyMode privacy_mode_;
const int load_flags_;
const bool expect_spdy_;
- bool ignore_limits_;
DISALLOW_COPY_AND_ASSIGN(SSLSocketParams);
};
@@ -99,6 +97,7 @@ class SSLConnectJob : public ConnectJob {
// job.
SSLConnectJob(const std::string& group_name,
RequestPriority priority,
+ ClientSocketPool::IgnoreLimits ignore_limits,
const scoped_refptr<SSLSocketParams>& params,
const base::TimeDelta& timeout_duration,
TransportClientSocketPool* transport_pool,
@@ -205,6 +204,7 @@ class NET_EXPORT_PRIVATE SSLClientSocketPool
int RequestSocket(const std::string& group_name,
const void* connect_params,
RequestPriority priority,
+ IgnoreLimits ignore_limits,
ClientSocketHandle* handle,
const CompletionCallback& callback,
const BoundNetLog& net_log) override;

Powered by Google App Engine
This is Rietveld 408576698