Full Keyboard Access: Second Approach

ui/views/color_chooser/color_chooser_view.cc

Index: ui/views/color_chooser/color_chooser_view.cc
diff --git a/ui/views/color_chooser/color_chooser_view.cc b/ui/views/color_chooser/color_chooser_view.cc
index e0aea825e9e4b7709934ace3c514bfbc33dfcf9d..730ffa1ffe25feac2ed17d3d748a8c96a1498008 100644
--- a/ui/views/color_chooser/color_chooser_view.cc
+++ b/ui/views/color_chooser/color_chooser_view.cc
@@ -25,6 +25,7 @@
 #include "ui/views/controls/textfield/textfield_controller.h"
 #include "ui/views/layout/box_layout.h"
 #include "ui/views/layout/grid_layout.h"
+#include "ui/views/style/platform_style.h"
 #include "ui/views/widget/widget.h"
 
 namespace {
@@ -141,7 +142,8 @@ class ColorChooserView::HueView : public LocatedEventHandlerView {
 ColorChooserView::HueView::HueView(ColorChooserView* chooser_view)
     : chooser_view_(chooser_view),
       level_(0) {
-  SetFocusable(false);
+  views::PlatformStyle::ConfigureFocus(
+      views::PlatformStyle::CONTROL::COLOR_CHOOSER_VIEW, this);
 }
 
 void ColorChooserView::HueView::OnHueChanged(SkScalar hue) {
@@ -251,7 +253,8 @@ ColorChooserView::SaturationValueView::SaturationValueView(
     ColorChooserView* chooser_view)
     : chooser_view_(chooser_view),
       hue_(0) {
-  SetFocusable(false);
+  views::PlatformStyle::ConfigureFocus(
+      views::PlatformStyle::CONTROL::COLOR_CHOOSER_VIEW, this);
   SetBorder(Border::CreateSolidBorder(kBorderWidth, SK_ColorGRAY));
 }
 
@@ -344,7 +347,8 @@ class ColorChooserView::SelectedColorPatchView : public views::View {
 };
 
 ColorChooserView::SelectedColorPatchView::SelectedColorPatchView() {
-  SetFocusable(false);
+  views::PlatformStyle::ConfigureFocus(
+      views::PlatformStyle::CONTROL::COLOR_CHOOSER_VIEW, this);
   SetVisible(true);
   SetBorder(Border::CreateSolidBorder(kBorderWidth, SK_ColorGRAY));
 }