diff options
author | wtc@google.com <wtc@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2008-09-29 17:42:29 +0000 |
---|---|---|
committer | wtc@google.com <wtc@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2008-09-29 17:42:29 +0000 |
commit | efb8bc81fd2d43607bf50cfa4cc12317d1a4c986 (patch) | |
tree | bae56fd6be197315f7335f06c24973e5ee9635a3 /chrome/browser/cert_store.h | |
parent | 4d2b21a7f31a3eb41d5d5d6292ca2c7bc75f8ace (diff) | |
download | chromium_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.h | 3 |
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); |