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

Issue 664925 link

Starred by 0 users

Issue metadata

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



Sign in to add a comment

Regression: In Switch person window, warning image is seen white in color.

Reported by lpa...@etouch.net, Nov 14 2016

Issue description

Version: 56.0.2919.0 (Official Build) 9fd27ddeb13974b24cfa42261b940f8de378117a-refs/heads/master@{#431801} (32/64-bit)
OS: Windows (7,8,10), Mac (10.11.6, 10.12.1), Linux (14.04 LTS)

Precondition: Sign in to chrome.

What steps will reproduce the problem?
1) Launch chrome, go to chrome://settings, from 'People' section add/import a supervised user.
2) Switch to parent user, click on avatar icon and select 'Exit and childlock' option (Switch person window opens).
3) Click on locked profile and enter the valid password (Parent user gets unlocked).
4) Now again click on avatar icon and select 'Exit and childlock' option.
5) In locked profile enter invalid password and observe the Warning image.

Warning image is seen in white color.

Warning image should be seen in yellow color.

This is a Regression issue broken in M-56, will soon update other info
Manual bisect:
Good build: 56.0.2909.0 
Bad build: 56.0.2912.0 
 
warning_actual.mov
5.0 MB Download
warning_expected.mp4
1.5 MB View Download

Comment 1 by lpa...@etouch.net, Nov 14 2016

Owner: mahmadi@chromium.org
Status: Assigned (was: Unconfirmed)
Note: Above issue is not reproducible in Chromium builds as unable to Sign in to chrome. Hence providing suspect from Changelog URL.

Changelog URL:
https://chromium.googlesource.com/chromium/src/+log/56.0.2909.0..56.0.2912.0?pretty=fuller&n=10000

Suspecting: r430023
I don't see anything wrong with the color of the icon. Could you please send a screenshot of how it used to look like in a good build?

Comment 3 by lpa...@etouch.net, Nov 15 2016

With response to comment #2: Please refer the attached screenshot for good build behaviour.
warning.jpg
141 KB View Download
Cc: mahmadi@chromium.org
Owner: alemate@chromium.org
alemate@, this is a result of your change: https://codereview.chromium.org/2467783002
I personally don't favor one color over the other. Is the white color intended though?
Cc: rbasuvula@chromium.org
Just for update, Still able to reproduce the issue on Ubuntu 14.04,Mac 10.11.6 using latest chrome version 57.0.2935.0.

alemate@ Could you please look into this issue.

Thanks!
Just for update, Still able to reproduce the issue on Ubuntu 14.04,Mac 10.11.6 using latest chrome version 57.0.2956.0.

alemate @ Could you please look into this issue.

Thanks!
Just to update, Still able to reproduce the issue on win10 using latest chrome canary version 57.0.2969.0

alemate @ Could you please look into this issue.
Still able to reproduce the issue on Ubuntu 14.04 using latest chrome version 57.0.2984.0.

alemate @ Could you please look into this issue.

Thanks!
Just to update, still able to reproduce this issue on mac 10.12.2 using latest canary #58.0.2990.0.

alemate@ - Gentle Ping...!!

Could you please have a look into this issue.

Thanks...!!
Project Member

Comment 10 by sheriffbot@chromium.org, Feb 23 2017

Status: Available (was: Assigned)
--Chrome Identity automated triaging--

This bug is Assigned and has gone one month without any activity, so it is being moved to Available to indicate that it is not actively being worked on. If you are working on this bug, please mark yourself as the owner and move back to Assigned. Please see https://goo.gl/78kbny for more details. Please remove the Services>SignIn or UI>Browser>Profiles components if this bug isn't related to Chrome Identity.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Status: Started (was: Available)
Status: Fixed (was: Started)
This is fixed in https://codereview.chromium.org/2727583002 . (I incorrectly attributed CL to a different issue).

Sign in to add a comment