Allow custom security descriptors when creating named pipes on Windows.

Allow custom security descriptors when creating named pipes on Windows.

Remoting requires control over the security descriptor used for its
named pipes. This adds this as an option when creating server handles
from named pipes on Windows.

BUG=604282
Committed: https://crrev.com/404ec514001d24ae6e6d14495648645569f0bbd3
Cr-Commit-Position: refs/heads/master@{#427266}

[Sam McNally, 2016-10-24 04:07:43]

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

[Sam McNally, 2016-10-24 04:07:46]

Patchset #1 (id:1) has been deleted

[commit-bot: I haz the power, 2016-10-24 04:07:54]

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

[Sam McNally, 2016-10-24 04:12:25]

Patchset #1 (id:20001) has been deleted

[Sam McNally, 2016-10-24 04:16:04]

Patchset #1 (id:40001) has been deleted

[Sam McNally, 2016-10-24 04:16:39]

Patchset #1 (id:60001) has been deleted

[Sam McNally, 2016-10-24 04:16:52]

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

[commit-bot: I haz the power, 2016-10-24 04:17:05]

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

[Sam McNally, 2016-10-24 04:36:37]

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

[commit-bot: I haz the power, 2016-10-24 04:36:46]

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

[Sam McNally, 2016-10-24 04:36:50]

Patchset #2 (id:100001) has been deleted

[commit-bot: I haz the power, 2016-10-24 04:38:38]

Dry run: Try jobs failed on following builders:
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)

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

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

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

Dry run: This issue passed the CQ dry run.

[Sam McNally, 2016-10-24 05:35:15]

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

[Sam McNally, 2016-10-24 05:35:17]

Patchset #1 (id:80001) has been deleted

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

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

[Sam McNally, 2016-10-24 05:47:19]

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

[commit-bot: I haz the power, 2016-10-24 05:47:33]

Dry run: 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, 2016-10-24 06:52:43]

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

[commit-bot: I haz the power, 2016-10-24 06:52:44]

Dry run: This issue passed the CQ dry run.

[Sam McNally, 2016-10-24 22:30:41]

Patchset #1 (id:120001) has been deleted

[Sam McNally, 2016-10-24 22:34:04]

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

[Sam McNally, 2016-10-24 22:34:21]

Patchset #1 (id:140001) has been deleted

[commit-bot: I haz the power, 2016-10-24 22:34:38]

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

[Sam McNally, 2016-10-24 23:25:27]

sammc@chromium.org changed reviewers:
+ rockot@chromium.org, thestig@chromium.org

[Sam McNally, 2016-10-24 23:25:28]

+thestig for //chrome

+rockot for //mojo

[commit-bot: I haz the power, 2016-10-24 23:36:32]

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

[commit-bot: I haz the power, 2016-10-24 23:36:33]

Dry run: Try jobs failed on following builders:
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)

[Lei Zhang, 2016-10-24 23:45:20]

On 2016/10/24 23:25:28, Sam McNally wrote:
> +thestig for //chrome
> 
> +rockot for //mojo

Looks reasonable, but I defer to rockot.

[Ken Rockot(use gerrit already), 2016-10-25 01:50:20]

lgtm

[Lei Zhang, 2016-10-25 01:57:15]

lgtm

BTW, android_clang_dbg_recipe has been acting up, so it's not you.

[Sam McNally, 2016-10-25 03:02:42]

The CQ bit was checked by sammc@chromium.org

[commit-bot: I haz the power, 2016-10-25 03:02:58]

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, 2016-10-25 03:39:44]

Committed patchset #1 (id:160001)

[commit-bot: I haz the power, 2016-10-25 03:46:42]

Description was changed from

==========
Allow custom security descriptors when creating named pipes on Windows.

Remoting requires control over the security descriptor used for its
named pipes. This adds this as an option when creating server handles
from named pipes on Windows.

BUG=604282
==========

to

==========
Allow custom security descriptors when creating named pipes on Windows.

Remoting requires control over the security descriptor used for its
named pipes. This adds this as an option when creating server handles
from named pipes on Windows.

BUG=604282

Committed: https://crrev.com/404ec514001d24ae6e6d14495648645569f0bbd3
Cr-Commit-Position: refs/heads/master@{#427266}
==========

[commit-bot: I haz the power, 2016-10-25 03:46:43]

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