Adjust initial activation/focus Task Manager/Views TableView.

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 ab17e37d723aad00bee65cb576e12a2cc833c984..8044b6d0fc20e5e5bab9923493f20212f294b31b 100644
--- a/ui/views/controls/table/table_view.cc
+++ b/ui/views/controls/table/table_view.cc
@@ -626,8 +626,11 @@ void TableView::OnFocus() {
   ScrollView* scroll_view = ScrollView::GetScrollViewForContents(this);
   if (scroll_view)
     scroll_view->SetHasFocusIndicator(true);
-  SchedulePaintForSelection();
 
+  if (selection_model_.active() == -1)

[sky, 2017/03/23 17:50:17]

&& RowCount() > 0 ? Maybe add test coverage for the empty model case?

[Evan Stade, 2017/03/23 22:56:26]

I didn't add that check because I was copying AdvanceSelection which doesn't
make the check. But it seems there are other places like OnKeyPressed that do
check, so added.

I had to move some DCHECKs in ModelToView to make the test actually catch this
problem (the selection model doesn't or care about what rows actually exist, so
it's happy to select a non existent row).

+    SelectByViewIndex(0);
+
+  SchedulePaintForSelection();
   NotifyAccessibilityEvent(ui::AX_EVENT_FOCUS, true);
 }