[mojo-edk] Stop using the sync broker on OS X

[mojo-edk] Stop using the sync broker on OS X

The sync broker is only needed for sandboxed shared
buffer allocation on POSIX, but it's not needed for
Mach shared buffers.

This stops using the broker on OS X.

BUG=591742, 590979
Committed: https://crrev.com/018a4e46c477ada7307ed5c7dad7eb58b7da75b5
Cr-Commit-Position: refs/heads/master@{#381908}

[Ken Rockot(use gerrit already), 2016-03-18 05:12:44]

rockot@chromium.org changed reviewers:
+ amistry@chromium.org

[Anand Mistry (off Chromium), 2016-03-18 05:40:12]

LGTM

https://codereview.chromium.org/1809363002/diff/1/mojo/edk/system/node_contro...
File mojo/edk/system/node_controller.cc (right):

https://codereview.chromium.org/1809363002/diff/1/mojo/edk/system/node_contro...
mojo/edk/system/node_controller.cc:221: // TODO(amistry): Fix sync broker and
re-enable on OSX.
Please delete this comment and add 590979 to the bug line.

[Ken Rockot(use gerrit already), 2016-03-18 05:57:46]

Description was changed from

==========
[mojo-edk] Stop using the sync broker on OS X

The sync broker is only needed for sandboxed shared
buffer allocation on POSIX, but it's not needed for
Mach shared buffers.

This stops using the broker on OS X.

BUG=591742
==========

to

==========
[mojo-edk] Stop using the sync broker on OS X

The sync broker is only needed for sandboxed shared
buffer allocation on POSIX, but it's not needed for
Mach shared buffers.

This stops using the broker on OS X.

BUG=591742,590979
==========

[Ken Rockot(use gerrit already), 2016-03-18 05:57:50]

On 2016/03/18 at 05:40:12, amistry wrote:
> LGTM
> 
>
https://codereview.chromium.org/1809363002/diff/1/mojo/edk/system/node_contro...
> File mojo/edk/system/node_controller.cc (right):
> 
>
https://codereview.chromium.org/1809363002/diff/1/mojo/edk/system/node_contro...
> mojo/edk/system/node_controller.cc:221: // TODO(amistry): Fix sync broker and
re-enable on OSX.
> Please delete this comment and add 590979 to the bug line.

done + done

[Ken Rockot(use gerrit already), 2016-03-18 05:57:52]

The CQ bit was checked by rockot@chromium.org

[Ken Rockot(use gerrit already), 2016-03-18 05:57:52]

The patchset sent to the CQ was uploaded after l-g-t-m from amistry@chromium.org
Link to the patchset: https://codereview.chromium.org/1809363002/#ps20001
(title: " ")

[commit-bot: I haz the power, 2016-03-18 05:58:03]

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

[commit-bot: I haz the power, 2016-03-18 07:34:43]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-18 07:36:17]

Description was changed from

==========
[mojo-edk] Stop using the sync broker on OS X

The sync broker is only needed for sandboxed shared
buffer allocation on POSIX, but it's not needed for
Mach shared buffers.

This stops using the broker on OS X.

BUG=591742,590979
==========

to

==========
[mojo-edk] Stop using the sync broker on OS X

The sync broker is only needed for sandboxed shared
buffer allocation on POSIX, but it's not needed for
Mach shared buffers.

This stops using the broker on OS X.

BUG=591742,590979

Committed: https://crrev.com/018a4e46c477ada7307ed5c7dad7eb58b7da75b5
Cr-Commit-Position: refs/heads/master@{#381908}
==========

[commit-bot: I haz the power, 2016-03-18 07:36:18]

Patchset 2 (id:??) landed as
https://crrev.com/018a4e46c477ada7307ed5c7dad7eb58b7da75b5
Cr-Commit-Position: refs/heads/master@{#381908}