Add the Sign-At-Most-Once restriction the enterprise.platformKeys API.

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"
 
 class Profile;
 
 namespace net {
 class X509Certificate;
 typedef std::vector<scoped_refptr<X509Certificate> > CertificateList;
 }
 
 namespace chromeos {
 
 namespace platform_keys {
 
-// If the generation was successful, |public_key_spki_der| will contain the DER
-// encoding of the SubjectPublicKeyInfo of the generated key and |error_message|
-// will be empty. If it failed, |public_key_spki_der| will be empty and
-// |error_message| contain an error message.
+namespace subtle {

[pneubeck (no reviews), 2014/06/10 17:50:11]

I could move these functions to another file.
But to simplify the review, I put them into a namespace at first.

+// Functions of this namespace shouldn't be called directly from the context of
+// an extension. Instead use PlatfromKeysService which enforces restrictions

[eroman, 2014/06/12 05:50:45]

typeo Platfrom -> Platform

[pneubeck (no reviews), 2014/06/12 09:21:36]

Done.

+// upon extensions.
+
 typedef base::Callback<void(const std::string& public_key_spki_der,
                             const std::string& error_message)>
     GenerateKeyCallback;
 
 // Generates a RSA key with |modulus_length|. |token_id| is currently ignored,
 // instead the user token associated with |profile| 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,
                     const GenerateKeyCallback& callback,
                     Profile* profile);
 
-// If signing was successful, |signature| will be contain the signature and
-// |error_message| will be empty. If it failed, |signature| will be empty and
-// |error_message| contain an error message.
 typedef base::Callback<void(const std::string& signature,
                             const std::string& error_message)> SignCallback;
 
 // Signs |data| 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 |profile| 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,
           const std::string& data,
           const SignCallback& callback,
           Profile* profile);
 
+}  // namespace subtle
+
 // 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
@@ skipping 30 matching lines @@
 void RemoveCertificate(const std::string& token_id,
                        scoped_refptr<net::X509Certificate> certificate,
                        const RemoveCertificateCallback& callback,
                        Profile* profile);
 
 }  // namespace platform_keys
 
 }  // namespace chromeos
 
 #endif  // CHROME_BROWSER_CHROMEOS_PLATFORM_KEYS_PLATFORM_KEYS_H_