summaryrefslogtreecommitdiffstats
path: root/views
diff options
context:
space:
mode:
authorrafaelw@chromium.org <rafaelw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2010-05-25 22:51:29 +0000
committerrafaelw@chromium.org <rafaelw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2010-05-25 22:51:29 +0000
commitc74eee96a9da828ca67a88abbfe67a0e998b3467 (patch)
tree8899dea6215d83f4280aaaac91749b9cc432b7d5 /views
parentc07a637e84b8810ca6421fbc023209fe90aab328 (diff)
downloadchromium_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.cc12
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);