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

Unified Diff: LayoutTests/crypto/aes-kw-generateKey-parseAlgorithm-failures.html

Issue 243853004: [webcrypto] Reject failed operations with a DOMException rather than null. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Fix compile warning Created 6 years, 8 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
Index: LayoutTests/crypto/aes-kw-generateKey-parseAlgorithm-failures.html
diff --git a/LayoutTests/crypto/aes-kw-generateKey-parseAlgorithm-failures.html b/LayoutTests/crypto/aes-kw-generateKey-parseAlgorithm-failures.html
index a1ebe38039a34a7b40daff6ae8b0fcb5f5c40cd5..9c17fada6e3c1dff36e8232720acb1d705217473 100644
--- a/LayoutTests/crypto/aes-kw-generateKey-parseAlgorithm-failures.html
+++ b/LayoutTests/crypto/aes-kw-generateKey-parseAlgorithm-failures.html
@@ -18,23 +18,19 @@ keyUsages = ['wrapKey', 'unwrapKey'];
Promise.resolve(null).then(function() {
return crypto.subtle.generateKey({ name: 'aes-kw' }, extractable, keyUsages);
}).then(failAndFinishJSTest, function(result) {
- error = result;
- shouldBeNull("error");
+ logError(result);
return crypto.subtle.generateKey({ name: 'aes-kw', length: 70000 }, extractable, keyUsages);
}).then(failAndFinishJSTest, function(result) {
- error = result;
- shouldBeNull("error");
+ logError(result);
return crypto.subtle.generateKey({ name: 'aes-kw', length: -3 }, extractable, keyUsages);
}).then(failAndFinishJSTest, function(result) {
- error = result;
- shouldBeNull("error");
+ logError(result);
return crypto.subtle.generateKey({ name: 'aes-kw', length: -Infinity }, extractable, keyUsages);
}).then(failAndFinishJSTest, function(result) {
- error = result;
- shouldBeNull("error");
+ logError(result);
}).then(finishJSTest, failAndFinishJSTest);
</script>

Powered by Google App Engine
This is Rietveld 408576698