More WaitableEvent stragglers migrated to enum-based constructor

More WaitableEvent stragglers migrated to enum-based constructor

Previous manual regex was finding "new WaitableEvent(...)" and
"...event_(...)", i.e. heap allocs and initializer lists but
was missing stack variables.

This change uses the following regex to catch those and manually fix them:
"WaitableEvent\ [\w]+\((true|false),"

Also fixed missing space presubmit error in media_message_fifo_unittest.cc

BUG=612843
R=fdoray@chromium.org
TBR=danakj@chromium.org

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

[gab, 2016-06-09 12:54:44]

Description was changed from

==========
More WaitableEvent stragglers migrated to enum-based constructor

Previous manual regex was finding "new WaitableEvent(...)" and
"...event_(...)", i.e. heap allocs and initializer lists but
was missing stack variables.

This change uses the following regex to catch those and manually fix them:
"WaitableEvent\ [\w]+\((true|false),"

BUG=612843
==========

to

==========
More WaitableEvent stragglers migrated to enum-based constructor

Previous manual regex was finding "new WaitableEvent(...)" and
"...event_(...)", i.e. heap allocs and initializer lists but
was missing stack variables.

This change uses the following regex to catch those and manually fix them:
"WaitableEvent\ [\w]+\((true|false),"

Also fixed missing space presubmit error in media_message_fifo_unittest.cc

BUG=612843
==========

[gab, 2016-06-09 12:55:28]

Description was changed from

==========
More WaitableEvent stragglers migrated to enum-based constructor

Previous manual regex was finding "new WaitableEvent(...)" and
"...event_(...)", i.e. heap allocs and initializer lists but
was missing stack variables.

This change uses the following regex to catch those and manually fix them:
"WaitableEvent\ [\w]+\((true|false),"

Also fixed missing space presubmit error in media_message_fifo_unittest.cc

BUG=612843
==========

to

==========
More WaitableEvent stragglers migrated to enum-based constructor

Previous manual regex was finding "new WaitableEvent(...)" and
"...event_(...)", i.e. heap allocs and initializer lists but
was missing stack variables.

This change uses the following regex to catch those and manually fix them:
"WaitableEvent\ [\w]+\((true|false),"

Also fixed missing space presubmit error in media_message_fifo_unittest.cc

BUG=612843
R=fdoray@chromium.org
TBR=danakj@chromium.org
==========

[gab, 2016-06-09 12:55:28]

gab@chromium.org changed reviewers:
+ danakj@chromium.org, fdoray@chromium.org

[gab, 2016-06-09 12:56:01]

As for previous ones, sending to CQ with TBR danakj.

Francois please double check me on manual change (self-review lgtm).

[gab, 2016-06-09 12:56:06]

The CQ bit was checked by gab@chromium.org

[commit-bot: I haz the power, 2016-06-09 12:56:14]

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

[fdoray, 2016-06-09 13:25:54]

lgtm

[commit-bot: I haz the power, 2016-06-09 14:18:25]

Description was changed from

==========
More WaitableEvent stragglers migrated to enum-based constructor

Previous manual regex was finding "new WaitableEvent(...)" and
"...event_(...)", i.e. heap allocs and initializer lists but
was missing stack variables.

This change uses the following regex to catch those and manually fix them:
"WaitableEvent\ [\w]+\((true|false),"

Also fixed missing space presubmit error in media_message_fifo_unittest.cc

BUG=612843
R=fdoray@chromium.org
TBR=danakj@chromium.org
==========

to

==========
More WaitableEvent stragglers migrated to enum-based constructor

Previous manual regex was finding "new WaitableEvent(...)" and
"...event_(...)", i.e. heap allocs and initializer lists but
was missing stack variables.

This change uses the following regex to catch those and manually fix them:
"WaitableEvent\ [\w]+\((true|false),"

Also fixed missing space presubmit error in media_message_fifo_unittest.cc

BUG=612843
R=fdoray@chromium.org
TBR=danakj@chromium.org
==========

[commit-bot: I haz the power, 2016-06-09 14:18:26]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-09 14:18:33]

CQ bit was unchecked

[commit-bot: I haz the power, 2016-06-09 14:20:20]

Description was changed from

==========
More WaitableEvent stragglers migrated to enum-based constructor

Previous manual regex was finding "new WaitableEvent(...)" and
"...event_(...)", i.e. heap allocs and initializer lists but
was missing stack variables.

This change uses the following regex to catch those and manually fix them:
"WaitableEvent\ [\w]+\((true|false),"

Also fixed missing space presubmit error in media_message_fifo_unittest.cc

BUG=612843
R=fdoray@chromium.org
TBR=danakj@chromium.org
==========

to

==========
More WaitableEvent stragglers migrated to enum-based constructor

Previous manual regex was finding "new WaitableEvent(...)" and
"...event_(...)", i.e. heap allocs and initializer lists but
was missing stack variables.

This change uses the following regex to catch those and manually fix them:
"WaitableEvent\ [\w]+\((true|false),"

Also fixed missing space presubmit error in media_message_fifo_unittest.cc

BUG=612843
R=fdoray@chromium.org
TBR=danakj@chromium.org

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

[commit-bot: I haz the power, 2016-06-09 14:20:21]

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

[danakj, 2016-06-09 18:42:31]

LGTM