Comment #23 on issue 25520 by [email protected]: Create our own FTP LIST parsing code http://code.google.com/p/chromium/issues/detail?id=25520
The following revision refers to this bug:
http://src.chromium.org/viewvc/chrome?view=rev&revision=33807
------------------------------------------------------------------------
r33807 | [email protected] | 2009-12-04 01:58:32 -0800 (Fri, 04 Dec
2009) | 9 lines
Changed paths:
M
http://src.chromium.org/viewvc/chrome/trunk/src/net/ftp/ftp_directory_listing_buffer.cc?r1=33807&r2=33806
M
http://src.chromium.org/viewvc/chrome/trunk/src/net/ftp/ftp_directory_listing_buffer_unittest.cc?r1=33807&r2=33806
A
http://src.chromium.org/viewvc/chrome/trunk/src/net/ftp/ftp_directory_listing_parser.cc
A
http://src.chromium.org/viewvc/chrome/trunk/src/net/ftp/ftp_directory_listing_parser.h
A
http://src.chromium.org/viewvc/chrome/trunk/src/net/ftp/ftp_directory_listing_parser_ls.cc
A
http://src.chromium.org/viewvc/chrome/trunk/src/net/ftp/ftp_directory_listing_parser_ls.h
A
http://src.chromium.org/viewvc/chrome/trunk/src/net/ftp/ftp_directory_listing_parser_ls_unittest.cc
A
http://src.chromium.org/viewvc/chrome/trunk/src/net/ftp/ftp_directory_listing_parser_unittest.h
A
http://src.chromium.org/viewvc/chrome/trunk/src/net/ftp/ftp_directory_listing_parser_vms.cc
A
http://src.chromium.org/viewvc/chrome/trunk/src/net/ftp/ftp_directory_listing_parser_vms.h
A
http://src.chromium.org/viewvc/chrome/trunk/src/net/ftp/ftp_directory_listing_parser_vms_unittest.cc
A
http://src.chromium.org/viewvc/chrome/trunk/src/net/ftp/ftp_directory_listing_parser_windows.cc
A
http://src.chromium.org/viewvc/chrome/trunk/src/net/ftp/ftp_directory_listing_parser_windows.h
A
http://src.chromium.org/viewvc/chrome/trunk/src/net/ftp/ftp_directory_listing_parser_windows_unittest.cc
D /trunk/src/net/ftp/ftp_directory_listing_parsers.cc
D /trunk/src/net/ftp/ftp_directory_listing_parsers.h
D /trunk/src/net/ftp/ftp_directory_listing_parsers_unittest.cc
M
http://src.chromium.org/viewvc/chrome/trunk/src/net/ftp/ftp_util.cc?r1=33807&r2=33806
M
http://src.chromium.org/viewvc/chrome/trunk/src/net/ftp/ftp_util.h?r1=33807&r2=33806
M
http://src.chromium.org/viewvc/chrome/trunk/src/net/net.gyp?r1=33807&r2=33806
M
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/ftp_directory_listing_response_delegate.cc?r1=33807&r2=33806
Split FTP LIST parsing code into individual files for each listing style.
This turns out to be much better code organisation strategy. We're going
to have even more parsers.
TEST=Covered by net_unittests.
BUG=25520
Review URL: http://codereview.chromium.org/465035
------------------------------------------------------------------------
--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings
--
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
