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

Issue 604293 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner: ----
Closed: May 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression:Added users are not displayed in switch person overlay.

Reported by vku...@etouch.net, Apr 18 2016

Issue description

Chrome Version: 52.0.2711.0 (Official Build)Revision 2f8c7fcf0ca7d22c7ef943ea5e3256914bef70f1-refs/heads/master@{#387833} (32/64-bit)
OS: All, Win 7(Aero enabled)

What steps will reproduce the problem?
1.Freshly launch chrome and click on switch person from avatar icon
2.Click on Add person,select any avatar icon and click on save button, observe the switch person overlay.

Actual: Added users are not displayed in switch person overlay.
Expected: Added users should be displayed in switch person overlay.

This is a regression issue broken in 'M52' and will soon update other info
 

Comment 1 by vku...@etouch.net, Apr 18 2016

Labels: hasbisect
Owner: jam@chromium.org
Status: Assigned (was: Unconfirmed)
Correction: 
Precondition: Make sure 'Material Design User Manager' flag is enable for new UI of switch user overlay.

Manual regression range:
Good Build: 52.0.2709.0
Bad Build:  52.0.2710.0

CL:
https://chromium.googlesource.com/chromium/src/+log/52.0.2709.0..52.0.2710.0?pretty=fuller&n=10000
(Unable to narrow down the range since getting all good chromium builds between this range via tool)

Suspecting: 387687 ?
Kindly help to re-assign, if your changes are not cause for this issue.

Actual_Switch.mov
1.2 MB Download
Expected_Switch.mov
1.3 MB Download
Labels: ReleaseBlock-Stable
adding RB-label, please change if required.
Cc: mahmadi@chromium.org
The Material Design User Manager is still under development and is behind 'Material Design User Manager' flag. This particular bug should be fixed once
https://codereview.chromium.org/1642323004/ lands

Comment 5 by jam@chromium.org, Apr 18 2016

Owner: vku...@etouch.net

Comment 6 by vku...@etouch.net, Apr 20 2016

Owner: ----
With response to comment #4
We'll wait for the CL to land in the canary.
Cc: tkonch...@chromium.org
Labels: TE-Verified-M52 TE-Verified-52.0.2730.0
Status: Fixed (was: Assigned)
Tested the same on win7, mac 10.11 and Linux 14.04 chrome version 52.0.2730.0 - Added users are displayed in switch person overlay.

Please find the screenshot

Fix works as expected
Screen Shot 2016-05-10 at 12.24.15 PM.png
137 KB View Download
Labels: Merge-TBD
[Auto-generated comment by a script] We noticed that this issue is targeted for M-52; it appears the fix may have landed after branch point, meaning a merge might be required. Please confirm if a merge is required here - if so add Merge-Request-52 label, otherwise remove Merge-TBD label. Thanks.
Labels: -Merge-TBD
[Bulk edit]

Our blockerbot script was offline; it was recently brought back online, and thus labeled many old issues (including this one) erroneously.  Removing Merge-TBD label since all milestones for this issue are already completed; no further work should be done.

Sign in to add a comment