Fix cyclic dependency between ProfilePolicyConnector and PrefService.

chrome/browser/chromeos/policy/policy_cert_verifier.h

 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_CHROMEOS_POLICY_POLICY_CERT_VERIFIER_H_
 #define CHROME_BROWSER_CHROMEOS_POLICY_POLICY_CERT_VERIFIER_H_
 
 #include <vector>
 
 #include "base/basictypes.h"
-#include "base/callback.h"
+#include "base/callback_list.h"
 #include "base/compiler_specific.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
+#include "base/memory/weak_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;
 }
 
 namespace policy {
 
 // Wraps a MultiThreadedCertVerifier to make it use the additional trust anchors
 // configured by the ONC user 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.
-  explicit PolicyCertVerifier(const base::Closure& anchor_used_callback);
+  PolicyCertVerifier();
   virtual ~PolicyCertVerifier();
 
   void InitializeOnIOThread();
+  void ShutdownOnUIThread();
+
+  scoped_ptr<base::CallbackList<void()>::Subscription>
+      RegisterAnchorUsedCallback(const base::Closure& cb);
 
   void SetTrustAnchors(const net::CertificateList& trust_anchors);
 
   // CertVerifier:
   // Note: |callback| can be null.
   virtual int Verify(net::X509Certificate* cert,
                      const std::string& hostname,
                      int flags,
                      net::CRLSet* crl_set,
                      net::CertVerifyResult* verify_result,
                      const net::CompletionCallback& callback,
                      RequestHandle* out_req,
                      const net::BoundNetLog& net_log) OVERRIDE;
 
   virtual void CancelRequest(RequestHandle req) OVERRIDE;
 
   // CertTrustAnchorProvider:
   virtual const net::CertificateList& GetAdditionalTrustAnchors() OVERRIDE;
 
  private:
+  void RunCallback();
+
+  static void MaybeSignalAnchorUse(
+      int error,
+      const net::CertVerifyResult& verify_result,
+      const base::WeakPtr<PolicyCertVerifier>& weak_ptr);
+
+  static void CompleteAndSignalAnchorUse(
+      const net::CompletionCallback& completion_callback,
+      const net::CertVerifyResult* verify_result,
+      const base::WeakPtr<PolicyCertVerifier>& weak_ptr,
+      int error);
+
   net::CertificateList trust_anchors_;
-  base::Closure anchor_used_callback_;
   scoped_ptr<CertVerifier> delegate_;
 
+  // Must be constructed, modified, and destructed on the UI thread.
+  scoped_ptr<base::CallbackList<void()> > anchor_used_callback_list_;
+
+  // Must be constructed, modified, and invalidated on the UI thread.
+  base::WeakPtrFactory<PolicyCertVerifier> weak_ptr_factory_;
+
+  // Must be constructed and dereferenced on the UI thread. This WeakPtr has to
+  // be used for callbacks from the IO thread to the UI thread.
+  base::WeakPtr<PolicyCertVerifier> weak_ptr_;
+
   DISALLOW_COPY_AND_ASSIGN(PolicyCertVerifier);
 };
 
 }  // namespace policy
 
 #endif  // CHROME_BROWSER_CHROMEOS_POLICY_POLICY_CERT_VERIFIER_H_