FTP: Multiple fixes for localized directory listings:

net/ftp/ftp_directory_listing_parser_unittest.cc

 // Copyright (c) 2011 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 "net/ftp/ftp_directory_listing_parser.h"
 
 #include "base/file_util.h"
 #include "base/format_macros.h"
 #include "base/path_service.h"
 #include "base/string_number_conversions.h"
@@ skipping 25 matching lines @@
     "dir-listing-ls-12",
     "dir-listing-ls-13",
     "dir-listing-ls-14",
     "dir-listing-ls-15",
     "dir-listing-ls-16",
     "dir-listing-ls-17",
     "dir-listing-ls-18",
     "dir-listing-ls-19",
     "dir-listing-ls-20",  // TODO(phajdan.jr): should use windows-1251 encoding.
     "dir-listing-ls-21",  // TODO(phajdan.jr): should use windows-1251 encoding.
     "dir-listing-ls-22",  // TODO(phajdan.jr): should use windows-1251 encoding.

[jungshik at Google, 2011/04/01 18:28:41]

Are these TODO still valid given that you're adding windows-1251 in this CL? 

     "dir-listing-ls-23",
     "dir-listing-ls-24",
+
+    // Tests for Russian listings. The only difference between those
+    // files is character encoding:
+    "dir-listing-ls-25",  // UTF-8
+    "dir-listing-ls-26",  // KOI8-R
+    "dir-listing-ls-27",  // windows-1251
+
     "dir-listing-netware-1",
     "dir-listing-netware-2",
     "dir-listing-vms-1",
     "dir-listing-vms-2",
     "dir-listing-vms-3",
     "dir-listing-vms-4",
     "dir-listing-vms-5",
     "dir-listing-windows-1",
     "dir-listing-windows-2",
   };
@@ skipping 74 matching lines @@
       EXPECT_EQ(day_of_month, time_exploded.day_of_month);
       EXPECT_EQ(hour, time_exploded.hour);
       EXPECT_EQ(minute, time_exploded.minute);
     }
   }
 }
 
 }  // namespace
 
 }  // namespace net