Revert of [mojo] Use a pipe path to initialise Mojo in elevated utility processes.

Revert of [mojo] Use a pipe path to initialise Mojo in elevated utility processes. (patchset #22 id:420001 of https://codereview.chromium.org/1893313003/ )

Reason for revert:
This is failing DrMemory tests. See https://build.chromium.org/p/chromium.memory.fyi/builders/Windows%20Unit%20%28DrMemory%20full%29%20%281%29/builds/4939 for details

Original issue's description:
> [mojo] Use a pipe path to initialise Mojo in elevated utility processes.
>
> Elevated processes can't be passed HANDLEs, so instead, IPC channels
> must be initialised by passing a pipe path on the command line. For security,
> passing a pipe path is only done for elevated process and no other process
> types.
>
> BUG=604282
>
> Committed: https://crrev.com/32ca1d648267a6ce2dae1417b6710089b2555f42
> Cr-Commit-Position: refs/heads/master@{#395533}

TBR=rockot@chromium.org,forshaw@chromium.org,nick@chromium.org,amistry@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=604282
Committed: https://crrev.com/c69c114f40bc1d477ca436cc3b981d535659e58b
Cr-Commit-Position: refs/heads/master@{#395714}

[Irfan, 2016-05-24 20:18:22]

Created Revert of [mojo] Use a pipe path to initialise Mojo in elevated utility
processes.

[Irfan, 2016-05-24 20:18:37]

Description was changed from

==========
Revert of [mojo] Use a pipe path to initialise Mojo in elevated utility
processes. (patchset #22 id:420001 of
https://codereview.chromium.org/1893313003/ )

Reason for revert:
This is failing DrMemory tests. See
https://build.chromium.org/p/chromium.memory.fyi/builders/Windows%20Unit%20%2...
for details

Original issue's description:
> [mojo] Use a pipe path to initialise Mojo in elevated utility processes.
> 
> Elevated processes can't be passed HANDLEs, so instead, IPC channels
> must be initialised by passing a pipe path on the command line. For security,
> passing a pipe path is only done for elevated process and no other process
> types.
> 
> BUG=604282
> 
> Committed: https://crrev.com/32ca1d648267a6ce2dae1417b6710089b2555f42
> Cr-Commit-Position: refs/heads/master@{#395533}

TBR=rockot@chromium.org,forshaw@chromium.org,nick@chromium.org,amistry@chromi...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=604282
==========

to

==========
Revert of [mojo] Use a pipe path to initialise Mojo in elevated utility
processes. (patchset #22 id:420001 of
https://codereview.chromium.org/1893313003/ )

Reason for revert:
This is failing DrMemory tests. See
https://build.chromium.org/p/chromium.memory.fyi/builders/Windows%20Unit%20%2...
for details

Original issue's description:
> [mojo] Use a pipe path to initialise Mojo in elevated utility processes.
> 
> Elevated processes can't be passed HANDLEs, so instead, IPC channels
> must be initialised by passing a pipe path on the command line. For security,
> passing a pipe path is only done for elevated process and no other process
> types.
> 
> BUG=604282
> 
> Committed: https://crrev.com/32ca1d648267a6ce2dae1417b6710089b2555f42
> Cr-Commit-Position: refs/heads/master@{#395533}

TBR=rockot@chromium.org,forshaw@chromium.org,nick@chromium.org,amistry@chromi...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=604282
==========

[Irfan, 2016-05-24 20:18:37]

isheriff@chromium.org changed reviewers:
+ thestig@chromium.org

[Irfan, 2016-05-24 20:28:37]

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

[commit-bot: I haz the power, 2016-05-24 20:29:13]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2009033002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2009033002/1

[commit-bot: I haz the power, 2016-05-24 20:29:25]

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

[commit-bot: I haz the power, 2016-05-24 20:29:26]

Dry run: This issue passed the CQ dry run.

[Irfan, 2016-05-24 20:48:19]

Multiple UNITIALIZED READ complaints that end up at:

# 3 mojo_system_impl.dll!mojo::edk::`anonymous
namespace'::ChannelWin::WriteNoLock [mojo\edk\system\channel_win.cc:290]
# 4 mojo_system_impl.dll!mojo::edk::`anonymous namespace'::ChannelWin::Write  
[mojo\edk\system\channel_win.cc:108]

[Lei Zhang, 2016-05-24 22:24:01]

The CQ bit was checked by thestig@chromium.org

[commit-bot: I haz the power, 2016-05-24 22:24:35]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2009033002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2009033002/1

[commit-bot: I haz the power, 2016-05-24 22:24:36]

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

[commit-bot: I haz the power, 2016-05-24 22:24:37]

No L-G-T-M from a valid reviewer yet. 
CQ run can only be started by full committers or once the patch has
received an L-G-T-M from a full committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[ncarter (slow), 2016-05-24 22:32:56]

rubber stamp lgtm for revert

[ncarter (slow), 2016-05-24 22:33:03]

The CQ bit was checked by nick@chromium.org

[commit-bot: I haz the power, 2016-05-24 22:33:41]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2009033002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2009033002/1

[commit-bot: I haz the power, 2016-05-24 22:36:02]

Description was changed from

==========
Revert of [mojo] Use a pipe path to initialise Mojo in elevated utility
processes. (patchset #22 id:420001 of
https://codereview.chromium.org/1893313003/ )

Reason for revert:
This is failing DrMemory tests. See
https://build.chromium.org/p/chromium.memory.fyi/builders/Windows%20Unit%20%2...
for details

Original issue's description:
> [mojo] Use a pipe path to initialise Mojo in elevated utility processes.
> 
> Elevated processes can't be passed HANDLEs, so instead, IPC channels
> must be initialised by passing a pipe path on the command line. For security,
> passing a pipe path is only done for elevated process and no other process
> types.
> 
> BUG=604282
> 
> Committed: https://crrev.com/32ca1d648267a6ce2dae1417b6710089b2555f42
> Cr-Commit-Position: refs/heads/master@{#395533}

TBR=rockot@chromium.org,forshaw@chromium.org,nick@chromium.org,amistry@chromi...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=604282
==========

to

==========
Revert of [mojo] Use a pipe path to initialise Mojo in elevated utility
processes. (patchset #22 id:420001 of
https://codereview.chromium.org/1893313003/ )

Reason for revert:
This is failing DrMemory tests. See
https://build.chromium.org/p/chromium.memory.fyi/builders/Windows%20Unit%20%2...
for details

Original issue's description:
> [mojo] Use a pipe path to initialise Mojo in elevated utility processes.
> 
> Elevated processes can't be passed HANDLEs, so instead, IPC channels
> must be initialised by passing a pipe path on the command line. For security,
> passing a pipe path is only done for elevated process and no other process
> types.
> 
> BUG=604282
> 
> Committed: https://crrev.com/32ca1d648267a6ce2dae1417b6710089b2555f42
> Cr-Commit-Position: refs/heads/master@{#395533}

TBR=rockot@chromium.org,forshaw@chromium.org,nick@chromium.org,amistry@chromi...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=604282
==========

[commit-bot: I haz the power, 2016-05-24 22:36:03]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-05-24 22:37:53]

Description was changed from

==========
Revert of [mojo] Use a pipe path to initialise Mojo in elevated utility
processes. (patchset #22 id:420001 of
https://codereview.chromium.org/1893313003/ )

Reason for revert:
This is failing DrMemory tests. See
https://build.chromium.org/p/chromium.memory.fyi/builders/Windows%20Unit%20%2...
for details

Original issue's description:
> [mojo] Use a pipe path to initialise Mojo in elevated utility processes.
> 
> Elevated processes can't be passed HANDLEs, so instead, IPC channels
> must be initialised by passing a pipe path on the command line. For security,
> passing a pipe path is only done for elevated process and no other process
> types.
> 
> BUG=604282
> 
> Committed: https://crrev.com/32ca1d648267a6ce2dae1417b6710089b2555f42
> Cr-Commit-Position: refs/heads/master@{#395533}

TBR=rockot@chromium.org,forshaw@chromium.org,nick@chromium.org,amistry@chromi...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=604282
==========

to

==========
Revert of [mojo] Use a pipe path to initialise Mojo in elevated utility
processes. (patchset #22 id:420001 of
https://codereview.chromium.org/1893313003/ )

Reason for revert:
This is failing DrMemory tests. See
https://build.chromium.org/p/chromium.memory.fyi/builders/Windows%20Unit%20%2...
for details

Original issue's description:
> [mojo] Use a pipe path to initialise Mojo in elevated utility processes.
>
> Elevated processes can't be passed HANDLEs, so instead, IPC channels
> must be initialised by passing a pipe path on the command line. For security,
> passing a pipe path is only done for elevated process and no other process
> types.
>
> BUG=604282
>
> Committed: https://crrev.com/32ca1d648267a6ce2dae1417b6710089b2555f42
> Cr-Commit-Position: refs/heads/master@{#395533}

TBR=rockot@chromium.org,forshaw@chromium.org,nick@chromium.org,amistry@chromi...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=604282

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

[commit-bot: I haz the power, 2016-05-24 22:37:55]

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