views::TableView - request focus on tap.

ui/views/controls/table/table_view.cc

Index: ui/views/controls/table/table_view.cc
diff --git a/ui/views/controls/table/table_view.cc b/ui/views/controls/table/table_view.cc
index 6e05566534e2390c329380af58ea0e358c0ccd37..6d1b338bbe6ddcb2b3d57737a823acd2b808cc8a 100644
--- a/ui/views/controls/table/table_view.cc
+++ b/ui/views/controls/table/table_view.cc
@@ -409,6 +409,7 @@ bool TableView::OnMousePressed(const ui::MouseEvent& event) {
 }
 
 void TableView::OnGestureEvent(ui::GestureEvent* event) {
+  RequestFocus();

[sky, 2017/03/28 17:41:24]

Should we only request focus on ET_GESTURE_TAP_DOWN?

[Evan Stade, 2017/03/28 17:43:17]

I am hoping Terry knows what the standard behavior for focusing on gestures is.
I guessed that it should go here based on OnMousePressed.

[tdanderson, 2017/03/28 21:26:22]

Doing a quick audit of other surfaces it doesn't look like we have a standard
way of focusing with gestures. But for consistency with mouse behavior you
should call RequestFocus() on ET_GESTURE_TAP_DOWN as sky@ suggests (and also
call SetHandled() on the event in this case).

   if (event->type() != ui::ET_GESTURE_TAP)
     return;