Mark TreeViewTests as FAILS_ rather than DISABLED_

views/controls/table/table_view_unittest.cc

 // Copyright (c) 2010 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 <vector>
 
 #include "app/table_model.h"
 #include "app/table_model_observer.h"
 #include "base/message_loop.h"
 #include "base/string_util.h"
@@ skipping 216 matching lines @@
 class NullModelTableViewTest : public TableViewTest {
  protected:
   // Creates the model.
   TestTableModel* CreateModel() {
     return NULL;
   }
 };
 
 // Tests -----------------------------------------------------------------------
 
+// Several failing tests below: http://crbug.com/45015
 // Tests various sorting permutations.
-TEST_F(TableViewTest, DISABLED_Sort) {
+TEST_F(TableViewTest, FAILS_Sort) {
   // Sort by first column descending.
   TableView::SortDescriptors sort;
   sort.push_back(TableView::SortDescriptor(0, false));
   table_->SetSortDescriptors(sort);
   VeriyViewOrder(2, 1, 0, -1);
   if (HasFatalFailure())
     return;
 
   // Sort by second column ascending, first column descending.
   sort.clear();
   sort.push_back(TableView::SortDescriptor(1, true));
   sort.push_back(TableView::SortDescriptor(0, false));
   sort[1].ascending = false;
   table_->SetSortDescriptors(sort);
   VeriyViewOrder(1, 0, 2, -1);
   if (HasFatalFailure())
     return;
 
   // Clear the sort.
   table_->SetSortDescriptors(TableView::SortDescriptors());
   VeriyViewOrder(0, 1, 2, -1);
   if (HasFatalFailure())
     return;
 }
 
 // Tests changing the model while sorted.
-TEST_F(TableViewTest, DISABLED_SortThenChange) {
+TEST_F(TableViewTest, FAILS_SortThenChange) {
   // Sort by first column descending.
   TableView::SortDescriptors sort;
   sort.push_back(TableView::SortDescriptor(0, false));
   table_->SetSortDescriptors(sort);
   VeriyViewOrder(2, 1, 0, -1);
   if (HasFatalFailure())
     return;
 
   model_->ChangeRow(0, 3, 1);
   VeriyViewOrder(0, 2, 1, -1);
 }
 
 // Tests adding to the model while sorted.
-TEST_F(TableViewTest, DISABLED_AddToSorted) {
+TEST_F(TableViewTest, FAILS_AddToSorted) {
   // Sort by first column descending.
   TableView::SortDescriptors sort;
   sort.push_back(TableView::SortDescriptor(0, false));
   table_->SetSortDescriptors(sort);
   VeriyViewOrder(2, 1, 0, -1);
   if (HasFatalFailure())
     return;
 
   // Add row so that it occurs first.
   model_->AddRow(0, 5, -1);
   VeriyViewOrder(0, 3, 2, 1, -1);
   if (HasFatalFailure())
     return;
 
   // Add row so that it occurs last.
   model_->AddRow(0, -1, -1);
   VeriyViewOrder(1, 4, 3, 2, 0, -1);
 }
 
 // Tests selection on sort.
-TEST_F(TableViewTest, DISABLED_PersistSelectionOnSort) {
+TEST_F(TableViewTest, FAILS_PersistSelectionOnSort) {
   // Select row 0.
   table_->Select(0);
 
   // Sort by first column descending.
   TableView::SortDescriptors sort;
   sort.push_back(TableView::SortDescriptor(0, false));
   table_->SetSortDescriptors(sort);
   VeriyViewOrder(2, 1, 0, -1);
   if (HasFatalFailure())
     return;
 
   // Make sure 0 is still selected.
   EXPECT_EQ(0, table_->FirstSelectedRow());
 }
 
 // Tests selection iterator with sort.
-TEST_F(TableViewTest, PersistMultiSelectionOnSort) {
+TEST_F(TableViewTest, FAILS_PersistMultiSelectionOnSort) {
   SetUpMultiSelectTestState(true);
 }
 
 // Tests selection persists after a change when sorted with iterator.
-TEST_F(TableViewTest, PersistMultiSelectionOnChangeWithSort) {
+TEST_F(TableViewTest, FAILS_PersistMultiSelectionOnChangeWithSort) {
   SetUpMultiSelectTestState(true);
   if (HasFatalFailure())
     return;
 
   model_->ChangeRow(0, 3, 1);
 
   VerifySelectedRows(1, 0, -1);
 }
 
 // Tests selection persists after a remove when sorted with iterator.
-TEST_F(TableViewTest, DISABLED_PersistMultiSelectionOnRemoveWithSort) {
+TEST_F(TableViewTest, FAILS_PersistMultiSelectionOnRemoveWithSort) {
   SetUpMultiSelectTestState(true);
   if (HasFatalFailure())
     return;
 
   model_->RemoveRow(0);
 
   VerifySelectedRows(0, -1);
 }
 
 // Tests selection persists after a add when sorted with iterator.
-TEST_F(TableViewTest, DISABLED_PersistMultiSelectionOnAddWithSort) {
+TEST_F(TableViewTest, FAILS_PersistMultiSelectionOnAddWithSort) {
   SetUpMultiSelectTestState(true);
   if (HasFatalFailure())
     return;
 
   model_->AddRow(3, 4, 4);
 
   VerifySelectedRows(0, 1, -1);
 }
 
 // Tests selection persists after a change with iterator.
-TEST_F(TableViewTest, DISABLED_PersistMultiSelectionOnChange) {
+TEST_F(TableViewTest, FAILS_PersistMultiSelectionOnChange) {
   SetUpMultiSelectTestState(false);
   if (HasFatalFailure())
     return;
 
   model_->ChangeRow(0, 3, 1);
 
   VerifySelectedRows(1, 0, -1);
 }
 
 // Tests selection persists after a remove with iterator.
-TEST_F(TableViewTest, DISABLED_PersistMultiSelectionOnRemove) {
+TEST_F(TableViewTest, FAILS_PersistMultiSelectionOnRemove) {
   SetUpMultiSelectTestState(false);
   if (HasFatalFailure())
     return;
 
   model_->RemoveRow(0);
 
   VerifySelectedRows(0, -1);
 }
 
 // Tests selection persists after a add with iterator.
-TEST_F(TableViewTest, DISABLED_PersistMultiSelectionOnAdd) {
+TEST_F(TableViewTest, FAILS_PersistMultiSelectionOnAdd) {
   SetUpMultiSelectTestState(false);
   if (HasFatalFailure())
     return;
 
   model_->AddRow(3, 4, 4);
 
   VerifySelectedRows(1, 0, -1);
 }
 
 TEST_F(NullModelTableViewTest, NullModel) {
@@ skipping 165 matching lines @@
   table_->FocusRow(0);
   EXPECT_EQ(0, table_->GetFirstFocusedRow());
 
   table_->FocusRow(2);
   EXPECT_EQ(2, table_->GetFirstFocusedRow());
 
   table_->ClearRowFocus();
   EXPECT_EQ(-1, table_->GetFirstSelectedRow());
 }
 #endif