Improve TableView unit tests and fix bugs that were uncovered.

ui/views/controls/table/table_view.h

 // 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.
 
 #ifndef UI_VIEWS_CONTROLS_TABLE_TABLE_VIEW_VIEWS_H_
 #define UI_VIEWS_CONTROLS_TABLE_TABLE_VIEW_VIEWS_H_
 
 #include <vector>
 
 #include "base/macros.h"
+#include "base/memory/weak_ptr.h"
 #include "ui/base/models/list_selection_model.h"
 #include "ui/base/models/table_model.h"
 #include "ui/base/models/table_model_observer.h"
 #include "ui/gfx/font_list.h"
 #include "ui/views/view.h"
 #include "ui/views/views_export.h"
 
 // A TableView is a view that displays multiple rows with any number of columns.
 // TableView is driven by a TableModel. The model returns the contents
 // to display. TableModel also has an Observer which is used to notify
@@ skipping 268 matching lines @@
   // 1.
   GroupRange GetGroupRange(int model_index) const;
 
   // Used by both GetTooltipText methods. Returns true if there is a tooltip and
   // sets |tooltip| and/or |tooltip_origin| as appropriate, each of which may be
   // NULL.
   bool GetTooltipImpl(const gfx::Point& location,
                       base::string16* tooltip,
                       gfx::Point* tooltip_origin) const;
 
+  // Selects the first row if the table has focus and no row is currently
+  // selected. This is a delayed task posted by OnFocus because things that
+  // request focus are also likely to select a row or rows.
+  void SelectFirstRowIfNoneSelected();
+
   ui::TableModel* model_;
 
   std::vector<ui::TableColumn> columns_;
 
   // The set of visible columns. The values of these point to |columns_|. This
   // may contain a subset of |columns_|.
   std::vector<VisibleColumn> visible_columns_;
 
   // The header. This is only created if more than one column is specified or
   // the first column has a non-empty title.
@@ skipping 31 matching lines @@
 
   // Mappings used when sorted.
   std::vector<int> view_to_model_;
   std::vector<int> model_to_view_;
 
   TableGrouper* grouper_;
 
   // True if in SetVisibleColumnWidth().
   bool in_set_visible_column_width_;
 
+  base::WeakPtrFactory<TableView> weak_factory_;
+
   DISALLOW_COPY_AND_ASSIGN(TableView);
 };
 
 }  // namespace views
 
 #endif  // UI_VIEWS_CONTROLS_TABLE_TABLE_VIEW_VIEWS_H_