Refactor client cert private key handling.

net/ssl/client_key_store.h

 // Copyright 2015 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.
 
 #ifndef NET_SSL_CLIENT_KEY_STORE_H_
 #define NET_SSL_CLIENT_KEY_STORE_H_
 
 #include <memory>
 #include <vector>
 
 #include "base/callback.h"
 #include "base/lazy_instance.h"
 #include "base/macros.h"
 #include "base/synchronization/lock.h"
 #include "net/base/net_export.h"
 
 namespace net {
 
 class SSLPrivateKey;
 class X509Certificate;
 
+// TODO(mattm): This ends up still being used due to the platformKeys api. Move
+// it under chrome/browser/chromeos/net or remove it and call
+// CertificateProviderService directly?
 // TODO(rsleevi, davidben): Remove this once https://crbug.com/394131 is fixed.
 // A certificate and key store that allows several external certificate
 // providers to expose certificates and keys through this store. All currently
 // provided certificates will be accessible through |FetchClientCertPrivateKey|.
 // Methods of this singleton can be called from any thread.
 class NET_EXPORT ClientKeyStore {
  public:
   class CertKeyProvider {
    public:
     // This can be called from any thread.
@@ skipping 33 matching lines @@
 
   base::Lock lock_;
   std::vector<CertKeyProvider*> providers_;
 
   DISALLOW_COPY_AND_ASSIGN(ClientKeyStore);
 };
 
 }  // namespace net
 
 #endif  // NET_SSL_CLIENT_KEY_STORE_H_