Mojo: POSIX: Add a PlatformChannelSendmsgWithHandles().

Mojo: POSIX: Add a PlatformChannelSendmsgWithHandles().

This wraps sendmsg() and allows FDs to be sent. It will replace
PlatformChannelSendHandles().

R=yzshen@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=272203

[yzshen1, 2014-05-21 23:50:02]

LGTM

https://codereview.chromium.org/297803008/diff/20001/mojo/embedder/platform_c...
File mojo/embedder/platform_channel_utils_posix.cc (right):

https://codereview.chromium.org/297803008/diff/20001/mojo/embedder/platform_c...
mojo/embedder/platform_channel_utils_posix.cc:86:
DCHECK_GT(num_platform_handles, 0u);
nit: It might be good to also DCHECK that it is not greater than
kPlatformChannelMaxNumHandles.

[viettrungluu, 2014-05-22 00:21:44]

Thanks.

https://codereview.chromium.org/297803008/diff/20001/mojo/embedder/platform_c...
File mojo/embedder/platform_channel_utils_posix.cc (right):

https://codereview.chromium.org/297803008/diff/20001/mojo/embedder/platform_c...
mojo/embedder/platform_channel_utils_posix.cc:86:
DCHECK_GT(num_platform_handles, 0u);
On 2014/05/21 23:50:02, yzshen1 wrote:
> nit: It might be good to also DCHECK that it is not greater than
> kPlatformChannelMaxNumHandles.

Done.

[viettrungluu, 2014-05-22 00:21:47]

The CQ bit was checked by viettrungluu@chromium.org

[commit-bot: I haz the power, 2014-05-22 00:25:35]

CQ is trying da patch. Follow status at

https://chromium-status.appspot.com/cq/viettrungluu@chromium.org/297803008/40001

[commit-bot: I haz the power, 2014-05-22 09:37:08]

FYI, CQ is re-trying this CL (attempt #1).
Please consider checking whether the failures are real,
and report flakes to chrome-troopers@google.com.
The failing builders are:
  win_gpu_triggered_tests on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/win_gpu_triggered...)

[commit-bot: I haz the power, 2014-05-22 09:39:45]

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

[commit-bot: I haz the power, 2014-05-22 09:39:45]

Try jobs failed on following builders:
  win_gpu_triggered_tests on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/win_gpu_triggered...)

[viettrungluu, 2014-05-22 16:36:55]

Committed patchset #3 manually as r272203 (presubmit successful).