Fix a pair of bugs in UNIX->VMS path conversion.

Fix a pair of bugs in UNIX->VMS path conversion.

In particular:
* Fix conversion of UNIX file name "/////file" and "////" as a UNIX
  path.  Both of these were broken, due to the same bug.
* Remove a DCHECK that would trigger on "//" as a file name. It's a
  valid UNIX path, if not a file name, and can still be passed through
  the file name conversion code.  Since there's no code to prevent
  that from happening, and the conversion code handles "/", the DCHECK
  made no sense.

BUG=675111
Committed: https://crrev.com/dbb6723c81a3d595aa3cc593acac9bb93b328398
Cr-Commit-Position: refs/heads/master@{#441423}

[mmenke, 2017-01-04 18:01:32]

The CQ bit was checked by mmenke@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-04 18:01:59]

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

[mmenke, 2017-01-04 18:09:53]

Description was changed from

==========
Fix a pair of bugs in UNIX->VMS path conversion.

In particular:
* Remove a DCHECK that would trigger on "//" as a file name. It's a
    valid UNIX path, if not a file name, and can still be passed through
    the file name conversion code.  Since there's no code to prevent
    that from happening, and the conversion code handles "/", the DCHECK
    made no sense.
* Fix conversion of UNIX file name "/////file" and "////" as a UNIX
    path.  Both of these were broken, due to a single bug.

BUG=675111
==========

to

==========
Fix a pair of bugs in UNIX->VMS path conversion.

In particular:
* Fix conversion of UNIX file name "/////file" and "////" as a UNIX
  path.  Both of these were broken, due to the same bug.
* Remove a DCHECK that would trigger on "//" as a file name. It's a
  valid UNIX path, if not a file name, and can still be passed through
  the file name conversion code.  Since there's no code to prevent
  that from happening, and the conversion code handles "/", the DCHECK
  made no sense.

BUG=675111
==========

[mmenke, 2017-01-04 18:48:05]

mmenke@chromium.org changed reviewers:
+ eroman@chromium.org

[mmenke, 2017-01-04 18:48:06]

¡Feliz Año Nuevo!

[commit-bot: I haz the power, 2017-01-04 18:58:53]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-01-04 18:58:54]

Dry run: This issue passed the CQ dry run.

[eroman, 2017-01-04 18:59:35]

lgtm

[mmenke, 2017-01-04 19:00:57]

The CQ bit was checked by mmenke@chromium.org

[commit-bot: I haz the power, 2017-01-04 19:01:30]

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-04 19:09:07]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1483556457842950, "parent_rev":
"63604b70c1dc39c8aec8a41e0fe07d4c1645cb68", "commit_rev":
"78b554c8b269eb6a2341c1cf7fd2c0c117dc4c78"}

[commit-bot: I haz the power, 2017-01-04 19:09:39]

Description was changed from

==========
Fix a pair of bugs in UNIX->VMS path conversion.

In particular:
* Fix conversion of UNIX file name "/////file" and "////" as a UNIX
  path.  Both of these were broken, due to the same bug.
* Remove a DCHECK that would trigger on "//" as a file name. It's a
  valid UNIX path, if not a file name, and can still be passed through
  the file name conversion code.  Since there's no code to prevent
  that from happening, and the conversion code handles "/", the DCHECK
  made no sense.

BUG=675111
==========

to

==========
Fix a pair of bugs in UNIX->VMS path conversion.

In particular:
* Fix conversion of UNIX file name "/////file" and "////" as a UNIX
  path.  Both of these were broken, due to the same bug.
* Remove a DCHECK that would trigger on "//" as a file name. It's a
  valid UNIX path, if not a file name, and can still be passed through
  the file name conversion code.  Since there's no code to prevent
  that from happening, and the conversion code handles "/", the DCHECK
  made no sense.

BUG=675111

Review-Url: https://codereview.chromium.org/2610973002
==========

[commit-bot: I haz the power, 2017-01-04 19:09:39]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2017-01-04 19:12:24]

Description was changed from

==========
Fix a pair of bugs in UNIX->VMS path conversion.

In particular:
* Fix conversion of UNIX file name "/////file" and "////" as a UNIX
  path.  Both of these were broken, due to the same bug.
* Remove a DCHECK that would trigger on "//" as a file name. It's a
  valid UNIX path, if not a file name, and can still be passed through
  the file name conversion code.  Since there's no code to prevent
  that from happening, and the conversion code handles "/", the DCHECK
  made no sense.

BUG=675111

Review-Url: https://codereview.chromium.org/2610973002
==========

to

==========
Fix a pair of bugs in UNIX->VMS path conversion.

In particular:
* Fix conversion of UNIX file name "/////file" and "////" as a UNIX
  path.  Both of these were broken, due to the same bug.
* Remove a DCHECK that would trigger on "//" as a file name. It's a
  valid UNIX path, if not a file name, and can still be passed through
  the file name conversion code.  Since there's no code to prevent
  that from happening, and the conversion code handles "/", the DCHECK
  made no sense.

BUG=675111

Committed: https://crrev.com/dbb6723c81a3d595aa3cc593acac9bb93b328398
Cr-Commit-Position: refs/heads/master@{#441423}
==========

[commit-bot: I haz the power, 2017-01-04 19:12:25]

Patchset 1 (id:??) landed as
https://crrev.com/dbb6723c81a3d595aa3cc593acac9bb93b328398
Cr-Commit-Position: refs/heads/master@{#441423}