Override rtc::Event with Chromium's implementation

Preparation of overriding WebRTC's rtc::Event with Chromium's implementation.
This change will give WebRTC the benefits of Chromium's checks and integration with thread and sequence runners (e.g. io and waitable checks).

Followup changes include enabling the code from the WebRTC side (requires build file changes) and integrating the TaskQueue implementation in Chromium with sequence runners.

BUG=689520
NOTRY=true

Review-Url: https://codereview.chromium.org/2975553002
Cr-Commit-Position: refs/heads/master@{#486737}
Committed: https://chromium.googlesource.com/chromium/src/+/07019da77f12dc06fd8a2e66061c8a79c90bc439

[tommi1, 2017-07-13 21:08:50]

Description was changed from

==========
Override rtc::Event with Chromium's implementation

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

to

==========
Preparation of overriding WebRTC's rtc::Event with Chromium's implementation.
This change will give WebRTC the benefits of Chromium's checks and integration
with thread and sequence runners (e.g. io and waitable checks).

Followup changes include enabling the code from the WebRTC side (requires build
file changes) and integrating the TaskQueue implementation in Chromium with
sequence runners.

BUG=689520
==========

[tommi1, 2017-07-13 21:08:50]

tommi@google.com changed reviewers:
+ grunell@chromium.org

[tommi1, 2017-07-13 21:09:02]

tommi@google.com changed reviewers:
+ tommi@google.com

[tommi1, 2017-07-13 21:09:56]

Description was changed from

==========
Preparation of overriding WebRTC's rtc::Event with Chromium's implementation.
This change will give WebRTC the benefits of Chromium's checks and integration
with thread and sequence runners (e.g. io and waitable checks).

Followup changes include enabling the code from the WebRTC side (requires build
file changes) and integrating the TaskQueue implementation in Chromium with
sequence runners.

BUG=689520
==========

to

==========
Preparation of overriding WebRTC's rtc::Event with Chromium's implementation.
This change will give WebRTC the benefits of Chromium's checks and integration
with thread and sequence runners (e.g. io and waitable checks).

Followup changes include enabling the code from the WebRTC side (requires build
file changes) and integrating the TaskQueue implementation in Chromium with
sequence runners.

BUG=689520
NOTRY=true
==========

[tommi (sloooow) - chröme, 2017-07-13 21:10:24]

tommi@chromium.org changed reviewers:
- tommi@google.com

[tommi (sloooow) - chröme, 2017-07-13 21:10:42]

Hej Henrik - please take a look. The code is inactive as is, hence the 'notry'.

[Henrik Grunell, 2017-07-14 10:25:48]

Looks good, but should it be webrtc/rtc_base/event.h/cc that are overridden?
webrtc/base/event.h is deprecated.

https://cs.chromium.org/chromium/src/third_party/webrtc/base/event.h?q=webrtc...

[tommi (sloooow) - chröme, 2017-07-14 10:40:56]

Move and update files

[tommi (sloooow) - chröme, 2017-07-14 10:41:12]

ah indeed! thanks for catching that (I started this CL one week ago, I guess
things change quickly!).  I also see that task_queue has already been moved.
Nice.  Done.

[Henrik Grunell, 2017-07-14 10:46:43]

lgtm

[tommi (sloooow) - chröme, 2017-07-14 11:37:06]

The CQ bit was checked by tommi@chromium.org

[commit-bot: I haz the power, 2017-07-14 11:37:19]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-07-14 11:42:09]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1500032226290170,
"parent_rev": "765ed10683220896163a799d285ce99567d5bd80", "commit_rev":
"07019da77f12dc06fd8a2e66061c8a79c90bc439"}

[commit-bot: I haz the power, 2017-07-14 11:42:22]

Description was changed from

==========
Preparation of overriding WebRTC's rtc::Event with Chromium's implementation.
This change will give WebRTC the benefits of Chromium's checks and integration
with thread and sequence runners (e.g. io and waitable checks).

Followup changes include enabling the code from the WebRTC side (requires build
file changes) and integrating the TaskQueue implementation in Chromium with
sequence runners.

BUG=689520
NOTRY=true
==========

to

==========
Preparation of overriding WebRTC's rtc::Event with Chromium's implementation.
This change will give WebRTC the benefits of Chromium's checks and integration
with thread and sequence runners (e.g. io and waitable checks).

Followup changes include enabling the code from the WebRTC side (requires build
file changes) and integrating the TaskQueue implementation in Chromium with
sequence runners.

BUG=689520
NOTRY=true

Review-Url: https://codereview.chromium.org/2975553002
Cr-Commit-Position: refs/heads/master@{#486737}
Committed:
https://chromium.googlesource.com/chromium/src/+/07019da77f12dc06fd8a2e66061c...
==========

[commit-bot: I haz the power, 2017-07-14 11:42:23]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/07019da77f12dc06fd8a2e66061c...

[tommi (sloooow) - chröme, 2017-07-14 11:50:41]

A revert of this CL (patchset #2 id:20001) has been created in
https://codereview.chromium.org/2981863002/ by tommi@chromium.org.

The reason for reverting is: Reverting temporarily until the webrtc change can
be rolled with it..

[findit-for-me, 2017-07-14 11:59:28]

Findit (https://goo.gl/kROfz5) confirmed this CL at revision 486737 as the
culprit for
failures in the build cycles as shown on:
https://findit-for-me.appspot.com/waterfall/culprit?key=ag9zfmZpbmRpdC1mb3Itb...