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

Unified Diff: chrome/browser/chromeos/policy/policy_cert_verifier.h

Issue 24153012: Fix cyclic dependency between ProfilePolicyConnector and PrefService. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix/Extend NetworkConfigurationUpdater unit test. Created 7 years, 1 month 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: chrome/browser/chromeos/policy/policy_cert_verifier.h
diff --git a/chrome/browser/chromeos/policy/policy_cert_verifier.h b/chrome/browser/chromeos/policy/policy_cert_verifier.h
index c671161b3cc2661c04c787693bc709602bd3c7a2..9d11fec8e513b3c5ed232230c3197b4349625daa 100644
--- a/chrome/browser/chromeos/policy/policy_cert_verifier.h
+++ b/chrome/browser/chromeos/policy/policy_cert_verifier.h
@@ -12,10 +12,12 @@
#include "base/compiler_specific.h"
#include "base/memory/ref_counted.h"
#include "base/memory/scoped_ptr.h"
+#include "net/base/completion_callback.h"
#include "net/cert/cert_trust_anchor_provider.h"
#include "net/cert/cert_verifier.h"
namespace net {
+class CertVerifyResult;
class X509Certificate;
typedef std::vector<scoped_refptr<X509Certificate> > CertificateList;
}
@@ -27,15 +29,18 @@ namespace policy {
class PolicyCertVerifier : public net::CertVerifier,
public net::CertTrustAnchorProvider {
public:
- // This object must be created on the UI thread. It's member functions and
- // destructor must be called on the IO thread. |anchor_used_callback| is
- // called on the IO thread everytime a certificate from the additional trust
- // anchors (set with SetTrustAnchors) is used.
+ // Except for tests, PolicyCertVerifier should only be created by
+ // NetworkPolicyService, which is the counterpart of this class on the UI
+ // thread. Except of the constructor, all methods and the destructor must be
+ // called on the IO thread. Calls |anchor_used_callback| on the IO thread
+ // everytime a certificate from the additional trust anchors (set with
+ // SetTrustAnchors) is used.
explicit PolicyCertVerifier(const base::Closure& anchor_used_callback);
virtual ~PolicyCertVerifier();
void InitializeOnIOThread();
+ // Sets the additional trust anchors.
void SetTrustAnchors(const net::CertificateList& trust_anchors);
// CertVerifier:

Powered by Google App Engine
This is Rietveld 408576698