Fix the "ls -l" FTP LIST format parser to understand correctly

Fix the "ls -l" FTP LIST format parser to understand correctly
listings sent by VMS MultiNet servers emulating Unix.

TEST=Covered by net_unittests.
BUG=22193, 25520
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=31217

[wtc, 2009-11-05 19:01:20]

I have a "drive-by" comment.  I'll let eroman do the review.

http://codereview.chromium.org/368006/diff/1/7
File net/ftp/ftp_directory_listing_parsers.cc (right):

http://codereview.chromium.org/368006/diff/1/7#newcode245
Line 245: return !received_nonempty_line_;
Can you add a comment here?  This is not obvious to me at all.

[Paweł Hajdan Jr., 2009-11-05 20:59:37]

On 2009/11/05 19:01:20, wtc wrote:
> I have a "drive-by" comment.  I'll let eroman do the review.
> 
> http://codereview.chromium.org/368006/diff/1/7
> File net/ftp/ftp_directory_listing_parsers.cc (right):
> 
> http://codereview.chromium.org/368006/diff/1/7#newcode245
> Line 245: return !received_nonempty_line_;
> Can you add a comment here?  This is not obvious to me at all.

Added a comment and changed the code a little. Should be more clear now.

[eroman, 2009-11-05 21:38:32]

LGTM