Commit 2066f648 authored by Jan Wilken Dörrie's avatar Jan Wilken Dörrie Committed by Commit Bot

[Passwords] Add 'Hide Password' string and remove show password functionality

This change adds a string for "Hide Password" which is supposed to be
used in the more actions menu in the Password Check page. Furthermore,
this change removes the "Show Password" menu entry for now, as there is
no functionality.

Bug: 1047726
Change-Id: I0f96d1794715b97754b9d3141dd75c3b0534149e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2100748
Commit-Queue: Jan Wilken Dörrie <jdoerrie@chromium.org>
Commit-Queue: Vasilii Sukhanov <vasilii@chromium.org>
Auto-Submit: Jan Wilken Dörrie <jdoerrie@chromium.org>
Reviewed-by: default avatarVasilii Sukhanov <vasilii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#749722}
parent 553c6cea
...@@ -407,6 +407,9 @@ ...@@ -407,6 +407,9 @@
<message name="IDS_SETTINGS_COMPROMISED_PASSWORD_SHOW" desc="Action menu item for a row which displays a compromised password. The password value, which is initialy obfuscated, will be show in plain text."> <message name="IDS_SETTINGS_COMPROMISED_PASSWORD_SHOW" desc="Action menu item for a row which displays a compromised password. The password value, which is initialy obfuscated, will be show in plain text.">
Show password Show password
</message> </message>
<message name="IDS_SETTINGS_COMPROMISED_PASSWORD_HIDE" desc="Action menu item for a row which displays a compromised password. The password value, which is shown in plain text, will be hidden again.">
Hide password
</message>
<message name="IDS_SETTINGS_COMPROMISED_PASSWORD_EDIT" desc="Action menu item for a row which displays a compromised password. It will open a modal dialog which will allow the user to edit the password and other metadata (username, website)."> <message name="IDS_SETTINGS_COMPROMISED_PASSWORD_EDIT" desc="Action menu item for a row which displays a compromised password. It will open a modal dialog which will allow the user to edit the password and other metadata (username, website).">
Edit password Edit password
</message> </message>
......
07f9122e0d93c125efe2bdc2861aaa8d4f0297ac
\ No newline at end of file
...@@ -124,10 +124,7 @@ ...@@ -124,10 +124,7 @@
</div> </div>
</div> </div>
<cr-action-menu id="moreActionsMenu" role-description="$i18n{menu}"> <cr-action-menu id="moreActionsMenu" role-description="$i18n{menu}">
<button id="menuShowPassword" class="dropdown-item" <!-- TODO(crbug.com/1047726) Add show / hide functionality -->
on-click="onMenuShowPasswordClick_">
$i18n{showCompromisedPassword}
</button>
<button id="menuEditPassword" class="dropdown-item" <button id="menuEditPassword" class="dropdown-item"
on-click="onMenuEditPasswordClick_"> on-click="onMenuEditPasswordClick_">
$i18n{editCompromisedPassword} $i18n{editCompromisedPassword}
......
...@@ -185,13 +185,6 @@ Polymer({ ...@@ -185,13 +185,6 @@ Polymer({
this.activeDialogAnchor_ = target; this.activeDialogAnchor_ = target;
}, },
/** @private */
onMenuShowPasswordClick_() {
this.$.moreActionsMenu.close();
// TODO(crbug.com/1047726) Implement dialog.
},
/** @private */ /** @private */
onMenuEditPasswordClick_() { onMenuEditPasswordClick_() {
this.$.moreActionsMenu.close(); this.$.moreActionsMenu.close();
......
...@@ -764,6 +764,7 @@ void AddAutofillStrings(content::WebUIDataSource* html_source, ...@@ -764,6 +764,7 @@ void AddAutofillStrings(content::WebUIDataSource* html_source,
{"noCompromisedCredentials", {"noCompromisedCredentials",
IDS_SETTINGS_NO_COMPROMISED_CREDENTIALS_LABEL}, IDS_SETTINGS_NO_COMPROMISED_CREDENTIALS_LABEL},
{"showCompromisedPassword", IDS_SETTINGS_COMPROMISED_PASSWORD_SHOW}, {"showCompromisedPassword", IDS_SETTINGS_COMPROMISED_PASSWORD_SHOW},
{"hideCompromisedPassword", IDS_SETTINGS_COMPROMISED_PASSWORD_HIDE},
{"editCompromisedPassword", IDS_SETTINGS_COMPROMISED_PASSWORD_EDIT}, {"editCompromisedPassword", IDS_SETTINGS_COMPROMISED_PASSWORD_EDIT},
{"removeCompromisedPassword", IDS_SETTINGS_COMPROMISED_PASSWORD_REMOVE}, {"removeCompromisedPassword", IDS_SETTINGS_COMPROMISED_PASSWORD_REMOVE},
{"removeCompromisedPasswordConfirmationTitle", {"removeCompromisedPasswordConfirmationTitle",
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment