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

Unified Diff: chrome/common/net/x509_certificate_model_unittest.cc

Issue 9940001: Fix imported server certs being distrusted in NSS 3.13. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: review fixes Created 8 years, 7 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: chrome/common/net/x509_certificate_model_unittest.cc
diff --git a/chrome/common/net/x509_certificate_model_unittest.cc b/chrome/common/net/x509_certificate_model_unittest.cc
index 233475e213d697e5f00751c4a4c94e7f830191fd..a745e6b26db4c16602401fbb17460f368d82ce4a 100644
--- a/chrome/common/net/x509_certificate_model_unittest.cc
+++ b/chrome/common/net/x509_certificate_model_unittest.cc
@@ -65,7 +65,7 @@ TEST(X509CertificateModelTest, GetTypeServer) {
x509_certificate_model::GetType(cert->os_cert_handle()));
EXPECT_TRUE(cert_db.SetCertTrust(cert, net::SERVER_CERT,
- net::CertDatabase::UNTRUSTED));
+ net::CertDatabase::EXPLICIT_DISTRUST));
wtc 2012/05/16 23:37:12 Does it matter whether we pass UNTRUSTED or EXPLIC
mattm 2012/05/18 03:40:54 Yeah, the first test after the #else is already te
EXPECT_EQ(net::SERVER_CERT,
x509_certificate_model::GetType(cert->os_cert_handle()));

Powered by Google App Engine
This is Rietveld 408576698