LabelButton: cache the last computed preferred size

chrome/browser/ui/views/profiles/profile_chooser_view.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/views/profiles/profile_chooser_view.h"
 
 #include "base/prefs/pref_service.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/lifetime/application_lifetime.h"
@@ skipping 131 matching lines @@
 };
 
 BackgroundColorHoverButton::BackgroundColorHoverButton(
     views::ButtonListener* listener,
     const base::string16& text,
     const gfx::ImageSkia& normal_icon,
     const gfx::ImageSkia& hover_icon)
     : views::LabelButton(listener, text) {
   SetBorder(views::Border::CreateEmptyBorder(0, views::kButtonHEdgeMarginNew,
                                              0, views::kButtonHEdgeMarginNew));
-  set_min_size(gfx::Size(0, kButtonHeight));
+  SetMinSize(gfx::Size(0, kButtonHeight));
   SetImage(STATE_NORMAL, normal_icon);
   SetImage(STATE_HOVERED, hover_icon);
   SetImage(STATE_PRESSED, hover_icon);
 }
 
 BackgroundColorHoverButton::~BackgroundColorHoverButton() {}
 
 void BackgroundColorHoverButton::OnPaint(gfx::Canvas* canvas) {
   if ((state() == STATE_PRESSED) || (state() == STATE_HOVERED) || HasFocus()) {
     canvas->DrawColor(GetNativeTheme()->GetSystemColor(
@@ skipping 997 matching lines @@
     const int kSmallImageSide = 32;
 
     gfx::Image image = profiles::GetSizedAvatarIcon(
         item.icon, true, kSmallImageSide, kSmallImageSide);
 
     views::LabelButton* button = new BackgroundColorHoverButton(
         this,
         item.name,
         *image.ToImageSkia(),
         *image.ToImageSkia());
-    button->set_min_size(gfx::Size(
+    button->SetMinSize(gfx::Size(
         0, kButtonHeight + views::kRelatedControlVerticalSpacing));
 
     open_other_profile_indexes_map_[button] = index;
 
     layout->StartRow(1, 0);
     layout->AddView(new views::Separator(views::Separator::HORIZONTAL));
     layout->StartRow(1, 0);
     layout->AddView(button);
   }
 
@@ skipping 29 matching lines @@
       l10n_util::GetStringUTF16(IDS_PROFILES_EXIT_GUEST) :
       l10n_util::GetStringFUTF16(IDS_PROFILES_NOT_YOU_BUTTON,
           profiles::GetAvatarNameForProfile(
               browser_->profile()->GetPath()));
   ui::ResourceBundle* rb = &ui::ResourceBundle::GetSharedInstance();
   users_button_ = new BackgroundColorHoverButton(
       this,
       text,
       *rb->GetImageSkiaNamed(IDR_ICON_PROFILES_MENU_AVATAR),
       *rb->GetImageSkiaNamed(IDR_ICON_PROFILES_MENU_AVATAR));
-  users_button_->set_min_size(gfx::Size(
+  users_button_->SetMinSize(gfx::Size(
       0, kButtonHeight + views::kRelatedControlVerticalSpacing));
 
   layout->StartRow(1, 0);
   layout->AddView(users_button_);
 
   if (enable_lock) {
     lock_button_ = new BackgroundColorHoverButton(
         this,
         base::string16(),
         *rb->GetImageSkiaNamed(IDR_ICON_PROFILES_MENU_LOCK),
         *rb->GetImageSkiaNamed(IDR_ICON_PROFILES_MENU_LOCK));
-    lock_button_->set_min_size(gfx::Size(
+    lock_button_->SetMinSize(gfx::Size(
       0, kButtonHeight + views::kRelatedControlVerticalSpacing));
     layout->AddView(lock_button_);
   }
   return view;
 }
 
 views::View* ProfileChooserView::CreateSupervisedUserDisclaimerView() {
   views::View* view = new views::View();
   views::GridLayout* layout = CreateSingleColumnLayout(
       view, kFixedMenuWidth - 2 * views::kButtonHEdgeMarginNew);
@@ skipping 250 matching lines @@
   layout->StartRowWithPadding(
       1, 0, 0, views::kUnrelatedControlVerticalSpacing);
   layout->AddView(end_preview_and_relaunch_button_);
 
   TitleCard* title_card = new TitleCard(
       IDS_PROFILES_END_PREVIEW, this, &end_preview_cancel_button_);
   return TitleCard::AddPaddedTitleCard(
       view, title_card, kFixedAccountRemovalViewWidth);
 }