Remove NSSCertDatabase from ClientCertStoreChromeOS unittest.

net/ssl/client_cert_store_chromeos.h

Index: net/ssl/client_cert_store_chromeos.h
diff --git a/net/ssl/client_cert_store_chromeos.h b/net/ssl/client_cert_store_chromeos.h
index 41339f1b5bb0687c5eea035274be4428d9b59648..087190ce11a6e1650630c25f1e23ae455fb2b0d0 100644
--- a/net/ssl/client_cert_store_chromeos.h
+++ b/net/ssl/client_cert_store_chromeos.h
@@ -40,21 +40,6 @@ class NET_EXPORT ClientCertStoreChromeOS : public ClientCertStoreNSS {
                          const base::Closure& callback,
                          crypto::ScopedPK11Slot private_slot);
 
-  // Allows tests to initialize the cert store with the given slots.

[pneubeck (no reviews), 2014/07/17 13:40:50]

I just noticed today these two functions that Matt initially added (likely
copy&pasted from the NSS implementation).
This indicates that he planned to indeed reuse the test templates in
client_cert_store_unittest-inl.h also for the ChromeOS implementation.

I updated the ChromeOS unittest to apply these test.
However, it wouldn't work to use these two functions only (i.e. don't import the
client certs into a DB but directly pass them to SelectClientCertsForTesting),
because the profile_filter_ does only accepts certs that _are_ stored in a
specific slot.

Thus, instead I removed these testing functions and test this class through its
public interface only (which is the preferred way anyways).
A minor glitch is that I have to import the private keys in advance, because the
test template doesn't provide them (could be changed of course).

In a follow up, I should update the NSS test to do the same.

-  // Must be called before SelectClientCertsForTesting.
-  void InitForTesting(crypto::ScopedPK11Slot public_slot,
-                      crypto::ScopedPK11Slot private_slot);
-
-  // A hook for testing. Filters |input_certs| using the logic being used to
-  // filter the system store when GetClientCerts() is called.
-  // Implemented by creating a list of certificates that otherwise would be
-  // extracted from the system store and filtering it using the common logic
-  // (less adequate than the approach used on Windows).
-  bool SelectClientCertsForTesting(const CertificateList& input_certs,
-                                   const SSLCertRequestInfo& cert_request_info,
-                                   CertificateList* selected_certs);
-
-
   std::string username_hash_;
   NSSProfileFilterChromeOS profile_filter_;