Implement chrome.platformKeys.getKeyPair().

chrome/browser/chromeos/platform_keys/platform_keys.h

 // Copyright 2014 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_PLATFORM_KEYS_PLATFORM_KEYS_H_
 #define CHROME_BROWSER_CHROMEOS_PLATFORM_KEYS_PLATFORM_KEYS_H_
 
 #include <string>
 #include <vector>
 
 #include "base/callback_forward.h"
 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
+#include "net/cert/x509_certificate.h"
 #include "net/ssl/ssl_client_cert_type.h"
 
 namespace content {
 class BrowserContext;
 }
 
-namespace net {
-class X509Certificate;
-typedef std::vector<scoped_refptr<X509Certificate> > CertificateList;
-}
-
 namespace chromeos {
 
 namespace platform_keys {
 
 // A token is a store for keys or certs and can provide cryptographic
 // operations.
 // ChromeOS provides itself a user token and conditionally a system wide token,
 // thus these tokens use static identifiers. The platform keys API is designed
 // to support arbitrary other tokens in the future, which could then use
 // run-time generated IDs.
 extern const char kTokenIdUser[];
 extern const char kTokenIdSystem[];
 
 // Supported hash algorithms.
 enum HashAlgorithm {
+  HASH_ALGORITHM_NONE,  // The value if no hash function is selected.
   HASH_ALGORITHM_SHA1,
   HASH_ALGORITHM_SHA256,
   HASH_ALGORITHM_SHA384,
   HASH_ALGORITHM_SHA512
 };
 
 struct ClientCertificateRequest {
   ClientCertificateRequest();
   ~ClientCertificateRequest();
 
@@ skipping 19 matching lines @@
 // ignored, instead the user token associated with |browser_context| is always
 // used. |callback| will be invoked with the resulting public key or an error.
 void GenerateRSAKey(const std::string& token_id,
                     unsigned int modulus_length_bits,
                     const GenerateKeyCallback& callback,
                     content::BrowserContext* browser_context);
 
 typedef base::Callback<void(const std::string& signature,
                             const std::string& error_message)> SignCallback;
 
-// Digests |data| with |hash_algorithm| and afterwards signs the digest with the
-// private key matching |public_key|, if that key is stored in the given token.
-// |token_id| is currently ignored, instead the user token associated with
-// |browser_context| is always used. |public_key| must be the DER encoding of a
-// SubjectPublicKeyInfo. |callback| will be invoked with the signature or an
-// error message.
-// Currently supports RSA keys only.
-void Sign(const std::string& token_id,
-          const std::string& public_key,
-          HashAlgorithm hash_algorithm,
-          const std::string& data,
-          const SignCallback& callback,
-          content::BrowserContext* browser_context);
+// Digests |data|, applies PKCS1 padding and afterwards signs the data with the
+// private key matching |params.public_key|. If a non empty token id is provided
+// and the key is not found in that token, the operation aborts. |callback| will
+// be invoked with the signature or an error message.
+void SignRSAPKCS1Digest(const std::string& token_id,
+                        const std::string& data,
+                        const std::string& public_key,
+                        HashAlgorithm hash_algorithm,
+                        const SignCallback& callback,
+                        content::BrowserContext* browser_context);
+
+// Applies PKCS1 padding and afterwards signs the data with the private key
+// matching |params.public_key|. |data| is not digested. If a non empty token id
+// is provided and the key is not found in that token, the operation aborts.
+// The size of |data| (number of octets) must be smaller than k - 11, where k
+// is the key size in octets.
+// |callback| will be invoked with the signature or an error message.
+void SignRSAPKCS1Raw(const std::string& token_id,
+                     const std::string& data,
+                     const std::string& public_key,
+                     const SignCallback& callback,
+                     content::BrowserContext* browser_context);
 
 // If the certificate request could be processed successfully, |matches| will
 // contain the list of matching certificates (which may be empty) and
 // |error_message| will be empty. If an error occurred, |matches| will be null
 // and |error_message| contain an error message.
 typedef base::Callback<void(scoped_ptr<net::CertificateList> matches,
                             const std::string& error_message)>
     SelectCertificatesCallback;
 
 // Returns the list of all certificates that match |request|. |callback| will be
 // invoked with these matches or an error message.
 void SelectClientCertificates(const ClientCertificateRequest& request,
                               const SelectCertificatesCallback& callback,
                               content::BrowserContext* browser_context);
 
 }  // namespace subtle
 
+// If possible, fills the output arguments with information about the key
+// certified by |certificate| and returns true.
+// If an error occurs, does not modify the output arguments and returns false.
+// It is handled as an error, if the key is an RSA key with public exponent
+// not equal to 65537.

[Ryan Sleevi, 2015/02/10 21:25:28]

Comment nit: This reads somewhat weird. Perhaps reword

// Obtains information about the public key in |certificate|
// If |certificate| contains an RSA key, sets |key_size_bits| to the modulus
length, |public_key_spki_der|
// to the DER encoding of the X.509 Subject Public Key Info, and |key_type| to
RSA.
// If |certificate| contains any other key type, or if the public exponent of
the RSA key in |certificate| is not F4, returns false and does not update any of
the output parameters.

[pneubeck (no reviews), 2015/02/11 14:37:04]

adapted the implementation to ensure that false is returned on non-RSA keys.

Done.

+// |public_key_spki_der|: Will be assigned the the X.509 Subject Public Key Info
+//   of the key in DER encoding, if not null.
+// |key_type|: Will be assigned the type of the key, if not null.
+// |key_size_bits|: Will the size of the key in bits, if not null.
+bool GetPublicKey(const scoped_refptr<net::X509Certificate>& certificate,
+                  std::string* out_public_key_spki_der,
+                  net::X509Certificate::PublicKeyType* out_key_type,
+                  size_t* out_key_size_bits);

[Ryan Sleevi, 2015/02/10 21:25:28]

naming nit: None of these need to be named |out_| for any stylistic reasons. It
does seem somewhat unnecessary (much like hungarian notation for variable
names), so I'd suggest it's just dropped and they become |public_key_spki_der|,
|key_type|, |key_size_bits|.

The pointer types are already strong indicators that they're out params (whether
pure out or in/out), per the Google style guide, so I feel that information is
already there (and in the comment)

[pneubeck (no reviews), 2015/02/11 14:37:04]

Done.

+
 // If the list of certificates could be successfully retrieved, |certs| will
 // contain the list of available certificates (maybe empty) and |error_message|
 // will be empty. If an error occurred, |certs| will be empty and
 // |error_message| contain an error message.
 typedef base::Callback<void(scoped_ptr<net::CertificateList> certs,
                             const std::string& error_message)>
     GetCertificatesCallback;
 
 // Returns the list of all certificates with stored private key available from
 // the given token. |token_id| is currently ignored, instead the user token
 // associated with |browser_context| is always used. |callback| will be invoked
 // with the list of available certificates or an error message.
 void GetCertificates(const std::string& token_id,
                      const GetCertificatesCallback& callback,
                      content::BrowserContext* browser_context);
 
 // If an error occurred during import, |error_message| will be set to an error
 // message.
 typedef base::Callback<void(const std::string& error_message)>
     ImportCertificateCallback;
 
 // Imports |certificate| to the given token if the certified key is already
 // stored in this token. Any intermediate of |certificate| will be ignored.
 // |token_id| is currently ignored, instead the user token associated with
 // |browser_context| is always used. |callback| will be invoked when the import
 // is finished, possibly with an error message.
 void ImportCertificate(const std::string& token_id,
-                       scoped_refptr<net::X509Certificate> certificate,
+                       const scoped_refptr<net::X509Certificate>& certificate,
                        const ImportCertificateCallback& callback,
                        content::BrowserContext* browser_context);
 
 // If an error occurred during removal, |error_message| will be set to an error
 // message.
 typedef base::Callback<void(const std::string& error_message)>
     RemoveCertificateCallback;
 
 // Removes |certificate| from the given token if present. Any intermediate of
 // |certificate| will be ignored. |token_id| is currently ignored, instead the
 // user token associated with |browser_context| is always used. |callback| will
 // be invoked when the removal is finished, possibly with an error message.
 void RemoveCertificate(const std::string& token_id,
-                       scoped_refptr<net::X509Certificate> certificate,
+                       const scoped_refptr<net::X509Certificate>& certificate,
                        const RemoveCertificateCallback& callback,
                        content::BrowserContext* browser_context);
 
 // If the list of available tokens could be successfully retrieved, |token_ids|
 // will contain the token ids. If an error occurs, |token_ids| will be NULL and
 // |error_message| will be set to an error message.
 typedef base::Callback<void(scoped_ptr<std::vector<std::string> > token_ids,
                             const std::string& error_message)>
     GetTokensCallback;
 
 // Gets the list of available tokens. |callback| will be invoked when the list
 // of available tokens is determined, possibly with an error message.
 // Must be called and calls |callback| on the UI thread.
 void GetTokens(const GetTokensCallback& callback,
                content::BrowserContext* browser_context);
 
 }  // namespace platform_keys
 
 }  // namespace chromeos
 
 #endif  // CHROME_BROWSER_CHROMEOS_PLATFORM_KEYS_PLATFORM_KEYS_H_