Check that usage isn't empty when generateKey() is called

content/child/webcrypto/nss/sym_key_nss.cc

 // 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.
 
 #include "content/child/webcrypto/nss/sym_key_nss.h"
 
 #include "base/logging.h"
 #include "content/child/webcrypto/crypto_data.h"
 #include "content/child/webcrypto/generate_key_result.h"
 #include "content/child/webcrypto/nss/key_nss.h"
@@ skipping 25 matching lines @@
   if (!pk11_key)
     return Status::OperationError();
 
   if (PK11_ExtractKeyValue(pk11_key.get()) != SECSuccess)
     return Status::OperationError();
 
   const SECItem* key_data = PK11_GetKeyData(pk11_key.get());
   if (!key_data)
     return Status::OperationError();
 
+  if (usages == 0)
+    return Status::ErrorCreateKeyEmptyUsages();

[eroman, 2014/11/21 23:35:34]

Please move this towards the top of the function (so it fails fast before more
work is done).

[nharper, 2014/11/22 00:26:45]

Done.

+
   scoped_ptr<SymKeyNss> handle(new SymKeyNss(
       pk11_key.Pass(), CryptoData(key_data->data, key_data->len)));
 
   result->AssignSecretKey(
       blink::WebCryptoKey::create(handle.release(),
                                   blink::WebCryptoKeyTypeSecret,
                                   extractable,
                                   algorithm,
                                   usages));
 
@@ skipping 29 matching lines @@
                                      blink::WebCryptoKeyTypeSecret,
                                      extractable,
                                      algorithm,
                                      usages);
   return Status::Success();
 }
 
 }  // namespace webcrypto
 
 }  // namespace content