Full Keyboard Access: Second Approach

ui/views/controls/label.cc

Index: ui/views/controls/label.cc
diff --git a/ui/views/controls/label.cc b/ui/views/controls/label.cc
index d1a6cafa9c1da072b6de2c4f3842769d92318266..8441a7f3353a56f9df89207987bb565b1a2c85a2 100644
--- a/ui/views/controls/label.cc
+++ b/ui/views/controls/label.cc
@@ -213,6 +213,8 @@ base::string16 Label::GetDisplayTextForTesting() {
 
 gfx::Insets Label::GetInsets() const {
   gfx::Insets insets = View::GetInsets();
+  // TODO(karandeepb): This won't work for Mac where labels are not focusable
+  // but accessibility focusable. Fix this.
   if (focusable()) {
     insets += gfx::Insets(kFocusBorderPadding, kFocusBorderPadding,
                           kFocusBorderPadding, kFocusBorderPadding);
@@ -435,6 +437,8 @@ void Label::MaybeBuildRenderTextLines() {
     return;
 
   gfx::Rect rect = GetContentsBounds();
+  // TODO(karandeepb): This won't work for Mac where labels are not focusable
+  // but accessibility focusable. Fix this.
   if (focusable())
     rect.Inset(kFocusBorderPadding, kFocusBorderPadding);
   if (rect.IsEmpty())