Fixing focus highlight on high DPI devices for accessibility

ui/message_center/views/notification_view.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/message_center/views/notification_view.h"
 
 #include "base/command_line.h"
 #include "base/strings/string_util.h"
 #include "base/strings/utf_string_conversions.h"
 #include "grit/ui_resources.h"
@@ skipping 304 matching lines @@
   NotificationButton(views::ButtonListener* listener);
   virtual ~NotificationButton();
 
   void SetIcon(const gfx::ImageSkia& icon);
   void SetTitle(const string16& title);
 
   // Overridden from views::View:
   virtual gfx::Size GetPreferredSize() OVERRIDE;
   virtual int GetHeightForWidth(int width) OVERRIDE;
   virtual void OnFocus() OVERRIDE;
-  virtual void OnPaintFocusBorder(gfx::Canvas* canvas) OVERRIDE;
 
   // Overridden from views::CustomButton:
   virtual void StateChanged() OVERRIDE;
 
  private:
   views::ImageView* icon_;
   views::Label* title_;
 };
 
 NotificationButton::NotificationButton(views::ButtonListener* listener)
     : views::CustomButton(listener),
       icon_(NULL),
       title_(NULL) {
   set_focusable(true);
   set_request_focus_on_press(false);
   set_notify_enter_exit_on_child(true);
   SetLayoutManager(
       new views::BoxLayout(views::BoxLayout::kHorizontal,
                            message_center::kButtonHorizontalPadding,
                            kButtonVecticalPadding,
                            message_center::kButtonIconToTitlePadding));
+  set_focus_border(views::FocusBorder::CreateSolidFocusBorder(
+      message_center::kFocusBorderColor,
+      gfx::Insets(1, 2, 2, 2)));
 }
 
 NotificationButton::~NotificationButton() {
 }
 
 void NotificationButton::SetIcon(const gfx::ImageSkia& image) {
   if (icon_ != NULL)
     delete icon_;  // This removes the icon from this view's children.
   if (image.isNull()) {
     icon_ = NULL;
@@ skipping 33 matching lines @@
 
 int NotificationButton::GetHeightForWidth(int width) {
   return message_center::kButtonHeight;
 }
 
 void NotificationButton::OnFocus() {
   views::CustomButton::OnFocus();
   ScrollRectToVisible(GetLocalBounds());
 }
 
-void NotificationButton::OnPaintFocusBorder(gfx::Canvas* canvas) {
-  if (HasFocus()) {
-    canvas->DrawRect(gfx::Rect(2, 1, width() - 4, height() - 3),
-                     message_center::kFocusBorderColor);
-  }
-}
-
 void NotificationButton::StateChanged() {
   if (state() == STATE_HOVERED || state() == STATE_PRESSED) {
     set_background(
         MakeBackground(message_center::kHoveredButtonBackgroundColor));
   } else {
     set_background(NULL);
   }
 }
 
 }  // namespace
@@ skipping 420 matching lines @@
   return message_view_ ?
          message_view_->GetLinesForWidthAndLimit(width, limit) : 0;
 }
 
 int NotificationView::GetMessageHeight(int width, int limit) {
   return message_view_ ?
          message_view_->GetSizeForWidthAndLines(width, limit).height() : 0;
 }
 
 }  // namespace message_center