views: Switch Checkbox over to LabelButton.

ui/views/controls/button/label_button.cc

 // Copyright (c) 2012 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 "ui/views/controls/button/label_button.h"
 
 #include "base/logging.h"
 #include "grit/ui_resources.h"
 #include "ui/base/animation/throb_animation.h"
 #include "ui/base/resource/resource_bundle.h"
@@ skipping 43 matching lines @@
 
   // Initialize the colors, border, and layout.
   SetStyle(style_);
 
   SetAccessibleName(text);
 }
 
 LabelButton::~LabelButton() {}
 
 const gfx::ImageSkia& LabelButton::GetImage(ButtonState for_state) {
-  if (for_state != STATE_NORMAL && button_state_images_[for_state].isNull())
-    return button_state_images_[STATE_NORMAL];
-  return button_state_images_[for_state];
+  if (for_state != STATE_NORMAL && button_state_images_[0][for_state].isNull())
+    return button_state_images_[0][STATE_NORMAL];
+  return button_state_images_[0][for_state];
+}
+
+const gfx::ImageSkia& LabelButton::GetFocusedImage(ButtonState for_state) {

[msw, 2013/05/18 03:46:30]

Don't make a separate GetFocusedImage. Check if the button is focused in
GetImage, instead of hardcoding [0] up there and [1] here, like this:
const size_t focused_index = HasFocus() ? 1 : 0;
... button_state_images_[focused_index][for_state] ...

[tfarina, 2013/05/18 18:43:23]

Done.

+  if (for_state != STATE_NORMAL && button_state_images_[1][for_state].isNull())
+    return button_state_images_[1][STATE_NORMAL];
+  return button_state_images_[1][for_state];
 }
 
 void LabelButton::SetImage(ButtonState for_state, const gfx::ImageSkia& image) {
-  button_state_images_[for_state] = image;
+  button_state_images_[0][for_state] = image;
   image_->SetImage(GetImage(state()));
 }
 
+void LabelButton::SetFocusedImage(ButtonState for_state,
+                                  const gfx::ImageSkia& image) {
+  button_state_images_[1][for_state] = image;
+  image_->SetImage(GetFocusedImage(state()));
+}
+
 const string16& LabelButton::GetText() const {
   return label_->text();
 }
 
 void LabelButton::SetText(const string16& text) {
   SetAccessibleName(text);
   label_->SetText(text);
 }
 
 void LabelButton::SetTextColor(ButtonState for_state, SkColor color) {
@@ skipping 103 matching lines @@
     size.set_width(std::min(max_size_.width(), size.width()));
   if (max_size_.height() > 0)
     size.set_height(std::min(max_size_.height(), size.height()));
   return size;
 }
 
 const char* LabelButton::GetClassName() const {
   return kViewClassName;
 }
 
+void LabelButton::GetExtraParams(ui::NativeTheme::ExtraParams* params) const {
+  params->button.checked = false;
+  params->button.indeterminate = false;
+  params->button.is_default = is_default_;
+  params->button.is_focused = HasFocus() && IsAccessibilityFocusable();
+  params->button.has_border = style() == STYLE_NATIVE_TEXTBUTTON;
+  params->button.classic_state = 0;
+  params->button.background_color = GetNativeTheme()->GetSystemColor(
+      ui::NativeTheme::kColorId_ButtonBackgroundColor);
+}
+
 void LabelButton::ResetColorsFromNativeTheme() {
   const ui::NativeTheme* theme = GetNativeTheme();
   SkColor colors[STATE_COUNT] = {
     theme->GetSystemColor(ui::NativeTheme::kColorId_ButtonEnabledColor),
     theme->GetSystemColor(ui::NativeTheme::kColorId_ButtonHoverColor),
     theme->GetSystemColor(ui::NativeTheme::kColorId_ButtonHoverColor),
     theme->GetSystemColor(ui::NativeTheme::kColorId_ButtonDisabledColor),
   };
 
   // Certain styles do not change text color when hovered or pressed.
@@ skipping 78 matching lines @@
   return ui::NativeTheme::kPushButton;
 }
 
 gfx::Rect LabelButton::GetThemePaintRect() const {
   return GetLocalBounds();
 }
 
 ui::NativeTheme::State LabelButton::GetThemeState(
     ui::NativeTheme::ExtraParams* params) const {
   GetExtraParams(params);
-  switch(state()) {
+  switch (state()) {
     case STATE_NORMAL:   return ui::NativeTheme::kNormal;
     case STATE_HOVERED:  return ui::NativeTheme::kHovered;
     case STATE_PRESSED:  return ui::NativeTheme::kPressed;
     case STATE_DISABLED: return ui::NativeTheme::kDisabled;
     case STATE_COUNT:    NOTREACHED() << "Unknown state: " << state();
   }
   return ui::NativeTheme::kNormal;
 }
 
 const ui::Animation* LabelButton::GetThemeAnimation() const {
@@ skipping 12 matching lines @@
   GetExtraParams(params);
   return ui::NativeTheme::kNormal;
 }
 
 ui::NativeTheme::State LabelButton::GetForegroundThemeState(
     ui::NativeTheme::ExtraParams* params) const {
   GetExtraParams(params);
   return ui::NativeTheme::kHovered;
 }
 
-void LabelButton::GetExtraParams(ui::NativeTheme::ExtraParams* params) const {
-  params->button.checked = false;
-  params->button.indeterminate = false;
-  params->button.is_default = is_default_;
-  params->button.is_focused = HasFocus() && IsAccessibilityFocusable();
-  params->button.has_border = style() == STYLE_NATIVE_TEXTBUTTON;
-  params->button.classic_state = 0;
-  params->button.background_color = GetNativeTheme()->GetSystemColor(
-      ui::NativeTheme::kColorId_ButtonBackgroundColor);
-}
-
 }  // namespace views