views: Switch Checkbox over to LabelButton.

ui/views/controls/button/radio_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/radio_button.h"
 
 #include "base/logging.h"
+#include "grit/ui_resources.h"
 #include "ui/base/accessibility/accessible_view_state.h"
+#include "ui/base/resource/resource_bundle.h"
 #include "ui/views/widget/widget.h"
 
 namespace views {
 
 // static
 const char RadioButton::kViewClassName[] = "views/RadioButton";
 
 RadioButton::RadioButton(const string16& label, int group_id)
     : Checkbox(label) {
+  SetStyle(STYLE_RADIO);
   SetGroup(group_id);
   set_focusable(true);
+
+  ui::ResourceBundle& rb = ui::ResourceBundle::GetSharedInstance();
+  SetImage(STATE_NORMAL, *rb.GetImageSkiaNamed(IDR_RADIO));
+  SetCustomImage(

[msw, 2013/05/21 00:35:17]

nit: ditto for 3 state args on SetCustomImage lines and ordering.

[tfarina, 2013/05/21 01:56:52]

Done.

+      true, false, STATE_NORMAL, *rb.GetImageSkiaNamed(IDR_RADIO_CHECKED));
+  SetCustomImage(true,
+                 false,
+                 STATE_HOVERED,
+                 *rb.GetImageSkiaNamed(IDR_RADIO_CHECKED_HOVER));
+  SetCustomImage(true,
+                 false,
+                 STATE_PRESSED,
+                 *rb.GetImageSkiaNamed(IDR_RADIO_CHECKED_PRESSED));
+  SetCustomImage(
+      false, true, STATE_NORMAL, *rb.GetImageSkiaNamed(IDR_RADIO_FOCUSED));
+  SetCustomImage(true,
+                 true,
+                 STATE_NORMAL,
+                 *rb.GetImageSkiaNamed(IDR_RADIO_FOCUSED_CHECKED));
+  SetCustomImage(true,
+                 true,
+                 STATE_HOVERED,
+                 *rb.GetImageSkiaNamed(IDR_RADIO_FOCUSED_CHECKED_HOVER));
+  SetCustomImage(true,
+                 true,
+                 STATE_PRESSED,
+                 *rb.GetImageSkiaNamed(IDR_RADIO_FOCUSED_CHECKED_PRESSED));
+  SetCustomImage(false,
+                 true,
+                 STATE_HOVERED,
+                 *rb.GetImageSkiaNamed(IDR_RADIO_FOCUSED_HOVER));
+  SetCustomImage(false,
+                 true,
+                 STATE_PRESSED,
+                 *rb.GetImageSkiaNamed(IDR_RADIO_FOCUSED_PRESSED));
+  SetImage(STATE_HOVERED, *rb.GetImageSkiaNamed(IDR_RADIO_HOVER));

[msw, 2013/05/21 00:35:17]

ditto for using SetCustomImage for everything.

[tfarina, 2013/05/21 01:56:52]

Done.

+  SetImage(STATE_PRESSED, *rb.GetImageSkiaNamed(IDR_RADIO_PRESSED));
 }
 
 RadioButton::~RadioButton() {
 }
 
 void RadioButton::SetChecked(bool checked) {
   if (checked == RadioButton::checked())
     return;
   if (checked) {
     // We can't just get the root view here because sometimes the radio
@@ skipping 48 matching lines @@
 bool RadioButton::IsGroupFocusTraversable() const {
   // When focusing a radio button with tab/shift+tab, only the selected button
   // from the group should be focused.
   return false;
 }
 
 void RadioButton::OnFocus() {
   Checkbox::OnFocus();
   SetChecked(true);
   ui::MouseEvent event(ui::ET_MOUSE_PRESSED, gfx::Point(), gfx::Point(), 0);
-  TextButtonBase::NotifyClick(event);
+  LabelButton::NotifyClick(event);
 }
 
 void RadioButton::NotifyClick(const ui::Event& event) {
   // Set the checked state to true only if we are unchecked, since we can't
   // be toggled on and off like a checkbox.
   if (!checked())
     SetChecked(true);
   RequestFocus();
-  TextButtonBase::NotifyClick(event);
+  LabelButton::NotifyClick(event);
 }
 
 ui::NativeTheme::Part RadioButton::GetThemePart() const {
   return ui::NativeTheme::kRadio;
 }
 
 }  // namespace views