diff options
author | rafaelw@chromium.org <rafaelw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-05-25 22:51:29 +0000 |
---|---|---|
committer | rafaelw@chromium.org <rafaelw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-05-25 22:51:29 +0000 |
commit | c74eee96a9da828ca67a88abbfe67a0e998b3467 (patch) | |
tree | 8899dea6215d83f4280aaaac91749b9cc432b7d5 /views | |
parent | c07a637e84b8810ca6421fbc023209fe90aab328 (diff) | |
download | chromium_src-c74eee96a9da828ca67a88abbfe67a0e998b3467.zip chromium_src-c74eee96a9da828ca67a88abbfe67a0e998b3467.tar.gz chromium_src-c74eee96a9da828ca67a88abbfe67a0e998b3467.tar.bz2 |
Add bug number in comments for each TreeViewTest marked as FAILS
TBR=phajdan.jr
TEST=NONE
Review URL: http://codereview.chromium.org/2223001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@48214 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'views')
-rw-r--r-- | views/controls/table/table_view_unittest.cc | 12 |
1 files changed, 11 insertions, 1 deletions
diff --git a/views/controls/table/table_view_unittest.cc b/views/controls/table/table_view_unittest.cc index b47fc41..64e8d90 100644 --- a/views/controls/table/table_view_unittest.cc +++ b/views/controls/table/table_view_unittest.cc @@ -234,7 +234,7 @@ class NullModelTableViewTest : public TableViewTest { // Tests ----------------------------------------------------------------------- -// Several failing tests below: http://crbug.com/45015 +// Failing: http://crbug.com/45015 // Tests various sorting permutations. TEST_F(TableViewTest, FAILS_Sort) { // Sort by first column descending. @@ -262,6 +262,7 @@ TEST_F(TableViewTest, FAILS_Sort) { return; } +// Failing: http://crbug.com/45015 // Tests changing the model while sorted. TEST_F(TableViewTest, FAILS_SortThenChange) { // Sort by first column descending. @@ -276,6 +277,7 @@ TEST_F(TableViewTest, FAILS_SortThenChange) { VeriyViewOrder(0, 2, 1, -1); } +// Failing: http://crbug.com/45015 // Tests adding to the model while sorted. TEST_F(TableViewTest, FAILS_AddToSorted) { // Sort by first column descending. @@ -297,6 +299,7 @@ TEST_F(TableViewTest, FAILS_AddToSorted) { VeriyViewOrder(1, 4, 3, 2, 0, -1); } +// Failing: http://crbug.com/45015 // Tests selection on sort. TEST_F(TableViewTest, FAILS_PersistSelectionOnSort) { // Select row 0. @@ -314,11 +317,13 @@ TEST_F(TableViewTest, FAILS_PersistSelectionOnSort) { EXPECT_EQ(0, table_->FirstSelectedRow()); } +// Failing: http://crbug.com/45015 // Tests selection iterator with sort. TEST_F(TableViewTest, FAILS_PersistMultiSelectionOnSort) { SetUpMultiSelectTestState(true); } +// Failing: http://crbug.com/45015 // Tests selection persists after a change when sorted with iterator. TEST_F(TableViewTest, FAILS_PersistMultiSelectionOnChangeWithSort) { SetUpMultiSelectTestState(true); @@ -330,6 +335,7 @@ TEST_F(TableViewTest, FAILS_PersistMultiSelectionOnChangeWithSort) { VerifySelectedRows(1, 0, -1); } +// Failing: http://crbug.com/45015 // Tests selection persists after a remove when sorted with iterator. TEST_F(TableViewTest, FAILS_PersistMultiSelectionOnRemoveWithSort) { SetUpMultiSelectTestState(true); @@ -341,6 +347,7 @@ TEST_F(TableViewTest, FAILS_PersistMultiSelectionOnRemoveWithSort) { VerifySelectedRows(0, -1); } +// Failing: http://crbug.com/45015 // Tests selection persists after a add when sorted with iterator. TEST_F(TableViewTest, FAILS_PersistMultiSelectionOnAddWithSort) { SetUpMultiSelectTestState(true); @@ -352,6 +359,7 @@ TEST_F(TableViewTest, FAILS_PersistMultiSelectionOnAddWithSort) { VerifySelectedRows(0, 1, -1); } +// Failing: http://crbug.com/45015 // Tests selection persists after a change with iterator. TEST_F(TableViewTest, FAILS_PersistMultiSelectionOnChange) { SetUpMultiSelectTestState(false); @@ -363,6 +371,7 @@ TEST_F(TableViewTest, FAILS_PersistMultiSelectionOnChange) { VerifySelectedRows(1, 0, -1); } +// Failing: http://crbug.com/45015 // Tests selection persists after a remove with iterator. TEST_F(TableViewTest, FAILS_PersistMultiSelectionOnRemove) { SetUpMultiSelectTestState(false); @@ -374,6 +383,7 @@ TEST_F(TableViewTest, FAILS_PersistMultiSelectionOnRemove) { VerifySelectedRows(0, -1); } +// Failing: http://crbug.com/45015 // Tests selection persists after a add with iterator. TEST_F(TableViewTest, FAILS_PersistMultiSelectionOnAdd) { SetUpMultiSelectTestState(false); |