| Index: chrome/browser/ui/views/profiles/profile_chooser_view.cc
|
| diff --git a/chrome/browser/ui/views/profiles/profile_chooser_view.cc b/chrome/browser/ui/views/profiles/profile_chooser_view.cc
|
| index 539491f322de4e84dffb23e766f33862bc3c6ed8..37fdd626fc9b81d6c019ffe5eb02e5d412a29468 100644
|
| --- a/chrome/browser/ui/views/profiles/profile_chooser_view.cc
|
| +++ b/chrome/browser/ui/views/profiles/profile_chooser_view.cc
|
| @@ -115,8 +115,7 @@ const int kVerticalSpacing = 16;
|
| const int kTitleViewNativeWidgetOffset = 8;
|
|
|
| bool IsProfileChooser(profiles::BubbleViewMode mode) {
|
| - return mode == profiles::BUBBLE_VIEW_MODE_PROFILE_CHOOSER ||
|
| - mode == profiles::BUBBLE_VIEW_MODE_FAST_PROFILE_CHOOSER;
|
| + return mode == profiles::BUBBLE_VIEW_MODE_PROFILE_CHOOSER;
|
| }
|
|
|
| // DEPRECATED: New user menu components should use views::BoxLayout instead.
|
| @@ -533,14 +532,6 @@ void ProfileChooserView::ShowBubble(
|
| views::View* anchor_view,
|
| Browser* browser,
|
| bool is_source_keyboard) {
|
| - // Don't start creating the view if it would be an empty fast user switcher.
|
| - // It has to happen here to prevent the view system from creating an empty
|
| - // container.
|
| - // Same for material design user menu since fast profile switcher will be
|
| - // migrated to the left-click menu.
|
| - if (view_mode == profiles::BUBBLE_VIEW_MODE_FAST_PROFILE_CHOOSER)
|
| - return;
|
| -
|
| if (IsShowing()) {
|
| if (tutorial_mode != profiles::TUTORIAL_MODE_NONE) {
|
| profile_bubble_->tutorial_mode_ = tutorial_mode;
|
| @@ -759,7 +750,6 @@ void ProfileChooserView::ShowView(profiles::BubbleViewMode view_to_display,
|
| break;
|
| case profiles::BUBBLE_VIEW_MODE_ACCOUNT_MANAGEMENT:
|
| case profiles::BUBBLE_VIEW_MODE_PROFILE_CHOOSER:
|
| - case profiles::BUBBLE_VIEW_MODE_FAST_PROFILE_CHOOSER:
|
| layout = CreateSingleColumnLayout(this, kFixedMenuWidth);
|
| sub_view = CreateProfileChooserView(avatar_menu);
|
| break;
|
| @@ -1003,9 +993,10 @@ void ProfileChooserView::StyledLabelLinkClicked(views::StyledLabel* label,
|
| chrome::ShowSettings(browser_);
|
| }
|
|
|
| -void ProfileChooserView::PopulateCompleteProfileChooserView(
|
| - views::GridLayout* layout,
|
| +views::View* ProfileChooserView::CreateProfileChooserView(
|
| AvatarMenu* avatar_menu) {
|
| + views::View* view = new views::View();
|
| + views::GridLayout* layout = CreateSingleColumnLayout(view, kFixedMenuWidth);
|
| // Separate items into active and alternatives.
|
| Indexes other_profiles;
|
| views::View* tutorial_view = NULL;
|
| @@ -1072,38 +1063,6 @@ void ProfileChooserView::PopulateCompleteProfileChooserView(
|
| layout->StartRow(0, 0);
|
| layout->AddView(option_buttons_view);
|
| }
|
| -}
|
| -
|
| -void ProfileChooserView::PopulateMinimalProfileChooserView(
|
| - views::GridLayout* layout,
|
| - AvatarMenu* avatar_menu) {
|
| - Indexes other_profiles;
|
| - for (size_t i = 0; i < avatar_menu->GetNumberOfItems(); ++i) {
|
| - const AvatarMenu::Item& item = avatar_menu->GetItemAt(i);
|
| - if (!item.active) {
|
| - other_profiles.push_back(i);
|
| - }
|
| - }
|
| -
|
| - layout->StartRow(1, 0);
|
| - layout->AddView(CreateOtherProfilesView(other_profiles));
|
| -}
|
| -
|
| -views::View* ProfileChooserView::CreateProfileChooserView(
|
| - AvatarMenu* avatar_menu) {
|
| - views::View* view = new views::View();
|
| - views::GridLayout* layout = CreateSingleColumnLayout(view, kFixedMenuWidth);
|
| -
|
| - if (view_mode_ == profiles::BUBBLE_VIEW_MODE_FAST_PROFILE_CHOOSER) {
|
| - PopulateMinimalProfileChooserView(layout, avatar_menu);
|
| - // The user is using right-click switching, no need to tell them about it.
|
| - PrefService* local_state = g_browser_process->local_state();
|
| - local_state->SetBoolean(
|
| - prefs::kProfileAvatarRightClickTutorialDismissed, true);
|
| - } else {
|
| - PopulateCompleteProfileChooserView(layout, avatar_menu);
|
| - }
|
| -
|
| return view;
|
| }
|
|
|
|
|