Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(831)

Unified Diff: components/webcrypto/algorithms/aes.cc

Issue 1355923002: [refactor] Misc post-NSS WebCrypto cleanups. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@util_split
Patch Set: add an explicit size_t --> unsigned cast Created 5 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « components/webcrypto/algorithm_dispatch.cc ('k') | components/webcrypto/algorithms/aes_cbc.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/webcrypto/algorithms/aes.cc
diff --git a/components/webcrypto/algorithms/aes.cc b/components/webcrypto/algorithms/aes.cc
index ed5bbb62a0f07cb1be6f5989b1c9978091ed184c..f42d508619b5f7261b0323cf1791a4baca4e3b17 100644
--- a/components/webcrypto/algorithms/aes.cc
+++ b/components/webcrypto/algorithms/aes.cc
@@ -56,7 +56,7 @@ Status AesAlgorithm::GenerateKey(const blink::WebCryptoAlgorithm& algorithm,
unsigned int keylen_bits = algorithm.aesKeyGenParams()->lengthBits();
- // BoringSSL does not support 192-bit AES.
+ // 192-bit AES is intentionally unsupported (http://crbug.com/533699).
if (keylen_bits == 192)
return Status::ErrorAes192BitUnsupported();
@@ -87,7 +87,7 @@ Status AesAlgorithm::ImportKeyRaw(const CryptoData& key_data,
blink::WebCryptoKey* key) const {
const unsigned int keylen_bytes = key_data.byte_length();
- // BoringSSL does not support 192-bit AES.
+ // 192-bit AES is intentionally unsupported (http://crbug.com/533699).
if (keylen_bytes == 24)
return Status::ErrorAes192BitUnsupported();
@@ -178,7 +178,7 @@ Status AesAlgorithm::GetKeyLength(
if (*length_bits == 128 || *length_bits == 256)
return Status::Success();
- // BoringSSL does not support 192-bit AES.
+ // 192-bit AES is intentionally unsupported (http://crbug.com/533699).
if (*length_bits == 192)
return Status::ErrorAes192BitUnsupported();
« no previous file with comments | « components/webcrypto/algorithm_dispatch.cc ('k') | components/webcrypto/algorithms/aes_cbc.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698