Compatibility improvements for "ls -l" FTP LIST response format.

Compatibility improvements for "ls -l" FTP LIST response format.

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

[eroman, 2009-12-08 18:32:58]

So the main difference here is you detect the column offset per line, rather
than just once per listing.

Should we error out if subsequent lines have a different offset than earlier
ones?

LGTM.

[Paweł Hajdan Jr., 2009-12-08 20:35:18]

On 2009/12/08 18:32:58, eroman wrote:
> So the main difference here is you detect the column offset per line, rather
> than just once per listing.

That's right.

> Should we error out if subsequent lines have a different offset than earlier
> ones?

We can't. Some FTP servers are a bit crazy, and different lines have different
offsets, depending on whether it's a file or a directory (just realized that I
didn't add a test case for that).

I'll fix that (add the test case) in a following CL. I have some test cases for
non-ASCII characters in listings.

[Paweł Hajdan Jr., 2009-12-09 13:51:26]

Just realized that this CL _had_ a test case for different column offsets in
different rows. I just missed that fact when replying earlier. :)

I'll have another part of compatibility fixes soon anyway.