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

Unified Diff: net/socket/client_socket_pool_manager_impl.cc

Issue 10916094: Move the NSS functions out of CertDatabase into a new NSSCertDatabase class. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Addressed comments Created 8 years, 3 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/client_socket_pool_manager_impl.cc
diff --git a/net/socket/client_socket_pool_manager_impl.cc b/net/socket/client_socket_pool_manager_impl.cc
index b67ad0d1ee13999ba24507ab3912c1bf3a606c63..d70827efab37107794618f31acb873328d25cec8 100644
--- a/net/socket/client_socket_pool_manager_impl.cc
+++ b/net/socket/client_socket_pool_manager_impl.cc
@@ -83,11 +83,13 @@ ClientSocketPoolManagerImpl::ClientSocketPoolManagerImpl(
ssl_for_https_proxy_pool_histograms_("SSLforHTTPSProxy"),
http_proxy_pool_histograms_("HTTPProxy"),
ssl_socket_pool_for_proxies_histograms_("SSLForProxies") {
- CertDatabase::AddObserver(this);
+ CertDatabase cert_db;
+ cert_db.AddObserver(this);
}
ClientSocketPoolManagerImpl::~ClientSocketPoolManagerImpl() {
- CertDatabase::RemoveObserver(this);
+ CertDatabase cert_db;
+ cert_db.RemoveObserver(this);
}
void ClientSocketPoolManagerImpl::FlushSocketPools() {
@@ -371,7 +373,7 @@ Value* ClientSocketPoolManagerImpl::SocketPoolInfoToValue() const {
return list;
}
-void ClientSocketPoolManagerImpl::OnUserCertAdded(const X509Certificate* cert) {
+void ClientSocketPoolManagerImpl::OnCertAdded(const X509Certificate* cert) {
FlushSocketPools();
}

Powered by Google App Engine
This is Rietveld 408576698