New issue
Advanced search Search tips

Issue 895785 link

Starred by 1 user
Status: Fixed
Owner:
jdoerrie@chromium.org
Closed: Oct 16
Cc:
nyerramilli@chromium.org
rbasuvula@chromium.org
dpa...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression:Unable to decrypt password even after entering valid credentials

Reported by pranjali...@etouch.net, Oct 16 
Chrome Version: 72.0.3582.0 (Official Build) db5d271436f9329304e394c13ed3fb573588c2de-refs/branch-heads/3582@{#1}(32/64 bit)

OS:  Win(7,8,8.1,10) , Mac(10.13.1 , 10.13.6 , 10.14.1)  and Linux(14.04 LTS)

Precondition: Atleast one saved password entry should be present in 'chrome://settings/passwords'.

What steps will reproduce the problem?
1.Launch chrome and navigate to 'chrome://settings/passwords'.
2.Click on eye icon of saved password such that authentication overlay will appear.
3.Now enter valid password and click on 'ok' button and observe.

Actual  : Unable to decrypt password even after entering valid credentials
Expected: Password should get decrypted after entering valid credentials.

This is Regression issue seen in M-72, and will soon update other info.
Good Build: 72.0.3581.0
Bad Build: 72.0.3582.0
Actual Result.mp4
289 KB View Download
Expected Result.mp4
277 KB View Download

Comment 1 by pranjali...@etouch.net, Oct 16

Labels: hasbisect
Owner: jdoerrie@chromium.org
 Unable to provide 'per-revision' bisect as it shows "We don't have enough builds to bisect . revlist: []" error message,Hence provided suspect through 'Chromium bisect'

Chromium bisect URL:
https://chromium.googlesource.com/chromium/src/+log/d04ab44adf9cc82ea2549f66c8a5b43a1aad34bc..484a185167ad7b93dde4613e4d883fc7504c6787

Suspect: https://chromium.googlesource.com/chromium/src/+/a0801f6becc4f7c1d8037d37e1ebb55aa38ae381	

@jdoerrie: Could you please check whether this is caused with respect to your change, if not please help us in assigning it to the right owner.

Thank You

Comment 2 by jdoerrie@google.com, Oct 16 

Fix is in https://crrev.com/c/1283129.

Comment 3 by rbasuvula@chromium.org, Oct 16

Labels: ReleaseBlock-Stable
Adding release blocker label for this issue.Please reduce priority or remove if not the case.

Thank You!
Project Member

Comment 4 by bugdroid1@chromium.org, Oct 16 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/1913a62d9bc634c9db0416103c0cc701aeda710e

commit 1913a62d9bc634c9db0416103c0cc701aeda710e
Author: Jan Wilken Doerrie <jdoerrie@chromium.org>
Date: Tue Oct 16 17:09:02 2018

[Password][Settings] Rename index to id in password_manager_proxy.js

This change is a follow-up to r599728 that renamed index to id in
passwords_private.idl. That change missed to update
password_manager_proxy.js, which is fixed by this change.

Bug:  892260 ,  895785 
Change-Id: Id1ae5e6e46f8dd66c51bb01988b537ec99a1975c
Reviewed-on: https://chromium-review.googlesource.com/c/1283129
Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
Commit-Queue: Jan Wilken Dörrie <jdoerrie@chromium.org>
Cr-Commit-Position: refs/heads/master@{#600024}
[modify] https://crrev.com/1913a62d9bc634c9db0416103c0cc701aeda710e/chrome/browser/resources/settings/passwords_and_forms_page/password_manager_proxy.js

Comment 5 by jdoerrie@google.com, Oct 16

Status: Fixed (was: Assigned)

Comment 6 by pranjali...@etouch.net, Oct 17

Labels: TE-Verified-M72 TE-Verified-72.0.3583.0
Update:

Rechecked the above issue on Win(7,8,8.1,10) , Mac(10.13.1 , 10.13.6 , 10.14.1)  and Linux(14.04 LTS) using latest canary build#72.0.3583.0 and issue is fixed.

Please refer attached screen cast.

Thank You...
Canary Behaviour.mp4
251 KB View Download
►

Sign in to add a comment