MacViews: Implement Full Keyboard Access.

chrome/browser/ui/views/website_settings/permission_selector_view.cc

Index: chrome/browser/ui/views/website_settings/permission_selector_view.cc
diff --git a/chrome/browser/ui/views/website_settings/permission_selector_view.cc b/chrome/browser/ui/views/website_settings/permission_selector_view.cc
index 09fe6bcbccc94b50ac5c5f722c2e01104bf6db72..660f249880cef826c1e4de5be836efb87e468a7b 100644
--- a/chrome/browser/ui/views/website_settings/permission_selector_view.cc
+++ b/chrome/browser/ui/views/website_settings/permission_selector_view.cc
@@ -19,6 +19,7 @@
 #include "ui/views/controls/label.h"
 #include "ui/views/controls/menu/menu_runner.h"
 #include "ui/views/layout/grid_layout.h"
+#include "ui/views/style/platform_style.h"
 #include "ui/views/view.h"
 #include "ui/views/widget/widget.h"
 
@@ -178,7 +179,12 @@ PermissionSelectorView::PermissionSelectorView(
           permission.source),
       menu_model_.get(), button_enabled);
   menu_button_->SetEnabled(button_enabled);

[tapted, 2016/02/23 03:01:19]

View::IsFocusable() has a check for |enabled_|, so this should just be
SetFocusable(true)

-  menu_button_->SetFocusable(button_enabled);
+
+  if (button_enabled)
+    views::PlatformStyle::SetControlStyleFocus(menu_button_);
+  else
+    menu_button_->SetFocusable(false);
+
   menu_button_->SetAccessibleName(
       WebsiteSettingsUI::PermissionTypeToUIString(permission.type));
   layout->AddView(menu_button_);