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

Issue 641855 link

Starred by 2 users

Issue metadata

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



Sign in to add a comment

Regression : Error Message Text is not visible in User Manager of Parent Profile

Project Member Reported by mm00333...@techmahindra.com, Aug 29 2016

Issue description

Version: 55.0.2843.0
OS: Ubuntu 14.04,Windows

What steps will reproduce the problem?
1.Launch chrome -> Sign into Profile
2.Now Create/import supervised user -> Then click on avatar Menu and choose option "Exit and childlock" (For First time Sign in Page is seen so enter correct password then profile gets unlocked and now again choose option "Exit and childlock" )
3.Now Switch person window is displayed -> Click on Locked profile -> Enter Wrong password -> Now observe Error Message Text 

Expected:  Error Message Text  should be visible in User Manager of Parent Profile
Actual:  Instead Error Message Text  is not visible 


This is Regression Issue broken in M-55
will provide bisect info soon

 

Comment 1 by ajha@chromium.org, Aug 29 2016

Labels: ReleaseBlock-Stable OS-Mac
Status: Untriaged (was: Unconfirmed)
Able to reproduce the issue on Mac OS 10.11.6, chrome version: 55.0.2842.0 as well.
Labels: -Needs-Bisect
Owner: varkha@chromium.org
Status: Assigned (was: Untriaged)
Manual good and Bad Builds:
Good Build: 55.0.2841.0
Bad Build:  55.0.2842.0

Note : Unable to provide narrow bisect as sign in to chrome in chromium builds is not possible, hence providing Manual change log info.

MANUAL CHANGELOG URL:
https://chromium.googlesource.com/chromium/src/+log/55.0.2841.0..55.0.2842.0?pretty=fuller&n=10000

Suspecting https://codereview.chromium.org/2283583003 from Manual Changelog

@varkha : Please feel free to re-assign if its not related to your change

Actual_Message.png
488 KB View Download
Expected_supervised error.png
134 KB View Download

Comment 3 by varkha@chromium.org, Aug 29 2016

#2, thanks, yes indeed. I have a draft CL at https://codereview.chromium.org/2286403003 that corrects that.
Project Member

Comment 4 by bugdroid1@chromium.org, Aug 30 2016

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

commit 292580044e75d7104f9fbf53bdd374143242ab2b
Author: varkha <varkha@chromium.org>
Date: Tue Aug 30 06:04:38 2016

[ash-md] Corrects a color in user manager error message

After https://codereview.chromium.org/2283583003 the text color (dark)
no longer matches the background (dark). Fix that by removing the text
color override for the login manager warnings.

BUG= 641855 
TEST=with the steps in the bug the color scheme for warnings should
     match the tooltips in web-ui (0.87 alpha white on 0.8 opacity
     black).
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2286403003
Cr-Commit-Position: refs/heads/master@{#415197}

[modify] https://crrev.com/292580044e75d7104f9fbf53bdd374143242ab2b/chrome/browser/resources/md_user_manager/user_manager.html

Comment 5 by varkha@chromium.org, Aug 30 2016

Status: Fixed (was: Assigned)

Sign in to add a comment