Fix focus rects for checkboxes and radio buttons:...

chrome/views/controls/button/radio_button.cc

 // Copyright (c) 2009 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/views/controls/button/radio_button.h"
 
 #include "chrome/views/widget/root_view.h"
 
 namespace views {
 
@@ skipping 63 matching lines @@
   }
   return NULL;
 }
 
 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::OnMouseReleased(const views::MouseEvent& event,
-                                  bool canceled) {
+void RadioButton::OnMouseReleased(const MouseEvent& event, bool canceled) {
   native_wrapper_->SetPushed(false);
-  // Call through to toggle the button only if we're not already checked, since
-  // radio buttons can't be toggled like checkboxes.
-  if (!checked())
-    Checkbox::OnMouseReleased(event, canceled);
+  // 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() && !canceled && HitTestLabel(event))
+    SetChecked(true);
+
+  ButtonPressed();
 }
 
 std::string RadioButton::GetClassName() const {
   return kViewClassName;
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // RadioButton, NativeButton overrides:
 
 void RadioButton::CreateWrapper() {
   native_wrapper_ = NativeButtonWrapper::CreateRadioButtonWrapper(this);
   native_wrapper_->UpdateLabel();
   native_wrapper_->UpdateChecked();
 }
 
 }  // namespace views