Adjust initial activation/focus Task Manager/Views TableView.

ui/views/controls/table/table_view.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ui/views/controls/table/table_view.h"
 
 #include <stddef.h>
 
 #include <algorithm>
 #include <map>
@@ skipping 608 matching lines @@
         gfx::Point(group_indicator_x, start_cell_bounds.CenterPoint().y()),
         kGroupingIndicatorSize / 2, grouping_flags);
     i = last + 1;
   }
 }
 
 void TableView::OnFocus() {
   ScrollView* scroll_view = ScrollView::GetScrollViewForContents(this);
   if (scroll_view)
     scroll_view->SetHasFocusIndicator(true);
+
+  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);
 }
 
 void TableView::OnBlur() {
   ScrollView* scroll_view = ScrollView::GetScrollViewForContents(this);
   if (scroll_view)
     scroll_view->SetHasFocusIndicator(false);
   SchedulePaintForSelection();
 }
 
@@ skipping 308 matching lines @@
   if (tooltip)
     *tooltip = text;
   if (tooltip_origin) {
     tooltip_origin->SetPoint(cell_bounds.x(),
                              cell_bounds.y() + kTextVerticalPadding);
   }
   return true;
 }
 
 }  // namespace views