New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 753688 link

Starred by 3 users
Status: Fixed
Owner:
wzang@chromium.org
Closed: Aug 2017
Cc:
nyerramilli@chromium.org
rbasuvula@chromium.org
█ msrchandra@chromium.org
jmukthavaram@chromium.org
█ ranjitkan@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression: Unnecessary Password text field is seen for User in 'Manage other people' window

Reported by nutan.ga...@etouch.net, Aug 9 2017 
Chrome Version: 62.0.3180.0 281632c0a2cc807f0334db4dea6b4bb6513f763f-refs/heads/master@{#492769}
OS: Mac(10.11.6, 10.12.3,10.12.5),Windows(7,8,10),Linux(14.04 LTS).

What steps will reproduce the problem?
1. Launch chrome, navigate to chrome://settings/people and Click on 'Manage other people' 
2. Click on 3 dot icon of User(Remove this person overlay open)
3. Observe behind the overlay

Actual: Unnecessary Password text field is seen for User
Expected: Password text field should not be seen

This is Regression issue broken in M-62 and will soon update other info.

Manual Bisect Info:
Good Build: 62.0.3179.0
Bad Build: 62.0.3180.0
Actual Result.mov
2.1 MB Download

Comment 1 by jmukthavaram@chromium.org, Aug 9 2017

Cc: jmukthavaram@chromium.org
Labels: hasbisect-per-revision ReleaseBlock-Stable
Owner: wzang@chromium.org
Status: Assigned (was: Unconfirmed)
Able to reproduce the issue on Mac 10.12.6,Windows 7 & Ubuntu 14.04 Using latest Canary-62.0.3179.0.
Manual Bisect info:
—————————---------
Good Build: 62.0.3179.0-Revision-492477
Bad Build: 62.0.3180.0-Revision-492769

Per revision bisect info:
———————————-------------
You are probably looking for a change made after 492560 (known good), but no later than 492561 (first known bad).
CHANGELOG URL:
The script might not always return single CL as suspect as some perf builds might get missing due to failure.
https://chromium.googlesource.com/chromium/src/+log/e06d40f52c4a69790bfb31deaea6595b8443fd2c..bad25d79920ef48c94769cbccf12d93d6e1a8e46

Possible suspect:
----------------
https://chromium.googlesource.com/chromium/src/+/bad25d79920ef48c94769cbccf12d93d6e1a8e46

wzang@Could you please take a look & reassign to the respective owner if it is not related to your change.

Since it is regressed recently, adding Release block stable label.

Thanks..!!

Comment 2 by wzang@chromium.org, Aug 9 2017

Status: Started (was: Assigned)

Comment 3 by ligim...@chromium.org, Aug 21 2017 

Your bug is tagged as Release block Stable. 

M62 is branching soon and we will be taking only CRITICAL merges. Please plan accordingly.
Project Member

Comment 4 by bugdroid1@chromium.org, Aug 24 2017 

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

commit 216133d4d7287a0cae0dc3a412157770e740c3f5
Author: Wenzhao Zang <wzang@chromium.org>
Date: Thu Aug 24 23:45:30 2017

Remove unnecessary password container

Bug:  753688 
Change-Id: I5ee2809c3705b55965507cb5f12a79b6f214bd6f
Reviewed-on: https://chromium-review.googlesource.com/627662
Reviewed-by: Alexander Alekseev <alemate@chromium.org>
Commit-Queue: Wenzhao (Colin) Zang <wzang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#497250}
[modify] https://crrev.com/216133d4d7287a0cae0dc3a412157770e740c3f5/ui/login/account_picker/user_pod_row.css

Comment 5 by wzang@chromium.org, Aug 24 2017 

Could you please verify if it is resolved? (The change is in ToT now and may land in the latest Canary in a day or two) If the issue persists it shouldn't be related to the change in #1. Thanks!

Comment 6 by msrchandra@chromium.org, Aug 25 2017

Labels: TE-Verified-M62 TE-Verified-62.0.3196.0
Tested the issue on Latest Canary# 62.0.3196.0 on Windows, Mac and Linux and found the issue to be fixed. Attaching a screen shot for reference.
Hence adding TE-Verified Labels.
Thank You.

Comment 7 by msrchandra@chromium.org, Aug 25 2017

753688.png
19.4 KB View Download

Comment 8 by wzang@chromium.org, Aug 25 2017 

The CL in #1 has been merged to M61, so do we need to merge the fix to M61 as well? Presumably M61 still has this issue.

Comment 9 by wzang@chromium.org, Aug 25 2017

Status: Fixed (was: Started)
►

Sign in to add a comment