Chromium Code Reviews| Index: chrome/browser/ui/webui/signin/signin_error_handler.cc |
| diff --git a/chrome/browser/ui/webui/signin/signin_error_handler.cc b/chrome/browser/ui/webui/signin/signin_error_handler.cc |
| index 26b9d8a4f0090cfe2eede730ffd3bdf057f62364..93fc7a32eae28cfaea18a1b2b6bf06971c69f580 100644 |
| --- a/chrome/browser/ui/webui/signin/signin_error_handler.cc |
| +++ b/chrome/browser/ui/webui/signin/signin_error_handler.cc |
| @@ -10,10 +10,14 @@ |
| #include "chrome/browser/profiles/profile_window.h" |
| #include "chrome/browser/signin/signin_ui_util.h" |
| #include "chrome/browser/ui/browser_window.h" |
| +#include "chrome/browser/ui/user_manager.h" |
| #include "chrome/browser/ui/webui/signin/signin_utils.h" |
| #include "content/public/browser/web_ui.h" |
| #include "url/gurl.h" |
| +SigninErrorHandler::SigninErrorHandler(bool is_system_profile) |
| + : is_system_profile_(is_system_profile) {} |
| + |
| void SigninErrorHandler::RegisterMessages() { |
| web_ui()->RegisterMessageCallback( |
| "confirm", |
| @@ -70,7 +74,11 @@ void SigninErrorHandler::HandleInitializedWithSize( |
| } |
| void SigninErrorHandler::CloseDialog() { |
| - Browser* browser = signin::GetDesktopBrowser(web_ui()); |
| - DCHECK(browser); |
| - browser->CloseModalSigninWindow(); |
| + if (is_system_profile_) { |
| + UserManager::HideReauthDialog(); |
| + } else { |
| + Browser* browser = signin::GetDesktopBrowser(web_ui()); |
| + DCHECK(browser); |
| + browser->CloseModalSigninWindow(); |
| + } |
|
Roger Tawa OOO till Jul 10th
2016/09/22 18:59:13
Can you add a comment why this change is needed?
zmin
2016/09/22 19:14:56
With the old CloseDialog logic, the user manager w
Roger Tawa OOO till Jul 10th
2016/09/22 19:27:20
Cool. Can you add a comment to the code explainin
zmin
2016/09/22 19:30:56
Done.
|
| } |