With this I'm able to view certs on an Aladdin eToken Pro. Exporting fails but
the card may just not allow it (Firefox can't export from it either).
Also if you've used the cert manager to unlock it, then it works for SSL client
auth. Supporting unlocking before the client auth cert selection will be a bit
more work...
wtc
LGTM. I have some comments and suggested changes below. Also note the comment marked with ...
http://codereview.chromium.org/6580058/diff/1012/chrome/app/generated_resources.grd File chrome/app/generated_resources.grd (right): http://codereview.chromium.org/6580058/diff/1012/chrome/app/generated_resources.grd#newcode6383 chrome/app/generated_resources.grd:6383: <message name="IDS_CRYPTO_MODULE_AUTH_DIALOG_TEXT_CERT_MANAGER" desc="The text in the dialog that prompts ...
LGTM! http://codereview.chromium.org/6580058/diff/11001/chrome/app/generated_resources.grd File chrome/app/generated_resources.grd (right): http://codereview.chromium.org/6580058/diff/11001/chrome/app/generated_resources.grd#newcode6439 chrome/app/generated_resources.grd:6439: <message name="IDS_CRYPTO_MODULE_AUTH_DIALOG_TEXT_LIST_CERTS" desc="The text in the dialog that ...
LGTM!
http://codereview.chromium.org/6580058/diff/11001/chrome/app/generated_resour...
File chrome/app/generated_resources.grd (right):
http://codereview.chromium.org/6580058/diff/11001/chrome/app/generated_resour...
chrome/app/generated_resources.grd:6439: <message
name="IDS_CRYPTO_MODULE_AUTH_DIALOG_TEXT_LIST_CERTS" desc="The text in the
dialog that prompts for security device password when using the certificate
manager without taking a specific action (eg, just loading the cert manager).">
Nit: since the name of the message has changed to say
"LIST_CERTS", the description of the message should be updated
to say "listing certificates" instead of "loading the
cert manager".
Issue 6580058: NSS: Unlock crypto devices when populating cert manager.
(Closed)
Created 7 years, 1 month ago by mattm
Modified 6 years, 11 months ago
Reviewers: wtc
Base URL: svn://svn.chromium.org/chrome/trunk/src
Comments: 19