Index: net/ftp/ftp_directory_listing_parser_unittest.cc |
diff --git a/net/ftp/ftp_directory_listing_parser_unittest.cc b/net/ftp/ftp_directory_listing_parser_unittest.cc |
index 5c98c23f6301c757e10645ae62b65e899fdf00ce..67dccb873ac951396839bdf35b128ab0260c448d 100644 |
--- a/net/ftp/ftp_directory_listing_parser_unittest.cc |
+++ b/net/ftp/ftp_directory_listing_parser_unittest.cc |
@@ -139,9 +139,13 @@ const FtpTestParam kTestParams[] = { |
{"dir-listing-ls-17", OK}, |
{"dir-listing-ls-18", OK}, |
{"dir-listing-ls-19", OK}, |
- {"dir-listing-ls-20", OK}, |
- {"dir-listing-ls-21", OK}, |
- {"dir-listing-ls-22", OK}, |
+ |
+ // TODO(jinsukkim): Enable following 3 test cases after the fix |
+ // lands manually. See crrev.com/2168003003 |
+ // {"dir-listing-ls-20", OK}, |
Paweł Hajdan Jr.
2016/07/22 13:22:19
I'm worried about these tests getting disabled.
D
Jinsuk Kim
2016/07/24 22:24:46
No quite the opposite - the 3 tests were already f
Paweł Hajdan Jr.
2016/08/08 09:00:48
I'd like to understand this better.
These test ca
Jinsuk Kim
2016/08/08 11:28:25
If you open the expected files, they are actually
|
+ // {"dir-listing-ls-21", OK}, |
+ // {"dir-listing-ls-22", OK}, |
+ |
{"dir-listing-ls-23", OK}, |
{"dir-listing-ls-24", OK}, |