Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(804)

Unified Diff: chrome/browser/ui/crypto_module_delegate_nss.h

Issue 2922423002: Remove NSSCryptoModuleDelegate. (Closed)
Patch Set: Created 3 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/net/nss_context_linux.cc ('k') | chrome/browser/ui/crypto_module_delegate_nss.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/crypto_module_delegate_nss.h
diff --git a/chrome/browser/ui/crypto_module_delegate_nss.h b/chrome/browser/ui/crypto_module_delegate_nss.h
index 45029f8b7c057f7f4cadc05652c01867c4565f3f..3af79b9105f04b850234e74e87a36489bc43892f 100644
--- a/chrome/browser/ui/crypto_module_delegate_nss.h
+++ b/chrome/browser/ui/crypto_module_delegate_nss.h
@@ -7,45 +7,26 @@
#include <string>
-#include "base/compiler_specific.h"
#include "base/macros.h"
#include "base/synchronization/waitable_event.h"
#include "chrome/browser/ui/crypto_module_password_dialog.h"
#include "crypto/nss_crypto_module_delegate.h"
#include "net/base/host_port_pair.h"
-namespace content {
-class ResourceContext;
-}
-
// Delegate to handle unlocking a slot or indicating which slot to store a key
// in. When passing to NSS functions which take a wincx argument, use the value
// returned from the wincx() method.
class ChromeNSSCryptoModuleDelegate
- : public crypto::NSSCryptoModuleDelegate {
+ : public crypto::CryptoModuleBlockingPasswordDelegate {
public:
// Create a ChromeNSSCryptoModuleDelegate. |reason| is used to select what
// string to show the user, |server| is displayed to indicate which connection
// is causing the dialog to appear. |slot| can be NULL.
ChromeNSSCryptoModuleDelegate(chrome::CryptoModulePasswordReason reason,
- const net::HostPortPair& server,
- crypto::ScopedPK11Slot slot);
+ const net::HostPortPair& server);
~ChromeNSSCryptoModuleDelegate() override;
- // Must be called on IO thread. Creates a delegate and returns it
- // synchronously or asynchronously to |callback|. If the delegate could not be
- // created, |callback| is called with NULL.
- static void CreateForResourceContext(
- chrome::CryptoModulePasswordReason reason,
- const net::HostPortPair& server,
- content::ResourceContext* context,
- const base::Callback<
- void(std::unique_ptr<ChromeNSSCryptoModuleDelegate>)>& callback);
-
- // crypto::NSSCryptoModuleDelegate implementation.
- crypto::ScopedPK11Slot RequestSlot() override;
-
// crypto::CryptoModuleBlockingPasswordDelegate implementation.
std::string RequestPassword(const std::string& slot_name,
bool retry,
@@ -67,9 +48,6 @@ class ChromeNSSCryptoModuleDelegate
std::string password_;
bool cancelled_;
- // The slot which will be returned by RequestSlot.
- crypto::ScopedPK11Slot slot_;
-
DISALLOW_COPY_AND_ASSIGN(ChromeNSSCryptoModuleDelegate);
};
« no previous file with comments | « chrome/browser/net/nss_context_linux.cc ('k') | chrome/browser/ui/crypto_module_delegate_nss.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698