DO NOT COMMIT (will be split in sub-components) - clang-tidy WaitableEvent refactor

DO NOT COMMIT (will be split in sub-components) - clang-tidy WaitableEvent refactor

BUG=612843
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

[gab, 2016-05-30 21:01:56]

Description was changed from

==========
DO NOT COMMIT (will be split in sub-components) - clang-tidy WaitableEvent
refactor

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

to

==========
DO NOT COMMIT (will be split in sub-components) - clang-tidy WaitableEvent
refactor

BUG=612843
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[gab, 2016-05-30 21:01:57]

Description was changed from

==========
DO NOT COMMIT (will be split in sub-components) - clang-tidy WaitableEvent
refactor

BUG=612843
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
DO NOT COMMIT (will be split in sub-components) - clang-tidy WaitableEvent
refactor

BUG=612843
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
==========

[gab, 2016-05-30 21:01:59]

Description was changed from

==========
DO NOT COMMIT (will be split in sub-components) - clang-tidy WaitableEvent
refactor

BUG=612843
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
==========

to

==========
DO NOT COMMIT (will be split in sub-components) - clang-tidy WaitableEvent
refactor

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

[gab, 2016-05-30 21:03:18]

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

[gab, 2016-05-30 21:03:20]

Etienne/Francois FYI, will be splitting this CL and send to component OWNERS for
review.

[gab, 2016-05-31 12:34:57]

FYI: patch set 1 was applying the fix across the entire codebase @r396726 on a
Linux+Release+component out dir with no build.

A few files couldn't be assessed by clang-tidy per compile failures for lack of
generated headers.

I then built the "chrome" target and re-ran the fix on the codebase which fixed
7 more files (ref. patch set 2:
https://codereview.chromium.org/2024723002/#ps20001).

[gab, 2016-06-07 15:46:01]

On 2016/05/31 12:34:57, gab wrote:
> FYI: patch set 1 was applying the fix across the entire codebase @r396726 on a
> Linux+Release+component out dir with no build.
> 
> A few files couldn't be assessed by clang-tidy per compile failures for lack
of
> generated headers.
> 
> I then built the "chrome" target and re-ran the fix on the codebase which
fixed
> 7 more files (ref. patch set 2:
> https://codereview.chromium.org/2024723002/#ps20001).

All of these landed. Closing.