fixes issue 4376: Press "Tab" key to navigate - Text focus is showing...

chrome/views/checkbox.cc

 // Copyright (c) 2006-2008 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/checkbox.h"
 
 #include "chrome/common/gfx/chrome_canvas.h"
 #include "chrome/views/checkbox.h"
 #include "chrome/views/hwnd_view.h"
 #include "chrome/views/label.h"
@@ skipping 66 matching lines @@
   int new_width = std::min(width() - (kCheckBoxWidth + kCheckBoxToLabel),
                            s.width());
   out->set_width(std::max(0, new_width));
   out->set_height(s.height());
 }
 
 void CheckBox::Paint(ChromeCanvas* canvas) {
   gfx::Rect r;
   ComputeTextRect(&r);
   // Paint the focus border if any.
-  if (HasFocus())
-    canvas->DrawFocusRect(r.x() - kFocusPaddingHorizontal,
+  if (HasFocus()) {
+    // Mirror left point for rectangle to draw focus for RTL text.
+    canvas->DrawFocusRect(MirroredLeftPointForRect(r) - kFocusPaddingHorizontal,
                           r.y() - kFocusPaddingVertical,
                           r.width() + kFocusPaddingHorizontal * 2,
                           r.height() + kFocusPaddingVertical * 2);
+  }
   PaintFloatingView(canvas, label_, r.x(), r.y(), r.width(), r.height());
 }
 
 void CheckBox::SetEnabled(bool enabled) {
   if (enabled_ == enabled)
     return;
   NativeButton::SetEnabled(enabled);
   label_->SetEnabled(enabled);
 }
 
@@ skipping 69 matching lines @@
 
 void CheckBox::OnMouseReleased(const MouseEvent& event,
                                bool canceled) {
   HighlightButton(false);
   if (!canceled && LabelHitTest(event))
     OnCommand(BN_CLICKED, 0, GetNativeControlHWND());
 }
 
 }  // namespace views