views: Switch Checkbox over to LabelButton.

ui/views/controls/button/label_button_border.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_border.h"
 
 #include "base/logging.h"
 #include "grit/ui_resources.h"
 #include "ui/base/animation/animation.h"
 #include "ui/gfx/canvas.h"
@@ skipping 12 matching lines @@
 const int kPressedImages[] = IMAGE_GRID(IDR_BUTTON_PRESSED);
 const int kFocusedNormalImages[] = IMAGE_GRID(IDR_BUTTON_FOCUSED_NORMAL);
 const int kFocusedHoveredImages[] = IMAGE_GRID(IDR_BUTTON_FOCUSED_HOVER);
 const int kFocusedPressedImages[] = IMAGE_GRID(IDR_BUTTON_FOCUSED_PRESSED);
 
 // The text-button hot and pushed image IDs; normal is unadorned by default.
 const int kTextHoveredImages[] = IMAGE_GRID(IDR_TEXTBUTTON_HOVER);
 const int kTextPressedImages[] = IMAGE_GRID(IDR_TEXTBUTTON_PRESSED);
 
 Button::ButtonState GetButtonState(ui::NativeTheme::State state) {
-  switch(state) {
+  switch (state) {
     case ui::NativeTheme::kDisabled: return Button::STATE_DISABLED;
     case ui::NativeTheme::kHovered:  return Button::STATE_HOVERED;
     case ui::NativeTheme::kNormal:   return Button::STATE_NORMAL;
     case ui::NativeTheme::kPressed:  return Button::STATE_PRESSED;
+    case ui::NativeTheme::kToggled:  return Button::STATE_TOGGLED;
     case ui::NativeTheme::kMaxState: NOTREACHED() << "Unknown state: " << state;
   }
   return Button::STATE_NORMAL;
 }
 
 // A helper function to paint the native theme or images as appropriate.
 void PaintHelper(LabelButtonBorder* border,
                  gfx::Canvas* canvas,
                  const ui::NativeTheme* theme,
                  ui::NativeTheme::Part part,
@@ skipping 38 matching lines @@
     SetPainter(false, Button::STATE_HOVERED,
                Painter::CreateImageGridPainter(kTextHoveredImages));
     SetPainter(false, Button::STATE_PRESSED,
                Painter::CreateImageGridPainter(kTextPressedImages));
   }
 }
 
 LabelButtonBorder::~LabelButtonBorder() {}
 
 void LabelButtonBorder::Paint(const View& view, gfx::Canvas* canvas) {
-  DCHECK(!strcmp(view.GetClassName(), LabelButton::kViewClassName));
+  //DCHECK(!strcmp(view.GetClassName(), LabelButton::kViewClassName));

[msw, 2013/05/16 22:40:05]

You can just remove this DCHECK.

[tfarina, 2013/05/18 00:54:08]

Done.

   const NativeThemeDelegate* native_theme_delegate =
       static_cast<const LabelButton*>(&view);
   ui::NativeTheme::Part part = native_theme_delegate->GetThemePart();
   gfx::Rect rect(native_theme_delegate->GetThemePaintRect());
   ui::NativeTheme::ExtraParams extra;
   const ui::NativeTheme* theme = view.GetNativeTheme();
   const ui::Animation* animation = native_theme_delegate->GetThemeAnimation();
   ui::NativeTheme::State state = native_theme_delegate->GetThemeState(&extra);
 
   if (animation && animation->is_animating()) {
@@ skipping 24 matching lines @@
 }
 
 gfx::Insets LabelButtonBorder::GetInsets() const {
   // Return the style-specific insets between button contents and edges.
   if (style() == Button::STYLE_BUTTON)
     return gfx::Insets(9, 13, 9, 13);
   if (style() == Button::STYLE_TEXTBUTTON)
     return gfx::Insets(5, 6, 5, 6);
   if (style() == Button::STYLE_NATIVE_TEXTBUTTON)
     return gfx::Insets(5, 12, 5, 12);
+  if (style() == Button::STYLE_CHECKBOX)

[msw, 2013/05/16 22:40:05]

Would you mind making this function a switch instead? Then
STYLE_NATIVE_TEXTBUTTON / STYLE_CHECKBOX / STYLE_RADIO could all use the same
case statement.

[tfarina, 2013/05/18 00:54:08]

Done.

+    return gfx::Insets(5, 12, 5, 12);
+  if (style() == Button::STYLE_RADIO)
+    return gfx::Insets(5, 12, 5, 12);
   NOTREACHED();
   return gfx::Insets();
 }
 
 Painter* LabelButtonBorder::GetPainter(bool focused,
                                        Button::ButtonState state) {
   return painters_[focused ? 1 : 0][state].get();
 }
 
 void LabelButtonBorder::SetPainter(bool focused,
                                    Button::ButtonState state,
                                    Painter* painter) {
   painters_[focused ? 1 : 0][state].reset(painter);
 }
 
 }  // namespace views