MacViews: Implement Full Keyboard Access.

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 8d46029a8041d452e6a702a0e891fe843c6b2aba..714aa891692ffe088f197318067db7daffcc4a0e 100644
--- a/ui/views/controls/label.cc
+++ b/ui/views/controls/label.cc
@@ -213,7 +213,7 @@ base::string16 Label::GetDisplayTextForTesting() {
 
 gfx::Insets Label::GetInsets() const {
   gfx::Insets insets = View::GetInsets();
-  if (focusable()) {
+  if (focusable() || accessibility_focusable()) {

[tapted, 2016/04/20 06:05:58]

oh butts - how many things call View::focusable()?  Can we replace it with
something like

bool View::CanFocus() const {
  return focus_behavior_ != FocusBehavior::NEVER;
}

[karandeepb, 2016/05/03 02:54:12]

Replaced this with focus_behavior() check. These are the only 2 instances in the
codebase.

     insets += gfx::Insets(kFocusBorderPadding, kFocusBorderPadding,
                           kFocusBorderPadding, kFocusBorderPadding);
   }
@@ -435,7 +435,7 @@ void Label::MaybeBuildRenderTextLines() {
     return;
 
   gfx::Rect rect = GetContentsBounds();
-  if (focusable())
+  if (focusable() || accessibility_focusable())
     rect.Inset(kFocusBorderPadding, kFocusBorderPadding);
   if (rect.IsEmpty())
     return;