summaryrefslogtreecommitdiffstats
path: root/chrome/browser/cert_store.h
diff options
context:
space:
mode:
authorwtc@google.com <wtc@google.com@0039d316-1c4b-4281-b951-d872f2087c98>2008-09-29 17:42:29 +0000
committerwtc@google.com <wtc@google.com@0039d316-1c4b-4281-b951-d872f2087c98>2008-09-29 17:42:29 +0000
commitefb8bc81fd2d43607bf50cfa4cc12317d1a4c986 (patch)
treebae56fd6be197315f7335f06c24973e5ee9635a3 /chrome/browser/cert_store.h
parent4d2b21a7f31a3eb41d5d5d6292ca2c7bc75f8ace (diff)
downloadchromium_src-efb8bc81fd2d43607bf50cfa4cc12317d1a4c986.zip
chromium_src-efb8bc81fd2d43607bf50cfa4cc12317d1a4c986.tar.gz
chromium_src-efb8bc81fd2d43607bf50cfa4cc12317d1a4c986.tar.bz2
Don't use cert_lock_ as a reentrant lock. RemoveCert is
renamed RemoveCertInternal to stress the fact that it should be called with the lock held. Fix miscellaneous nits complained by cpplint. R=jcampan Review URL: http://codereview.chromium.org/5024 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@2677 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/cert_store.h')
-rw-r--r--chrome/browser/cert_store.h3
1 files changed, 2 insertions, 1 deletions
diff --git a/chrome/browser/cert_store.h b/chrome/browser/cert_store.h
index ecfab73..742151a 100644
--- a/chrome/browser/cert_store.h
+++ b/chrome/browser/cert_store.h
@@ -52,7 +52,8 @@ class CertStore : public NotificationObserver {
~CertStore();
// Remove the specified cert from id_to_cert_ and cert_to_id_.
- void RemoveCert(int cert_id);
+ // NOTE: the caller (RemoveCertsForRenderProcesHost) must hold cert_lock_.
+ void RemoveCertInternal(int cert_id);
// Removes all the certs associated with the specified process from the store.
void RemoveCertsForRenderProcesHost(int render_process_host_id);