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

content/renderer/webcrypto_impl.cc

 // 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.
 
 #include "content/renderer/webcrypto_impl.h"
 
 #include "third_party/WebKit/public/platform/WebArrayBuffer.h"
+#include "third_party/WebKit/public/platform/WebCryptoAlgorithm.h"
+#include "third_party/WebKit/public/platform/WebCryptoKey.h"
 
 namespace content {
 
 void WebCryptoImpl::digest(
     const WebKit::WebCryptoAlgorithm& algorithm,
     const unsigned char* data,
     size_t data_size,
     WebKit::WebCryptoResult result) {
   WebKit::WebArrayBuffer buffer;
   if (!digestInternal(algorithm, data, data_size, &buffer)) {
     result.completeWithError();
   } else {
     result.completeWithBuffer(buffer);
   }
 }
 
+void WebCryptoImpl::importKey(
+    WebKit::WebCryptoKeyFormat format,
+    const unsigned char* key_data,
+    size_t key_data_size,
+    const WebKit::WebCryptoAlgorithm& algorithm,
+    bool extractable,
+    WebKit::WebCryptoKeyUsageMask usage_mask,
+    WebKit::WebCryptoResult result) {
+  WebKit::WebCryptoKeyType type;
+  switch (algorithm.id()) {
+    case WebKit::WebCryptoAlgorithmIdHmac:
+      type = WebKit::WebCryptoKeyTypeSecret;

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

For the future, we are better off having the "type" be set by imortKeyIntenal(),
just like how Elly does generateKeyInternal(). Later down the road when
importing other key types, it may not be known until the import has completed
whether it was a private or public key.

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

Done.

+      break;
+    default:
+      result.completeWithError();
+      return;
+  }
+
+  WebKit::WebCryptoKeyHandle* handle;
+  if (!importKeyInternal(format,
+                         key_data,
+                         key_data_size,
+                         algorithm,
+                         extractable,
+                         usage_mask,
+                         &handle)) {
+    result.completeWithError();

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

You need a return statement after this

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

Done.

+  }
+
+  WebKit::WebCryptoKey key(
+      WebKit::WebCryptoKey::create(
+          handle, type, extractable, algorithm, usage_mask));
+
+  result.completeWithKey(key);
+}
+
+void WebCryptoImpl::sign(
+    const WebKit::WebCryptoAlgorithm& algorithm,
+    const WebKit::WebCryptoKey& key,
+    const unsigned char* data,
+    size_t data_size,
+    WebKit::WebCryptoResult result) {
+  WebKit::WebArrayBuffer buffer;
+  if (!signInternal(algorithm, key.handle(), data, data_size, &buffer)) {
+    result.completeWithError();
+  } else {
+    result.completeWithBuffer(buffer);
+  }
+}
+
 }  // namespace content