Revert of Make FTP directory parser less strict

Revert of Make FTP directory parser less strict (patchset #3 id:40001 of https://codereview.chromium.org/2608213002/ )

Reason for revert:
Fails on all platforms.

https://uberchromegw.corp.google.com/i/chromium.win/builders/Win7%20Tests%20%281%29/builds/62042

Original issue's description:
> Make FTP directory parser less strict
> 
> FTP directory listing returns empty result if encoding detection or
> conversion fails. This CL puts less strict constraint on conversion
> so that the whole list can survive some filenames with broken chars
> and be shown to users. Other browsers seem to work this way.
> 
> The broken chars can happen due to wrong encoding detection, which
> cannot be avoided 100%, as reported in the bug.
> 
> BUG=676762
> R=mmenke@chromium.org, phajdan.jr@chromium.org
> 
> Review-Url: https://codereview.chromium.org/2608213002 .
> Cr-Commit-Position: refs/heads/master@{#441856}
> Committed: https://chromium.googlesource.com/chromium/src/+/ed3ef90ecc3c01246a3d838fbeefdab72c94c228

TBR=phajdan.jr@chromium.org,mmenke@chromium.org,jinsukkim@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=676762
Review-Url: https://codereview.chromium.org/2614103002
Cr-Commit-Position: refs/heads/master@{#441866}
Committed: https://chromium.googlesource.com/chromium/src/+/9c484365968491ca416461d7c982aca155ff26bf

[loyso (OOO), 2017-01-06 03:57:29]

The CQ bit was checked by loyso@chromium.org

[loyso (OOO), 2017-01-06 03:57:30]

Created Revert of Make FTP directory parser less strict

[commit-bot: I haz the power, 2017-01-06 03:57:42]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-06 03:58:30]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1483675049834040, "parent_rev":
"b4271bc30bd0c92be3fcf3c4cc92737edc55e7fb", "commit_rev":
"9c484365968491ca416461d7c982aca155ff26bf"}

[commit-bot: I haz the power, 2017-01-06 03:58:58]

Description was changed from

==========
Revert of Make FTP directory parser less strict (patchset #3 id:40001 of
https://codereview.chromium.org/2608213002/ )

Reason for revert:
Fails on all platforms.

https://uberchromegw.corp.google.com/i/chromium.win/builders/Win7%20Tests%20%...

Original issue's description:
> Make FTP directory parser less strict
> 
> FTP directory listing returns empty result if encoding detection or
> conversion fails. This CL puts less strict constraint on conversion
> so that the whole list can survive some filenames with broken chars
> and be shown to users. Other browsers seem to work this way.
> 
> The broken chars can happen due to wrong encoding detection, which
> cannot be avoided 100%, as reported in the bug.
> 
> BUG=676762
> R=mmenke@chromium.org, phajdan.jr@chromium.org
> 
> Review-Url: https://codereview.chromium.org/2608213002 .
> Cr-Commit-Position: refs/heads/master@{#441856}
> Committed:
https://chromium.googlesource.com/chromium/src/+/ed3ef90ecc3c01246a3d838fbeef...

TBR=phajdan.jr@chromium.org,mmenke@chromium.org,jinsukkim@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=676762
==========

to

==========
Revert of Make FTP directory parser less strict (patchset #3 id:40001 of
https://codereview.chromium.org/2608213002/ )

Reason for revert:
Fails on all platforms.

https://uberchromegw.corp.google.com/i/chromium.win/builders/Win7%20Tests%20%...

Original issue's description:
> Make FTP directory parser less strict
> 
> FTP directory listing returns empty result if encoding detection or
> conversion fails. This CL puts less strict constraint on conversion
> so that the whole list can survive some filenames with broken chars
> and be shown to users. Other browsers seem to work this way.
> 
> The broken chars can happen due to wrong encoding detection, which
> cannot be avoided 100%, as reported in the bug.
> 
> BUG=676762
> R=mmenke@chromium.org, phajdan.jr@chromium.org
> 
> Review-Url: https://codereview.chromium.org/2608213002 .
> Cr-Commit-Position: refs/heads/master@{#441856}
> Committed:
https://chromium.googlesource.com/chromium/src/+/ed3ef90ecc3c01246a3d838fbeef...

TBR=phajdan.jr@chromium.org,mmenke@chromium.org,jinsukkim@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=676762

Review-Url: https://codereview.chromium.org/2614103002
Cr-Commit-Position: refs/heads/master@{#441866}
Committed:
https://chromium.googlesource.com/chromium/src/+/9c484365968491ca416461d7c982...
==========

[commit-bot: I haz the power, 2017-01-06 03:58:59]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/9c484365968491ca416461d7c982...

[mmenke, 2017-01-06 04:02:26]

And for those viewing at home, a sample failure that is open source: 
https://build.chromium.org/p/chromium.mac/buildstatus?builder=Mac10.9%20Tests...