[WebCrypto] Throw syntaxError if keyUsage is empty in ImportKey

content/child/webcrypto/openssl/aes_key_openssl.cc

Index: content/child/webcrypto/openssl/aes_key_openssl.cc
diff --git a/content/child/webcrypto/openssl/aes_key_openssl.cc b/content/child/webcrypto/openssl/aes_key_openssl.cc
index 1f853b557a92df1ad300109a284363f490b63d7d..6357300d48e6ea9a9cf25d2be8eaf2939219c700 100644
--- a/content/child/webcrypto/openssl/aes_key_openssl.cc
+++ b/content/child/webcrypto/openssl/aes_key_openssl.cc
@@ -34,7 +34,7 @@ Status AesAlgorithm::GenerateKey(const blink::WebCryptoAlgorithm& algorithm,
                                  bool extractable,
                                  blink::WebCryptoKeyUsageMask usages,
                                  GenerateKeyResult* result) const {
-  Status status = CheckKeyCreationUsages(all_key_usages_, usages);
+  Status status = CheckKeyCreationUsages(all_key_usages_, usages, false);

[eroman, 2014/12/09 21:04:46]

Same comment as before, let's do the empty usage check here so it doesn't need
to be duplicated later.

[Habib Virji, 2014/12/15 18:48:55]

Done.

   if (status.IsError())
     return status;
 
@@ -54,7 +54,7 @@ Status AesAlgorithm::VerifyKeyUsagesBeforeImportKey(
   switch (format) {
     case blink::WebCryptoKeyFormatRaw:
     case blink::WebCryptoKeyFormatJwk:
-      return CheckKeyCreationUsages(all_key_usages_, usages);
+      return CheckKeyCreationUsages(all_key_usages_, usages, true);
     default:
       return Status::ErrorUnsupportedImportKeyFormat();
   }