My favorites | Sign in
Project Home Downloads Wiki Issues Code Search
New issue   Search
for
  Advanced search   Search tips   Subscriptions
Issue 161360: Error message doesn't close after successful sign-in
3 people starred this issue and may be notified of changes. Back to list
 
Project Member Reported by sfir...@chromium.org, Nov 15, 2012
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
Nov 15, 2012
#1 joaodasilva@chromium.org
(No comment was entered for this change.)
Cc: nkostylev@chromium.org ivankr@chromium.org
Nov 15, 2012
#2 scunning...@chromium.org
(No comment was entered for this change.)
Summary: Error message doesn't close after successful sign-in
Labels: -Feature-Enterprise Feature-SignIn Enterprise Sev-2
Nov 16, 2012
#4 nkostylev@chromium.org
Thanks for the fix!
Status: Started
Owner: joaodasilva@chromium.org
Nov 16, 2012
#5 bugdro...@chromium.org
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
------------------------------------------------------------------------
Nov 16, 2012
#6 joaodasilva@chromium.org
Requesting a merge. This is not a security nor stability issue, it just looks bad :-) See screenshot above.
Labels: Merge-Requested
Dec 3, 2012
#7 kar...@google.com
(No comment was entered for this change.)
Labels: -Merge-Requested Merge-Approved
Dec 3, 2012
#8 joaodasilva@chromium.org
Merged in http://crrev.com/170837.
Status: Fixed
Labels: -Merge-Approved Merge-Merged
Dec 3, 2012
#9 bugdro...@chromium.org
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
------------------------------------------------------------------------
Labels: merge-merged-1271
Dec 21, 2012
#10 sfir...@chromium.org
no issue on Daisy 3447.0.0\26.0.1366.0
Status: Verified
Mar 10, 2013
#11 bugdro...@chromium.org
(No comment was entered for this change.)
Labels: -Area-UI -Mstone-23 -Feature-SignIn -Enterprise Cr-UI M-23 Cr-Services-SignIn Cr-Enterprise
Sign in to add a comment

Powered by Google Project Hosting