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

Unified Diff: net/cert/internal/verify_signed_data_unittest.cc

Issue 1647953004: Roll src/third_party/boringssl/src 6c22f542f..2cdf39877 (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 11 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 | « DEPS ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/cert/internal/verify_signed_data_unittest.cc
diff --git a/net/cert/internal/verify_signed_data_unittest.cc b/net/cert/internal/verify_signed_data_unittest.cc
index d7e822869cd7de9f418b5c9964b688d0152ddc59..7cb519db1f385f8290def4fcb8c26e7243e8d69a 100644
--- a/net/cert/internal/verify_signed_data_unittest.cc
+++ b/net/cert/internal/verify_signed_data_unittest.cc
@@ -211,9 +211,7 @@ TEST(VerifySignedDataTest, EcdsaPrime256v1Sha512SpkiParamsNull) {
}
TEST(VerifySignedDataTest, RsaPkcs1Sha256UsingIdEaRsa) {
- // TODO(eroman): This should fail! (shouldn't recognize this weird OID).
- // See https://crbug.com/522228
- RunTestCase(SUCCESS, "rsa-pkcs1-sha256-using-id-ea-rsa.pem");
+ RunTestCase(FAILURE, "rsa-pkcs1-sha256-using-id-ea-rsa.pem");
davidben 2016/01/28 22:32:28 +eroman FYI
eroman 2016/01/28 23:32:00 lgtm
}
TEST(VerifySignedDataTest, RsaPkcs1Sha256SpkiNonNullParams) {
« no previous file with comments | « DEPS ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698