Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(25)

Unified Diff: chrome/browser/ui/views/profiles/user_manager_view.h

Issue 2552473002: Display error message when user try to open a locked supervised user profile when force-sign-in is … (Closed)
Patch Set: refactor Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/views/profiles/user_manager_view.h
diff --git a/chrome/browser/ui/views/profiles/user_manager_view.h b/chrome/browser/ui/views/profiles/user_manager_view.h
index 5ec3fd9436c08fc573f8d7912d42b16abe8f115b..3a0e032d8d0e09e8cedbd043e8df9a14bef45531 100644
--- a/chrome/browser/ui/views/profiles/user_manager_view.h
+++ b/chrome/browser/ui/views/profiles/user_manager_view.h
@@ -19,13 +19,14 @@
class ScopedKeepAlive;
class UserManagerView;
+// TODO(zmin): Rename the delgate as it's not only used to reauth now.
class ReauthDelegate : public views::DialogDelegateView,
public UserManager::BaseReauthDialogDelegate {
public:
ReauthDelegate(UserManagerView* parent,
views::WebView* web_view,
const std::string& email_address,
- signin_metrics::Reason reason);
+ const GURL& url);
~ReauthDelegate() override;
// UserManager::BaseReauthDialogDelegate:
@@ -85,20 +86,14 @@ class UserManagerView : public views::DialogDelegateView {
// Logs how long it took the UserManager to open.
void LogTimeToOpen();
- // Shows a dialog where the user can re-authenticate the profile with the
- // given |email|. This is called in the following scenarios:
- // -From the user manager when a profile is locked and the user's password is
- // detected to have been changed.
- // -From the user manager when a custodian account needs to be
- // reauthenticated.
- // reason| can be REASON_UNLOCK or REASON_REAUTHENTICATION to indicate
- // whether this is a reauth or unlock scenario.
- void ShowReauthDialog(content::BrowserContext* browser_context,
- const std::string& email,
- signin_metrics::Reason reason);
-
// Hides the reauth dialog if it is showing.
- void HideReauthDialog();
+ void HideDialog();
+
+ // Show a dialog where the user can auth the profile or see the auth error
+ // message.
+ void ShowDialog(content::BrowserContext* browser_context,
+ const std::string& email,
+ const GURL& url);
// Display sign in error message that is created by Chrome but not GAIA
// without browser window.

Powered by Google App Engine
This is Rietveld 408576698