Project: chromium Issues People Development process History Sign in
New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.
Issue 161360 Error message doesn't close after successful sign-in
Starred by 3 users Reported by sfirsov@chromium.org, Nov 15 2012 Back to list
Status: Verified
Owner:
Last visit 19 days ago
Closed: Dec 2012
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment
Version: 23.0.1271.91
OS: 2913.196.0

What steps will reproduce the problem?
1.Enroll Stout to domain
2.Sing-in user not from white list
3.Sing-in user from white list

What is the expected output? 
error message should close after succeed sing-in

What do you see instead?
error message still exist
 
Screenshot-R23.png
947 KB View Download
Cc: nkostylev@chromium.org ivankr@chromium.org
Labels: -Feature-Enterprise Feature-SignIn Enterprise Sev-2
Summary: Error message doesn't close after successful sign-in (was: NULL)
Owner: joaodasilva@chromium.org
Status: Started
Thanks for the fix!
Project Member Comment 5 by bugdroid1@chromium.org, Nov 16 2012
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=168235

------------------------------------------------------------------------
r168235 | joaodasilva@chromium.org | 2012-11-16T17:34:58.161420Z

Changed paths:
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/chromeos/settings/stub_cros_settings_provider.cc?r1=168235&r2=168234&pathrev=168235
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/resources/chromeos/login/screen_gaia_signin.js?r1=168235&r2=168234&pathrev=168235

Hide error bubbles after a successful sign-in.

This hides the user-not-whitelisted error from a previous login, for example.

BUG= 161360 


Review URL: https://chromiumcodereview.appspot.com/11316048
------------------------------------------------------------------------
Labels: Merge-Requested
Requesting a merge. This is not a security nor stability issue, it just looks bad :-) See screenshot above.
Comment 7 by kareng@google.com, Dec 3 2012
Labels: -Merge-Requested Merge-Approved
Labels: -Merge-Approved Merge-Merged
Status: Fixed
Merged in http://crrev.com/170837.
Project Member Comment 9 by bugdroid1@chromium.org, Dec 3 2012
Labels: merge-merged-1271
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=170837

------------------------------------------------------------------------
r170837 | joaodasilva@chromium.org | 2012-12-03T23:13:25.466772Z

Changed paths:
   M http://src.chromium.org/viewvc/chrome/branches/1271/src/chrome/browser/chromeos/settings/stub_cros_settings_provider.cc?r1=170837&r2=170836&pathrev=170837
   M http://src.chromium.org/viewvc/chrome/branches/1271/src/chrome/browser/resources/chromeos/login/screen_gaia_signin.js?r1=170837&r2=170836&pathrev=170837

Merge 168235
> Hide error bubbles after a successful sign-in.
> 
> This hides the user-not-whitelisted error from a previous login, for example.
> 
> BUG= 161360 
> 
> 
> Review URL: https://chromiumcodereview.appspot.com/11316048

TBR=joaodasilva@chromium.org
Review URL: https://codereview.chromium.org/11411336
------------------------------------------------------------------------
Status: Verified
no issue on Daisy 3447.0.0\26.0.1366.0
Project Member Comment 11 by bugdroid1@chromium.org, Mar 10 2013
Labels: -Area-UI -Mstone-23 -Feature-SignIn -Enterprise Cr-UI M-23 Cr-Services-SignIn Cr-Enterprise
Sign in to add a comment