Make ServerBoundCertStore interface async, move SQLiteServerBoundCertStore load onto DB thread.

net/base/server_bound_cert_store.h

 // Copyright (c) 2012 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 NET_BASE_SERVER_BOUND_CERT_STORE_H_
 #define NET_BASE_SERVER_BOUND_CERT_STORE_H_
 
 #include <list>
 #include <string>
 
+#include "base/callback.h"
 #include "base/threading/non_thread_safe.h"
 #include "base/time.h"
 #include "net/base/net_export.h"
 #include "net/base/ssl_client_cert_type.h"
 
 namespace net {
 
 // An interface for storing and retrieving server bound certs.
 // There isn't a domain bound certs spec yet, but the old origin bound
 // certificates are specified in
@@ skipping 37 matching lines @@
     std::string server_identifier_;
     SSLClientCertType type_;
     base::Time creation_time_;
     base::Time expiration_time_;
     std::string private_key_;
     std::string cert_;
   };
 
   typedef std::list<ServerBoundCert> ServerBoundCertList;
 
+  typedef base::Callback<void(
+      const std::string&,
+      SSLClientCertType,
+      base::Time,
+      const std::string&,
+      const std::string&)> GetCertCallback;
+  typedef base::Callback<void(const ServerBoundCertList&)> GetCertListCallback;
+
   virtual ~ServerBoundCertStore() {}
 
-  // TODO(rkn): File I/O may be required, so this should have an asynchronous
-  // interface.
-  // Returns true on success. |private_key_result| stores a DER-encoded
-  // PrivateKeyInfo struct, |cert_result| stores a DER-encoded certificate,
-  // |type| is the ClientCertificateType of the returned certificate,
-  // |creation_time| stores the start of the validity period of the certificate
-  // and |expiration_time| is the expiration time of the certificate.
-  // Returns false if no server bound cert exists for the specified server.
+  // GetServerBoundCert may return the result synchronously through the
+  // output parameters, in which case it will return true.  Otherwise it will
+  // return false and the callback will be called with the result
+  // asynchronously.
+  // In either case, the type will be CLIENT_CERT_INVALID_TYPE if no cert
+  // existed for the given |server_identifier|.
   virtual bool GetServerBoundCert(
       const std::string& server_identifier,
       SSLClientCertType* type,
-      base::Time* creation_time,
       base::Time* expiration_time,
       std::string* private_key_result,
-      std::string* cert_result) = 0;
+      std::string* cert_result,
+      const GetCertCallback& callback) = 0;
 
   // Adds a server bound cert and the corresponding private key to the store.
   virtual void SetServerBoundCert(
       const std::string& server_identifier,
       SSLClientCertType type,
       base::Time creation_time,
       base::Time expiration_time,
       const std::string& private_key,
       const std::string& cert) = 0;
 
   // Removes a server bound cert and the corresponding private key from the
   // store.
-  virtual void DeleteServerBoundCert(const std::string& server_identifier) = 0;
+  virtual void DeleteServerBoundCert(
+      const std::string& server_identifier,
+      const base::Closure& completion_callback) = 0;
 
   // Deletes all of the server bound certs that have a creation_date greater
   // than or equal to |delete_begin| and less than |delete_end|.  If a
   // base::Time value is_null, that side of the comparison is unbounded.
-  virtual void DeleteAllCreatedBetween(base::Time delete_begin,
-                                       base::Time delete_end) = 0;
+  virtual void DeleteAllCreatedBetween(
+      base::Time delete_begin,
+      base::Time delete_end,
+      const base::Closure& completion_callback) = 0;
 
   // Removes all server bound certs and the corresponding private keys from
   // the store.
-  virtual void DeleteAll() = 0;
+  virtual void DeleteAll(const base::Closure& completion_callback) = 0;
 
   // Returns all server bound certs and the corresponding private keys.
-  virtual void GetAllServerBoundCerts(
-      ServerBoundCertList* server_bound_certs) = 0;
+  virtual void GetAllServerBoundCerts(const GetCertListCallback& callback) = 0;
 
   // Helper function that adds all certs from |list| into this instance.
   void InitializeFrom(const ServerBoundCertList& list);
 
-  // Returns the number of certs in the store.
+  // Returns the number of certs in the store.  May return 0 if the backing
+  // store is not loaded yet.
   // Public only for unit testing.
   virtual int GetCertCount() = 0;
 
   // When invoked, instructs the store to keep session related data on
   // destruction.
   virtual void SetForceKeepSessionState() = 0;
 };
 
 }  // namespace net
 
 #endif  // NET_BASE_SERVER_BOUND_CERT_STORE_H_