Bug 408359 - Password Manager dialog's Show/Hide Passwords button does not have an access key. p=ehsan.akhgari@gmail.com, r=mano, a1.9=beltzner

toolkit/components/passwordmgr/content/passwordManager.js

@@ -47,6 +47,7 @@ var kSignonBundle;
 function SignonsStartup() {
   kSignonBundle = document.getElementById("signonBundle");
   document.getElementById("togglePasswords").label = kSignonBundle.getString("showPasswords");
+  document.getElementById("togglePasswords").accessKey = kSignonBundle.getString("showPasswordsAccessKey");
   LoadSignons();
   FocusFilterBox();
 }
@@ -150,6 +151,7 @@ function TogglePasswordVisible() {
 
   showingPasswords = !showingPasswords;
   document.getElementById("togglePasswords").label = kSignonBundle.getString(showingPasswords ? "hidePasswords" : "showPasswords");
+  document.getElementById("togglePasswords").accessKey = kSignonBundle.getString(showingPasswords ? "hidePasswordsAccessKey" : "showPasswordsAccessKey");
   document.getElementById("passwordCol").hidden = !showingPasswords;
   _filterPasswords();
 }

toolkit/locales/en-US/chrome/passwordmgr/passwordmgr.properties

@@ -52,7 +52,9 @@ passwordChangeText = Would you like to have Password Manager change the stored p
 passwordChangeTextNoUser = Would you like to have Password Manager change the stored password for this login?
 userSelectText = Please confirm which user you are changing the password for
 hidePasswords=Hide Passwords
+hidePasswordsAccessKey=H
 showPasswords=Show Passwords
+showPasswordsAccessKey=S
 noMasterPasswordPrompt=Are you sure you wish to show your passwords?
 removeAllPasswordsPrompt=Are you sure you wish to remove all passwords?
 removeAllPasswordsTitle=Remove all passwords