net: remove CryptoModuleList typedef

chrome/browser/ui/crypto_module_password_dialog_nss.cc

Index: chrome/browser/ui/crypto_module_password_dialog_nss.cc
diff --git a/chrome/browser/ui/crypto_module_password_dialog_nss.cc b/chrome/browser/ui/crypto_module_password_dialog_nss.cc
index a6270dcd469b3f1e94e46df11acc3b81891690a2..d0086b6695699a40e85060c10aa956fc8753f91d 100644
--- a/chrome/browser/ui/crypto_module_password_dialog_nss.cc
+++ b/chrome/browser/ui/crypto_module_password_dialog_nss.cc
@@ -17,10 +17,9 @@ using content::BrowserThread;
 
 namespace {
 
-bool ShouldShowDialog(const net::CryptoModule* module) {
+bool ShouldShowDialog(PK11SlotInfo* slot) {
   // The wincx arg is unused since we don't call PK11_SetIsLoggedInFunc.
-  return (PK11_NeedLogin(module->os_module_handle()) &&
-          !PK11_IsLoggedIn(module->os_module_handle(), NULL /* wincx */));
+  return (PK11_NeedLogin(slot) && !PK11_IsLoggedIn(slot, NULL /* wincx */));
 }
 
 // Basically an asynchronous implementation of NSS's PK11_DoPassword.
@@ -28,7 +27,7 @@ bool ShouldShowDialog(const net::CryptoModule* module) {
 // GotPassword for what is yet unimplemented.
 class SlotUnlocker {
  public:
-  SlotUnlocker(const net::CryptoModuleList& modules,
+  SlotUnlocker(const std::vector<crypto::ScopedPK11Slot>& modules,
                chrome::CryptoModulePasswordReason reason,
                const net::HostPortPair& server,
                gfx::NativeWindow parent,
@@ -41,7 +40,7 @@ class SlotUnlocker {
   void Done();
 
   size_t current_;
-  net::CryptoModuleList modules_;
+  std::vector<crypto::ScopedPK11Slot> modules_;
   chrome::CryptoModulePasswordReason reason_;
   net::HostPortPair server_;
   gfx::NativeWindow parent_;
@@ -49,19 +48,21 @@ class SlotUnlocker {
   PRBool retry_;
 };
 
-SlotUnlocker::SlotUnlocker(const net::CryptoModuleList& modules,
+SlotUnlocker::SlotUnlocker(const std::vector<crypto::ScopedPK11Slot>& modules,
                            chrome::CryptoModulePasswordReason reason,
                            const net::HostPortPair& server,
                            gfx::NativeWindow parent,
                            const base::Closure& callback)
     : current_(0),
-      modules_(modules),
       reason_(reason),
       server_(server),
       parent_(parent),
       callback_(callback),
       retry_(PR_FALSE) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
+  for (const auto& slot : modules) {
+    modules_.push_back(crypto::ScopedPK11Slot(PK11_ReferenceSlot(slot.get())));

[mattm, 2017/02/24 21:03:39]

hm

[Bence, 2017/02/24 22:43:20]

I was also looking at this code.  ScopedPK11Slot is a unique_ptr<PK11SlotInfo>
typedef, so it is not copyable, therefore it could not be passed by value to
this constructor.  But PK11_ReferenceSlot does not copy the PK11SlotInfo object,
it just increases its |RefCount| member.  After this line, the same PK11SlotInfo
instance will be owned by two unique_ptrs: one owned by the caller, one owned by
SlotUnlocker!  What am I missing?

[mattm, 2017/02/24 22:51:26]

Oh, that's actually okay. (I meant to delete that comment before I sent the
review.)

ScopedPK11Slot's Deleter is PK11_FreeSlot which decrements the refcount, so it's
not actually deleted unless there are no more references. ScopedPK11Slot is just
using unique_ptr to own references to the object.

+  }
 }
 
 void SlotUnlocker::Start() {
@@ -70,11 +71,8 @@ void SlotUnlocker::Start() {
   for (; current_ < modules_.size(); ++current_) {
     if (ShouldShowDialog(modules_[current_].get())) {
       ShowCryptoModulePasswordDialog(
-          modules_[current_]->GetTokenName(),
-          retry_,
-          reason_,
-          server_.host(),
-          parent_,
+          PK11_GetTokenName(modules_[current_].get()), retry_, reason_,
+          server_.host(), parent_,
           base::Bind(&SlotUnlocker::GotPassword, base::Unretained(this)));
       return;
     }
@@ -95,8 +93,8 @@ void SlotUnlocker::GotPassword(const std::string& password) {
   }
 
   // TODO(mattm): handle protectedAuthPath
-  SECStatus rv = PK11_CheckUserPassword(modules_[current_]->os_module_handle(),
-                                        password.c_str());
+  SECStatus rv =
+      PK11_CheckUserPassword(modules_[current_].get(), password.c_str());
   if (rv == SECWouldBlock) {
     // Incorrect password.  Try again.
     retry_ = PR_TRUE;
@@ -123,7 +121,7 @@ void SlotUnlocker::Done() {
 
 namespace chrome {
 
-void UnlockSlotsIfNecessary(const net::CryptoModuleList& modules,
+void UnlockSlotsIfNecessary(const std::vector<crypto::ScopedPK11Slot>& modules,
                             chrome::CryptoModulePasswordReason reason,
                             const net::HostPortPair& server,
                             gfx::NativeWindow parent,
@@ -143,9 +141,9 @@ void UnlockCertSlotIfNecessary(net::X509Certificate* cert,
                                const net::HostPortPair& server,
                                gfx::NativeWindow parent,
                                const base::Closure& callback) {
-  net::CryptoModuleList modules;
-  modules.push_back(net::CryptoModule::CreateFromHandle(
-      cert->os_cert_handle()->slot));
+  std::vector<crypto::ScopedPK11Slot> modules;
+  modules.push_back(
+      crypto::ScopedPK11Slot(PK11_ReferenceSlot(cert->os_cert_handle()->slot)));
   UnlockSlotsIfNecessary(modules, reason, server, parent, callback);
 }