Remove GetPkcs11Id from x509_certificate_model

chrome/browser/chromeos/options/cert_library.cc

 // Copyright 2013 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/browser/chromeos/options/cert_library.h"
 
 #include <algorithm>
 
 #include "base/command_line.h"
 #include "base/i18n/string_compare.h"
@@ skipping 143 matching lines @@
   return static_cast<int>(cert_list.size());
 }
 
 base::string16 CertLibrary::GetCertDisplayStringAt(CertType type,
                                                    int index) const {
   net::X509Certificate* cert = GetCertificateAt(type, index);
   bool hardware_backed = IsCertHardwareBackedAt(type, index);
   return GetDisplayString(cert, hardware_backed);
 }
 
-std::string CertLibrary::GetCertPEMAt(CertType type, int index) const {
-  return CertToPEM(*GetCertificateAt(type, index));
+std::string CertLibrary::GetServerCACertPEMAt(int index) const {
+  return CertToPEM(*GetCertificateAt(CERT_TYPE_SERVER_CA, index));
 }
 
-std::string CertLibrary::GetCertPkcs11IdAt(CertType type, int index) const {
-  net::X509Certificate* cert = GetCertificateAt(type, index);
-  return x509_certificate_model::GetPkcs11Id(cert->os_cert_handle());
+std::string CertLibrary::GetUserCertPkcs11IdAt(int index) const {
+  net::X509Certificate* cert = GetCertificateAt(CERT_TYPE_USER, index);
+  return CertLoader::GetPkcs11IdForCert(*cert);
 }
 
 bool CertLibrary::IsCertHardwareBackedAt(CertType type, int index) const {
   net::X509Certificate* cert = GetCertificateAt(type, index);
   return CertLoader::Get()->IsCertificateHardwareBacked(cert);
 }
 
-int CertLibrary::GetCertIndexByPEM(CertType type,
-                                   const std::string& pem_encoded) const {
-  int num_certs = NumCertificates(type);
+int CertLibrary::GetServerCACertIndexByPEM(
+    const std::string& pem_encoded) const {
+  int num_certs = NumCertificates(CERT_TYPE_SERVER_CA);
   for (int index = 0; index < num_certs; ++index) {
-    net::X509Certificate* cert = GetCertificateAt(type, index);
+    net::X509Certificate* cert = GetCertificateAt(CERT_TYPE_SERVER_CA, index);
     if (CertToPEM(*cert) != pem_encoded)
       continue;
     return index;
   }
   return -1;
 }
 
-int CertLibrary::GetCertIndexByPkcs11Id(CertType type,
-                                        const std::string& pkcs11_id) const {
-  int num_certs = NumCertificates(type);
+int CertLibrary::GetUserCertIndexByPkcs11Id(
+    const std::string& pkcs11_id) const {
+  int num_certs = NumCertificates(CERT_TYPE_USER);
   for (int index = 0; index < num_certs; ++index) {
-    net::X509Certificate* cert = GetCertificateAt(type, index);
-    net::X509Certificate::OSCertHandle cert_handle = cert->os_cert_handle();
-    std::string id = x509_certificate_model::GetPkcs11Id(cert_handle);
+    net::X509Certificate* cert = GetCertificateAt(CERT_TYPE_USER, index);
+    std::string id = CertLoader::GetPkcs11IdForCert(*cert);
     if (id == pkcs11_id)
       return index;
   }
   return -1;  // Not found.
 }
 
 void CertLibrary::OnCertificatesLoaded(const net::CertificateList& cert_list,
                                        bool initial_load) {
   CHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::UI));
   VLOG(1) << "CertLibrary::OnCertificatesLoaded: " << cert_list.size();
@@ skipping 65 matching lines @@
     return user_certs_;
   if (type == CERT_TYPE_SERVER)
     return server_certs_;
   if (type == CERT_TYPE_SERVER_CA)
     return server_ca_certs_;
   DCHECK(type == CERT_TYPE_DEFAULT);
   return certs_;
 }
 
 }  // namespace chromeos