Remove boolean-based WaitableEvent constructor.

Remove boolean-based WaitableEvent constructor.

BUG=612843
TBR=danakj@chromium.org

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

[gab, 2016-06-06 13:21:12]

Patchset #2 (id:20001) has been deleted

[gab, 2016-06-06 13:21:16]

Patchset #1 (id:1) has been deleted

[gab, 2016-06-07 21:58:50]

The CQ bit was checked by gab@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-06-07 21:59:26]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2030683005/40001

[commit-bot: I haz the power, 2016-06-07 22:13:39]

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

[commit-bot: I haz the power, 2016-06-07 22:13:40]

Dry run: Try jobs failed on following builders:
  chromeos_x86-generic_chromium_compile_only_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_x86-ge...)

[gab, 2016-06-08 22:42:28]

The CQ bit was checked by gab@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-06-08 22:42:58]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2030683005/40001

[commit-bot: I haz the power, 2016-06-08 22:56:17]

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

[commit-bot: I haz the power, 2016-06-08 22:56:18]

Dry run: Try jobs failed on following builders:
  mac_chromium_gn_rel on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_gn_r...)
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

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

The CQ bit was checked by gab@chromium.org to run a CQ dry run

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

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2030683005/40001

[commit-bot: I haz the power, 2016-06-09 12:29:39]

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

[commit-bot: I haz the power, 2016-06-09 12:29:40]

Dry run: Try jobs failed on following builders:
  chromeos_daisy_chromium_compile_only_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_daisy_...)

[gab, 2016-06-09 14:51:04]

The CQ bit was checked by gab@chromium.org to run a CQ dry run

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

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2030683005/40001

[gab, 2016-06-09 14:53:08]

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

[gab, 2016-06-09 14:53:08]

Dana PTAL, I will TBR this the minute it's green (happy to do follow-up for any
post-commit comments just don't want new usage of old constructor to sneak in!)

[commit-bot: I haz the power, 2016-06-09 15:06:44]

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

[commit-bot: I haz the power, 2016-06-09 15:06:45]

Dry run: Try jobs failed on following builders:
  chromeos_amd64-generic_chromium_compile_only_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64-...)

[gab, 2016-06-09 17:25:16]

Description was changed from

==========
Remove boolean-based WaitableEvent constructor.

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

to

==========
Remove boolean-based WaitableEvent constructor.

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

[gab, 2016-06-09 17:26:10]

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

[gab, 2016-06-09 17:26:10]

Francois PTAL, I'll TBR Dana in order to land this while it's green (if it does
turn green that is!)

[gab, 2016-06-09 17:26:18]

The CQ bit was checked by gab@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-06-09 17:26:44]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2030683005/40001

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

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

[commit-bot: I haz the power, 2016-06-09 18:08:19]

Dry run: This issue passed the CQ dry run.

[danakj, 2016-06-09 18:25:45]

LGTM!

https://codereview.chromium.org/2030683005/diff/40001/base/synchronization/wa...
File base/synchronization/waitable_event_posix.cc (right):

https://codereview.chromium.org/2030683005/diff/40001/base/synchronization/wa...
base/synchronization/waitable_event_posix.cc:46: WaitableEvent::~WaitableEvent()
= default;
thanks :)

[gab, 2016-06-09 18:27:05]

The CQ bit was checked by gab@chromium.org

[commit-bot: I haz the power, 2016-06-09 18:27:48]

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

[commit-bot: I haz the power, 2016-06-09 18:41:35]

Description was changed from

==========
Remove boolean-based WaitableEvent constructor.

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

to

==========
Remove boolean-based WaitableEvent constructor.

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

[commit-bot: I haz the power, 2016-06-09 18:41:36]

Committed patchset #1 (id:40001)

[commit-bot: I haz the power, 2016-06-09 18:41:45]

CQ bit was unchecked

[commit-bot: I haz the power, 2016-06-09 18:46:16]

Description was changed from

==========
Remove boolean-based WaitableEvent constructor.

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

to

==========
Remove boolean-based WaitableEvent constructor.

BUG=612843
TBR=danakj@chromium.org

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

[commit-bot: I haz the power, 2016-06-09 18:46:17]

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