Full Keyboard Access: Second Approach

ui/views/controls/link.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/link.h"
 
 #include "build/build_config.h"
 
 #include "base/logging.h"
 #include "base/strings/utf_string_conversions.h"
 #include "ui/accessibility/ax_view_state.h"
 #include "ui/base/cursor/cursor.h"
 #include "ui/base/material_design/material_design_controller.h"
 #include "ui/events/event.h"
 #include "ui/events/keycodes/keyboard_codes.h"
 #include "ui/gfx/canvas.h"
 #include "ui/gfx/color_palette.h"
 #include "ui/gfx/color_utils.h"
 #include "ui/gfx/font_list.h"
 #include "ui/native_theme/native_theme.h"
 #include "ui/views/controls/link_listener.h"
 #include "ui/views/native_cursor.h"
+#include "ui/views/style/platform_style.h"
 
 namespace views {
 
 const char Link::kViewClassName[] = "Link";
 
 Link::Link() : Link(base::string16()) {}
 
 Link::Link(const base::string16& title)
     : Label(title),
       requested_enabled_color_(gfx::kPlaceholderColor),
@@ skipping 122 matching lines @@
 void Link::SetFontList(const gfx::FontList& font_list) {
   Label::SetFontList(font_list);
   RecalculateFont();
 }
 
 void Link::SetText(const base::string16& text) {
   Label::SetText(text);
   // Disable focusability for empty links.  Otherwise Label::GetInsets() will
   // give them an unconditional 1-px. inset on every side to allow for a focus
   // border, when in this case we probably wanted zero width.
-  SetFocusable(!text.empty());
+  if (!text.empty())
+    PlatformStyle::ConfigureFocus(PlatformStyle::CONTROL::LINK, this);
 }
 
 void Link::OnNativeThemeChanged(const ui::NativeTheme* theme) {
   Label::SetEnabledColor(GetEnabledColor());
   SetDisabledColor(
       theme->GetSystemColor(ui::NativeTheme::kColorId_LinkDisabled));
 }
 
 void Link::SetEnabledColor(SkColor color) {
   requested_enabled_color_set_ = true;
@@ skipping 15 matching lines @@
 }
 
 void Link::Init() {
   listener_ = NULL;
   pressed_ = false;
   underline_ = true;
   RecalculateFont();
 
   // Label::Init() calls SetText(), but if that's being called from Label(), our
   // SetText() override will not be reached (because the constructed class is
-  // only a Label at the moment, not yet a Link).  So so the set_focusable()
-  // call explicitly here.
-  SetFocusable(!text().empty());
+  // only a Label at the moment, not yet a Link).  So explicitly set
+  // focusability here.
+  if (!text().empty())
+    PlatformStyle::ConfigureFocus(PlatformStyle::CONTROL::LINK, this);
 }
 
 void Link::SetPressed(bool pressed) {
   if (pressed_ != pressed) {
     pressed_ = pressed;
     Label::SetEnabledColor(GetEnabledColor());
     RecalculateFont();
     SchedulePaint();
   }
 }
@@ skipping 19 matching lines @@
   }
 
   if (!requested_pressed_color_set_ && GetNativeTheme())
     return GetNativeTheme()->GetSystemColor(
         ui::NativeTheme::kColorId_LinkPressed);
 
   return requested_pressed_color_;
 }
 
 }  // namespace views