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

Unified Diff: net/socket/client_socket_factory.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_factory.cc
diff --git a/net/socket/client_socket_factory.cc b/net/socket/client_socket_factory.cc
index f35431768d19e3e334b03334b477016e3c965cc4..2d433b09037732a1e772438ea37784596af776ed 100644
--- a/net/socket/client_socket_factory.cc
+++ b/net/socket/client_socket_factory.cc
@@ -55,16 +55,18 @@ class DefaultClientSocketFactory : public ClientSocketFactory,
base::SequencedWorkerPool::CONTINUE_ON_SHUTDOWN);
}
- CertDatabase::AddObserver(this);
+ CertDatabase cert_db;
+ cert_db.AddObserver(this);
}
virtual ~DefaultClientSocketFactory() {
// Note: This code never runs, as the factory is defined as a Leaky
// singleton.
- CertDatabase::RemoveObserver(this);
+ CertDatabase cert_db;
+ cert_db.RemoveObserver(this);
}
- virtual void OnUserCertAdded(const X509Certificate* cert) OVERRIDE {
+ virtual void OnCertAdded(const X509Certificate* cert) OVERRIDE {
ClearSSLSessionCache();
}

Powered by Google App Engine
This is Rietveld 408576698