Fix issues with replay_process in ipc fuzzer. Make it compile on windows.

Fix issues with replay_process in ipc fuzzer. Make it compile on windows.

R=tsepez@chromium.org,mbarbella@chromium.org
BUG=450268
Committed: https://crrev.com/c0b8224c61a3705fb9e8ca3cae4118ac1573f292
Cr-Commit-Position: refs/heads/master@{#312643}

[inferno, 2015-01-21 23:58:50]

inferno@chromium.org changed reviewers:
+ thakis@chromium.org

[inferno, 2015-01-21 23:58:50]

tsepez: for ipc_fuzzer changes.
thakis: for build/all.gyp changes

[Nico, 2015-01-22 00:12:59]

build/all.gyp lgtm

[Tom Sepez, 2015-01-22 00:13:06]

replay_process.cc LGTM

https://codereview.chromium.org/861113003/diff/1/tools/ipc_fuzzer/mutate/cf_p...
File tools/ipc_fuzzer/mutate/cf_package_builder.py (right):

https://codereview.chromium.org/861113003/diff/1/tools/ipc_fuzzer/mutate/cf_p...
tools/ipc_fuzzer/mutate/cf_package_builder.py:18: FUZZER_LIST_FOR_PLATFORM_DICT
= {
We should probably just have code to add a ".exe" later on for the windows
cases.  Having parallel lists seems clunky.

https://codereview.chromium.org/861113003/diff/1/tools/ipc_fuzzer/mutate/cf_p...
tools/ipc_fuzzer/mutate/cf_package_builder.py:27: elif
sys.platform.startswith('linux'):
nit: elif after return

[inferno, 2015-01-22 02:05:35]

On 2015/01/22 00:13:06, Tom Sepez wrote:
> replay_process.cc LGTM
> 

Added another tiny change to fix a null crasher with logging. please review.

>
https://codereview.chromium.org/861113003/diff/1/tools/ipc_fuzzer/mutate/cf_p...
> File tools/ipc_fuzzer/mutate/cf_package_builder.py (right):
> 
>
https://codereview.chromium.org/861113003/diff/1/tools/ipc_fuzzer/mutate/cf_p...
> tools/ipc_fuzzer/mutate/cf_package_builder.py:18:
FUZZER_LIST_FOR_PLATFORM_DICT
> = {
> We should probably just have code to add a ".exe" later on for the windows
> cases.  Having parallel lists seems clunky.
> 

Done.

>
https://codereview.chromium.org/861113003/diff/1/tools/ipc_fuzzer/mutate/cf_p...
> tools/ipc_fuzzer/mutate/cf_package_builder.py:27: elif
> sys.platform.startswith('linux'):
> nit: elif after return

Redone it using local var and assert.

[Tom Sepez, 2015-01-22 18:32:40]

Still lgtm

[inferno, 2015-01-22 18:34:31]

The CQ bit was checked by inferno@chromium.org

[commit-bot: I haz the power, 2015-01-22 18:35:39]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/861113003/80001

[commit-bot: I haz the power, 2015-01-22 18:41:23]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2015-01-22 18:42:57]

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