Migrate to enum-based constructor for callsites using non-inlined bools.

Migrate WaitableEvent to enum-based constructor for callsites using non-inlined bools.

As expected: clang-tidy couldn't auto-refactor those non-inlined bools

BUG=612843
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
NO_DEPENDENCY_CHECKS=true
TBR=Have owners approval, skipping NO_DEPENDENCY_CHECKS bug for presubmit (http://crbug.com/616912)

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

[gab, 2016-06-08 18:43:06]

Description was changed from

==========
Migrate to enum-based constructor for callsites using non-inlined bools.

BUG=
==========

to

==========
Migrate to enum-based constructor for callsites using non-inlined bools.

BUG=
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[gab, 2016-06-08 18:43:38]

Description was changed from

==========
Migrate to enum-based constructor for callsites using non-inlined bools.

BUG=
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
Migrate WaitableEvent to enum-based constructor for callsites using non-inlined
bools.

BUG=612843
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[gab, 2016-06-08 18:44:17]

Description was changed from

==========
Migrate WaitableEvent to enum-based constructor for callsites using non-inlined
bools.

BUG=612843
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
Migrate WaitableEvent to enum-based constructor for callsites using non-inlined
bools.

As expected: clang-tidy couldn't auto-refactor those non-inlined bools

BUG=612843
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[gab, 2016-06-08 18:46:11]

gab@chromium.org changed reviewers:
+ enne@chromium.org, fsamuel@chromium.org

[gab, 2016-06-08 18:46:12]

Tiny mechanical change:

@fsamuel for mus/gpu
@enne for cc/layers

Thanks,
Gab

[Fady Samuel, 2016-06-08 18:47:46]

mus lgtm

[enne (OOO), 2016-06-08 19:01:11]

lgtm

[gab, 2016-06-08 19:01:22]

Description was changed from

==========
Migrate WaitableEvent to enum-based constructor for callsites using non-inlined
bools.

As expected: clang-tidy couldn't auto-refactor those non-inlined bools

BUG=612843
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
Migrate WaitableEvent to enum-based constructor for callsites using non-inlined
bools.

As expected: clang-tidy couldn't auto-refactor those non-inlined bools

BUG=612843
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
NO_DEPENDENCY_CHECKS=true
==========

[gab, 2016-06-08 19:02:31]

Description was changed from

==========
Migrate WaitableEvent to enum-based constructor for callsites using non-inlined
bools.

As expected: clang-tidy couldn't auto-refactor those non-inlined bools

BUG=612843
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
NO_DEPENDENCY_CHECKS=true
==========

to

==========
Migrate WaitableEvent to enum-based constructor for callsites using non-inlined
bools.

As expected: clang-tidy couldn't auto-refactor those non-inlined bools

BUG=612843
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
NO_DEPENDENCY_CHECKS=true
TBR=Have owners skipping NO_DEPENDENCY_CHECKS bug for presubmit
(http://crbug.com/616912)
==========

[gab, 2016-06-08 19:02:42]

Description was changed from

==========
Migrate WaitableEvent to enum-based constructor for callsites using non-inlined
bools.

As expected: clang-tidy couldn't auto-refactor those non-inlined bools

BUG=612843
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
NO_DEPENDENCY_CHECKS=true
TBR=Have owners skipping NO_DEPENDENCY_CHECKS bug for presubmit
(http://crbug.com/616912)
==========

to

==========
Migrate WaitableEvent to enum-based constructor for callsites using non-inlined
bools.

As expected: clang-tidy couldn't auto-refactor those non-inlined bools

BUG=612843
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
NO_DEPENDENCY_CHECKS=true
TBR=Have owners approval, skipping NO_DEPENDENCY_CHECKS bug for presubmit
(http://crbug.com/616912)
==========

[gab, 2016-06-08 19:02:45]

The CQ bit was checked by gab@chromium.org

[commit-bot: I haz the power, 2016-06-08 19:03:33]

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

[commit-bot: I haz the power, 2016-06-08 20:13:40]

Description was changed from

==========
Migrate WaitableEvent to enum-based constructor for callsites using non-inlined
bools.

As expected: clang-tidy couldn't auto-refactor those non-inlined bools

BUG=612843
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
NO_DEPENDENCY_CHECKS=true
TBR=Have owners approval, skipping NO_DEPENDENCY_CHECKS bug for presubmit
(http://crbug.com/616912)
==========

to

==========
Migrate WaitableEvent to enum-based constructor for callsites using non-inlined
bools.

As expected: clang-tidy couldn't auto-refactor those non-inlined bools

BUG=612843
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
NO_DEPENDENCY_CHECKS=true
TBR=Have owners approval, skipping NO_DEPENDENCY_CHECKS bug for presubmit
(http://crbug.com/616912)
==========

[commit-bot: I haz the power, 2016-06-08 20:13:43]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-08 20:15:51]

Description was changed from

==========
Migrate WaitableEvent to enum-based constructor for callsites using non-inlined
bools.

As expected: clang-tidy couldn't auto-refactor those non-inlined bools

BUG=612843
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
NO_DEPENDENCY_CHECKS=true
TBR=Have owners approval, skipping NO_DEPENDENCY_CHECKS bug for presubmit
(http://crbug.com/616912)
==========

to

==========
Migrate WaitableEvent to enum-based constructor for callsites using non-inlined
bools.

As expected: clang-tidy couldn't auto-refactor those non-inlined bools

BUG=612843
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
NO_DEPENDENCY_CHECKS=true
TBR=Have owners approval, skipping NO_DEPENDENCY_CHECKS bug for presubmit
(http://crbug.com/616912)

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

[commit-bot: I haz the power, 2016-06-08 20:15:52]

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