Fix IPC fuzzer ChannelMojo usage

Fix IPC fuzzer ChannelMojo usage

R=thakis@chromium.org,sammc@chromium.org
BUG=
TBR=jam@chromium.org for content

Committed: https://crrev.com/cf739d070a7d45cab64b4b6fd8e9cf085ea45a42
Cr-Commit-Position: refs/heads/master@{#380006}

[Nico, 2016-03-08 22:43:41]

lgtm stamp, thanks!

[Ken Rockot(use gerrit already), 2016-03-08 22:46:29]

Description was changed from

==========
Fix IPC fuzzer ChannelMojo usage

R=thakis@chromium.org,sammc@chromium.org
BUG=
==========

to

==========
Fix IPC fuzzer ChannelMojo usage

R=thakis@chromium.org,sammc@chromium.org
BUG=
TBR=jam@chromium.org for content
==========

[Sam McNally, 2016-03-08 22:49:04]

LGTM, but it looks like https://codereview.chromium.org/1776753002 got here
first. (My fix got bogged down in trying to test it)

https://codereview.chromium.org/1773323002/diff/1/tools/ipc_fuzzer/message_re...
File tools/ipc_fuzzer/message_replay/replay_process.cc (right):

https://codereview.chromium.org/1773323002/diff/1/tools/ipc_fuzzer/message_re...
tools/ipc_fuzzer/message_replay/replay_process.cc:71: kPrimaryIPCChannel +
base::GlobalDescriptors::kBaseDescriptor);
I think kMojoIPCChannel needs to be Set() here too.

[Ken Rockot(use gerrit already), 2016-03-08 23:48:50]

On 2016/03/08 at 22:49:04, sammc wrote:
> LGTM, but it looks like https://codereview.chromium.org/1776753002 got here
first. (My fix got bogged down in trying to test it)
> 
>
https://codereview.chromium.org/1773323002/diff/1/tools/ipc_fuzzer/message_re...
> File tools/ipc_fuzzer/message_replay/replay_process.cc (right):
> 
>
https://codereview.chromium.org/1773323002/diff/1/tools/ipc_fuzzer/message_re...
> tools/ipc_fuzzer/message_replay/replay_process.cc:71: kPrimaryIPCChannel +
base::GlobalDescriptors::kBaseDescriptor);
> I think kMojoIPCChannel needs to be Set() here too.

Done.

I went ahead and properly initialized the Mojo IPC channel.

[Ken Rockot(use gerrit already), 2016-03-08 23:49:08]

The CQ bit was checked by rockot@chromium.org

[Ken Rockot(use gerrit already), 2016-03-08 23:49:08]

The patchset sent to the CQ was uploaded after l-g-t-m from sammc@chromium.org,
thakis@chromium.org
Link to the patchset: https://codereview.chromium.org/1773323002/#ps20001
(title: "do things more better")

[commit-bot: I haz the power, 2016-03-08 23:49:37]

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

[commit-bot: I haz the power, 2016-03-09 01:03:47]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-09 01:05:48]

Description was changed from

==========
Fix IPC fuzzer ChannelMojo usage

R=thakis@chromium.org,sammc@chromium.org
BUG=
TBR=jam@chromium.org for content
==========

to

==========
Fix IPC fuzzer ChannelMojo usage

R=thakis@chromium.org,sammc@chromium.org
BUG=
TBR=jam@chromium.org for content

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

[commit-bot: I haz the power, 2016-03-09 01:05:49]

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