Refactor client cert private key handling.

chrome/browser/chromeos/net/client_cert_store_chromeos_unittest.cc

Index: chrome/browser/chromeos/net/client_cert_store_chromeos_unittest.cc
diff --git a/chrome/browser/chromeos/net/client_cert_store_chromeos_unittest.cc b/chrome/browser/chromeos/net/client_cert_store_chromeos_unittest.cc
index 6cd545d612103c0ad9050cb037d723f39da9d53a..74e56b1e4a458f1305d79152ba28f9b6f8dc22c0 100644
--- a/chrome/browser/chromeos/net/client_cert_store_chromeos_unittest.cc
+++ b/chrome/browser/chromeos/net/client_cert_store_chromeos_unittest.cc
@@ -19,6 +19,7 @@
 #include "chrome/browser/chromeos/certificate_provider/certificate_provider.h"
 #include "crypto/scoped_test_nss_db.h"
 #include "net/cert/x509_certificate.h"
+#include "net/ssl/client_cert_identity_test_util.h"
 #include "net/ssl/ssl_cert_request_info.h"
 #include "net/test/cert_test_util.h"
 #include "net/test/test_data_directory.h"
@@ -74,9 +75,9 @@ class TestCertFilter : public ClientCertStoreChromeOS::CertFilter {
   scoped_refptr<net::X509Certificate> not_allowed_cert_;
 };
 
-void SaveCertsAndQuitCallback(net::CertificateList* out_certs,
+void SaveCertsAndQuitCallback(net::ClientCertIdentityList* out_certs,
                               base::Closure quit_closure,
-                              net::CertificateList in_certs) {
+                              net::ClientCertIdentityList in_certs) {
   *out_certs = std::move(in_certs);
   quit_closure.Run();
 }
@@ -119,7 +120,7 @@ TEST_F(ClientCertStoreChromeOSTest, RequestWaitsForNSSInitAndSucceeds) {
   scoped_refptr<net::SSLCertRequestInfo> request_all(
       new net::SSLCertRequestInfo());
 
-  net::CertificateList selected_certs;
+  net::ClientCertIdentityList selected_certs;
   base::RunLoop run_loop;
   store.GetClientCerts(*request_all,
                        base::Bind(SaveCertsAndQuitCallback, &selected_certs,
@@ -158,7 +159,7 @@ TEST_F(ClientCertStoreChromeOSTest, RequestsAfterNSSInitSucceed) {
       new net::SSLCertRequestInfo());
 
   base::RunLoop run_loop;
-  net::CertificateList selected_certs;
+  net::ClientCertIdentityList selected_certs;
   store.GetClientCerts(*request_all,
                        base::Bind(SaveCertsAndQuitCallback, &selected_certs,
                                   run_loop.QuitClosure()));
@@ -190,27 +191,27 @@ TEST_F(ClientCertStoreChromeOSTest, Filter) {
   {
     base::RunLoop run_loop;
     cert_filter->SetNotAllowedCert(cert_2);
-    net::CertificateList selected_certs;
+    net::ClientCertIdentityList selected_certs;
     store.GetClientCerts(*request_all,
                          base::Bind(SaveCertsAndQuitCallback, &selected_certs,
                                     run_loop.QuitClosure()));
     run_loop.Run();
 
     ASSERT_EQ(1u, selected_certs.size());
-    EXPECT_TRUE(cert_1->Equals(selected_certs[0].get()));
+    EXPECT_TRUE(cert_1->Equals(selected_certs[0]->certificate()));
   }
 
   {
     base::RunLoop run_loop;
     cert_filter->SetNotAllowedCert(cert_1);
-    net::CertificateList selected_certs;
+    net::ClientCertIdentityList selected_certs;
     store.GetClientCerts(*request_all,
                          base::Bind(SaveCertsAndQuitCallback, &selected_certs,
                                     run_loop.QuitClosure()));
     run_loop.Run();
 
     ASSERT_EQ(1u, selected_certs.size());
-    EXPECT_TRUE(cert_2->Equals(selected_certs[0].get()));
+    EXPECT_TRUE(cert_2->Equals(selected_certs[0]->certificate()));
   }
 }
 
@@ -241,14 +242,14 @@ TEST_F(ClientCertStoreChromeOSTest, CertRequestMatching) {
   request->cert_authorities = authority_1;
 
   base::RunLoop run_loop;
-  net::CertificateList selected_certs;
+  net::ClientCertIdentityList selected_certs;
   store.GetClientCerts(*request,
                        base::Bind(SaveCertsAndQuitCallback, &selected_certs,
                                   run_loop.QuitClosure()));
   run_loop.Run();
 
   ASSERT_EQ(1u, selected_certs.size());
-  EXPECT_TRUE(cert_1->Equals(selected_certs[0].get()));
+  EXPECT_TRUE(cert_1->Equals(selected_certs[0]->certificate()));
 }
 
 }  // namespace chromeos