Password shown in settings doesn't update with search |
|||||||||
Issue description
Chrome Version : 63.0.3223.8
OS Version: OS X 10.12.6
URLs (if applicable) :
Other browsers tested:
Add OK or FAIL after other browsers where you have tested this issue:
Safari 5:
Firefox 4.x:
IE 7/8/9:
What steps will reproduce the problem?
1. Save passwords for abc.com and xyz.com
2. Go to settings -> manage passwords, search for abc.com in "Search Passwords" field
3. Click show password, enter device password to reveal password for abc.com
4. Search for password for xyz.com
What is the expected result?
Should show the correct username and password for xyz.com
What happens instead of that?
Page shows the correct username, but the password shown is from the last search, abc.com. It doesn't update to show the password for xyz.com
Please provide any additional information below. Attach a screenshot if
possible.
UserAgentString: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/63.0.3223.8 Safari/537.36
,
Oct 5 2017
Able to reproduce the issue on reported version 63.0.3223.8 on Mac 10.12.6, working on the bisect
,
Oct 5 2017
,
Oct 6 2017
Able to reproduce the issue on reported version 63.0.3223.8, Latest Canary 63.0.3233.0 using Mac 10.12.6, Ubuntu 14.04 and Windows 10. This is a Non-Regression Issue as it is seen from M-62(62.0.3175.0) from the introduction of show password option on chrome://settings/passwords page, hence marking it as Untriaged.
,
Oct 6 2017
,
Oct 30 2017
Issue 779061 has been merged into this issue.
,
Oct 30 2017
Issue 779011 has been merged into this issue.
,
Nov 16 2017
Issue 782032 has been merged into this issue.
,
Nov 20 2017
,
Nov 27 2017
,
Nov 27 2017
Issue 788451 has been merged into this issue.
,
Nov 29 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/447990991a286e8085d04c41ef6c016dbdf25e0f commit 447990991a286e8085d04c41ef6c016dbdf25e0f Author: jdoerrie <jdoerrie@chromium.org> Date: Wed Nov 29 19:37:02 2017 Fix Behavior of Visible Passwords Prior to this change visible passwords were bound to a specific row in the list of passwords. This led to bugs when the list changed, e.g. when adding or deleting a password, or changing the list of passwords with a search filter. This change fixes this by grouping the password and the UiEntry into a newly introduced typedef. Bug: 786312 , 771126 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I25751d6a20b315290f2c52cc76946f85c413405f Reviewed-on: https://chromium-review.googlesource.com/779179 Reviewed-by: Hector Carmona <hcarmona@chromium.org> Reviewed-by: Devlin <rdevlin.cronin@chromium.org> Commit-Queue: Jan Wilken Dörrie <jdoerrie@chromium.org> Cr-Commit-Position: refs/heads/master@{#520200} [modify] https://crrev.com/447990991a286e8085d04c41ef6c016dbdf25e0f/chrome/browser/resources/settings/passwords_and_forms_page/password_edit_dialog.html [modify] https://crrev.com/447990991a286e8085d04c41ef6c016dbdf25e0f/chrome/browser/resources/settings/passwords_and_forms_page/password_list_item.html [modify] https://crrev.com/447990991a286e8085d04c41ef6c016dbdf25e0f/chrome/browser/resources/settings/passwords_and_forms_page/password_list_item.js [modify] https://crrev.com/447990991a286e8085d04c41ef6c016dbdf25e0f/chrome/browser/resources/settings/passwords_and_forms_page/passwords_section.html [modify] https://crrev.com/447990991a286e8085d04c41ef6c016dbdf25e0f/chrome/browser/resources/settings/passwords_and_forms_page/passwords_section.js [modify] https://crrev.com/447990991a286e8085d04c41ef6c016dbdf25e0f/chrome/browser/resources/settings/passwords_and_forms_page/show_password_behavior.js [modify] https://crrev.com/447990991a286e8085d04c41ef6c016dbdf25e0f/chrome/test/data/webui/settings/passwords_and_forms_browsertest.js [modify] https://crrev.com/447990991a286e8085d04c41ef6c016dbdf25e0f/chrome/test/data/webui/settings/settings_passwords_section_browsertest.js [modify] https://crrev.com/447990991a286e8085d04c41ef6c016dbdf25e0f/third_party/closure_compiler/externs/passwords_private.js
,
Dec 4 2017
Issue 788682 has been merged into this issue.
,
Dec 4 2017
,
Dec 6 2017
Issue 792425 has been merged into this issue.
,
Dec 26 2017
,
Jan 17 2018
Issue 803101 has been merged into this issue.
,
Jan 23 2018
Issue 804283 has been merged into this issue.
,
Jan 29 2018
Issue 806716 has been merged into this issue.
,
Jan 29 2018
The fix landed in 64.0.3281.0 |
|||||||||
►
Sign in to add a comment |
|||||||||
Comment 1 by nyerramilli@chromium.org
, Oct 4 2017