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

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

Issue 2151513002: Revamped signin/sync error surfacing on desktop user menu (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addressed rogerta's last comment Created 4 years, 5 months 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/profile_chooser_view.h
diff --git a/chrome/browser/ui/views/profiles/profile_chooser_view.h b/chrome/browser/ui/views/profiles/profile_chooser_view.h
index 40db9078441c31592ed7b3800988c8c42aa73a23..11682346cfac0e420af62ab4463071bc0c2d6bae 100644
--- a/chrome/browser/ui/views/profiles/profile_chooser_view.h
+++ b/chrome/browser/ui/views/profiles/profile_chooser_view.h
@@ -211,6 +211,12 @@ class ProfileChooserView : public content::WebContentsDelegate,
views::LabelButton** button,
views::ImageButton** close_button);
+ // Creates a header for signin and sync error surfacing for the user menu.
+ views::View* CreateSyncErrorViewIfNeeded();
+ views::View* CreateSyncErrorView(const int content_string_id,
+ const int button_string_id,
+ views::LabelButton** button_out);
+
// Create a view that shows various options for an upgrade user who is not
// the same person as the currently signed in user.
views::View* CreateSwitchUserView();
@@ -238,6 +244,13 @@ class ProfileChooserView : public content::WebContentsDelegate,
views::Link* tutorial_learn_more_link_;
views::ImageButton* tutorial_close_button_;
+ // Buttons in the signin/sync error header on top of the desktop user menu.
+ views::LabelButton* sync_error_signin_button_;
+ views::LabelButton* sync_error_passphrase_button_;
+ views::LabelButton* sync_error_upgrade_button_;
+ views::LabelButton* sync_error_signin_again_button_;
+ views::LabelButton* sync_error_signout_button_;
+
// Links and buttons displayed in the active profile card.
views::Link* manage_accounts_link_;
views::LabelButton* manage_accounts_button_;
« no previous file with comments | « chrome/browser/ui/views/profiles/new_avatar_button.cc ('k') | chrome/browser/ui/views/profiles/profile_chooser_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698