WebCrypto: Implement importKey() and sign() for HMAC in NSS

content/renderer/webcrypto_impl.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 CONTENT_RENDERER_WEBCRYPTO_IMPL_H_
 #define CONTENT_RENDERER_WEBCRYPTO_IMPL_H_
 
 #include "base/compiler_specific.h"
 
 #include "base/gtest_prod_util.h"
 #include "content/common/content_export.h"
 #include "third_party/WebKit/public/platform/WebCrypto.h"
 
 namespace content {
 
 class CONTENT_EXPORT WebCryptoImpl
     : NON_EXPORTED_BASE(public WebKit::WebCrypto) {
  public:
   virtual void digest(
       const WebKit::WebCryptoAlgorithm& algorithm,
       const unsigned char* data,
       size_t data_size,
       WebKit::WebCryptoResult result);
+  virtual void importKey(
+      WebKit::WebCryptoKeyFormat format,
+      const unsigned char* keyData,

[eroman, 2013/09/05 01:57:51]

style: key_data (unfortunately things can get confusing here because Blink and
Chromium have different styles!)

[Bryan Eyler, 2013/09/06 01:27:51]

Done.

+      size_t keyDataSize,

[eroman, 2013/09/05 01:57:51]

ditto

[Bryan Eyler, 2013/09/06 01:27:51]

Done.

+      const WebKit::WebCryptoAlgorithm& algorithm,
+      bool extractable,
+      WebKit::WebCryptoKeyUsageMask usage_mask,
+      WebKit::WebCryptoResult result);
+  virtual void sign(
+      const WebKit::WebCryptoAlgorithm& algorithm,
+      const WebKit::WebCryptoKey& key,
+      const unsigned char* data,
+      size_t data_size,
+      WebKit::WebCryptoResult result);
 
  protected:
   FRIEND_TEST_ALL_PREFIXES(WebCryptoImplTest, DigestSampleSets);
+  FRIEND_TEST_ALL_PREFIXES(WebCryptoImplTest, HMACSampleSets);
 
   bool digestInternal(
       const WebKit::WebCryptoAlgorithm& algorithm,
       const unsigned char* data,
       size_t data_size,
       WebKit::WebArrayBuffer* buffer);
+  bool importKeyInternal(
+      WebKit::WebCryptoKeyFormat format,
+      const unsigned char* key_data,
+      size_t key_data_size,
+      const WebKit::WebCryptoAlgorithm& algorithm,
+      bool extractable,

[eroman, 2013/09/05 01:57:51]

Can remove this it is unused.

[Bryan Eyler, 2013/09/06 01:27:51]

Done.

+      WebKit::WebCryptoKeyUsageMask usage_mask,
+      WebKit::WebCryptoKeyHandle** handle);

[eroman, 2013/09/05 01:57:51]

I asked on Elly's code review for generateKeyInternal() to be a scoped_ptr<>*
instead, I recommend the same approach for this function.

I also didn't notice on our previous code review which added digestIntenal(),
but these function names should be in chromium style. i.e.:

ImportKeyInternal().

[Bryan Eyler, 2013/09/06 01:27:51]

Done.  And fixed DigestInternal.

+  bool signInternal(
+      const WebKit::WebCryptoAlgorithm& algorithm,
+      const WebKit::WebCryptoKeyHandle* key,
+      const unsigned char* data,
+      size_t data_size,
+      WebKit::WebArrayBuffer* buffer);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_RENDERER_WEBCRYPTO_IMPL_H_