Fix imported server certs being distrusted in NSS 3.13.

chrome/common/net/x509_certificate_model_unittest.cc

 // Copyright (c) 2012 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 "chrome/common/net/x509_certificate_model.h"
 
 #include "base/file_path.h"
 #include "base/file_util.h"
 #include "base/path_service.h"
 #include "net/base/cert_database.h"
@@ skipping 10 matching lines @@
   // Remove this when OpenSSL build implements the necessary functions.
   EXPECT_EQ(net::UNKNOWN_CERT,
             x509_certificate_model::GetType(cert->os_cert_handle()));
 #else
   EXPECT_EQ(net::CA_CERT,
             x509_certificate_model::GetType(cert->os_cert_handle()));
 
   // Test that explicitly distrusted CA certs are still returned as CA_CERT
   // type. See http://crbug.com/96654.
   net::CertDatabase cert_db;
-  // TODO(mattm): This depends on the implementation details of SetCertTrust
-  // where calling with SERVER_CERT and UNTRUSTED causes a cert to be explicitly
-  // distrusted (trust set to CERTDB_TERMINAL_RECORD). See
-  // http://crbug.com/116411.  When I fix that bug I'll also add a way to set
-  // this directly.
-  EXPECT_TRUE(cert_db.SetCertTrust(cert, net::SERVER_CERT,
-                                   net::CertDatabase::UNTRUSTED));
+  EXPECT_TRUE(cert_db.SetCertTrust(cert, net::CA_CERT,
+                                   net::CertDatabase::DISTRUSTED_SSL));
 
   EXPECT_EQ(net::CA_CERT,
             x509_certificate_model::GetType(cert->os_cert_handle()));
 #endif
 }
 
 TEST(X509CertificateModelTest, GetTypeServer) {
   scoped_refptr<net::X509Certificate> cert(
       net::ImportCertFromFile(net::GetTestCertsDirectory(),
                               "google.single.der"));
   ASSERT_TRUE(cert.get());
 
 #if defined(USE_OPENSSL)
   // Remove this when OpenSSL build implements the necessary functions.
   EXPECT_EQ(net::UNKNOWN_CERT,
             x509_certificate_model::GetType(cert->os_cert_handle()));
 #else
+  // Test GetCertType with server certs and default trust.  Currently this

[wtc, 2012/05/22 00:28:39]

In these comments, "GetCertType" is a little confusing because
the test calls x509_certificate_model::GetType.  It may be
clearer to fully qualify GetCertType in the comments.

[mattm, 2012/05/26 03:41:35]

Done.

+  // doesn't work.
   // TODO(mattm): make GetCertType smarter so we can tell server certs even if
   // they have no trust bits set.
   EXPECT_EQ(net::UNKNOWN_CERT,
             x509_certificate_model::GetType(cert->os_cert_handle()));
 
   net::CertDatabase cert_db;
+  // Test GetCertType with server certs and explicit trust.
   EXPECT_TRUE(cert_db.SetCertTrust(cert, net::SERVER_CERT,
                                    net::CertDatabase::TRUSTED_SSL));
 
   EXPECT_EQ(net::SERVER_CERT,
             x509_certificate_model::GetType(cert->os_cert_handle()));
 
+  // Test GetCertType with server certs and explicit distrust.
   EXPECT_TRUE(cert_db.SetCertTrust(cert, net::SERVER_CERT,
-                                   net::CertDatabase::UNTRUSTED));
+                                   net::CertDatabase::DISTRUSTED_SSL));
 
   EXPECT_EQ(net::SERVER_CERT,
             x509_certificate_model::GetType(cert->os_cert_handle()));
 #endif
 }