Use RaceChecker instead of ThreadChecker in remote_bitrate_estimator.

Use RaceChecker instead of ThreadChecker in remote_bitrate_estimator.

BUG=webrtc:7826
Review-Url: https://codereview.webrtc.org/3006173002
Cr-Commit-Position: refs/heads/master@{#19778}
Committed: https://chromium.googlesource.com/external/webrtc/+/423227306112c4011a538bf87ae7f2534f39f724

[erikvarga1, 2017-09-06 14:48:49]

Description was changed from

==========
Use RaceChecker instead of ThreadChecker in remote_bitrate_estimator.

BUG=7826
==========

to

==========
Use RaceChecker instead of ThreadChecker in remote_bitrate_estimator.

BUG=webrtc:7826
==========

[erikvarga1, 2017-09-06 14:53:03]

The CQ bit was checked by erikvarga@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2017-09-06 14:53:10]

Dry run: CQ is trying da patch.

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

[erikvarga1, 2017-09-06 15:03:14]

erikvarga@webrtc.org changed reviewers:
+ danilchap@webrtc.org

[danilchap, 2017-09-06 15:15:44]

lgtm

https://codereview.webrtc.org/3006173002/diff/1/webrtc/modules/remote_bitrate...
File
webrtc/modules/remote_bitrate_estimator/remote_bitrate_estimator_abs_send_time.h
(right):

https://codereview.webrtc.org/3006173002/diff/1/webrtc/modules/remote_bitrate...
webrtc/modules/remote_bitrate_estimator/remote_bitrate_estimator_abs_send_time.h:28:
#include "webrtc/rtc_base/race_checker.h"
move one line up (or run git cl format)

[erikvarga1, 2017-09-06 15:21:28]

https://codereview.webrtc.org/3006173002/diff/1/webrtc/modules/remote_bitrate...
File
webrtc/modules/remote_bitrate_estimator/remote_bitrate_estimator_abs_send_time.h
(right):

https://codereview.webrtc.org/3006173002/diff/1/webrtc/modules/remote_bitrate...
webrtc/modules/remote_bitrate_estimator/remote_bitrate_estimator_abs_send_time.h:28:
#include "webrtc/rtc_base/race_checker.h"
On 2017/09/06 15:15:44, danilchap wrote:
> move one line up (or run git cl format)

Done.

[erikvarga1, 2017-09-06 15:22:22]

erikvarga@webrtc.org changed reviewers:
+ stefan@webrtc.org

[erikvarga1, 2017-09-06 15:23:21]

Stefan, can you also take a look at this?

[stefan-webrtc, 2017-09-11 15:02:11]

lgtm

[erikvarga1, 2017-09-11 15:51:58]

The CQ bit was checked by erikvarga@webrtc.org

[erikvarga1, 2017-09-11 15:51:58]

The patchset sent to the CQ was uploaded after l-g-t-m from danilchap@webrtc.org
Link to the patchset: https://codereview.webrtc.org/3006173002/#ps20001 (title:
"Patchset 1.")

[commit-bot: I haz the power, 2017-09-11 15:52:05]

CQ is trying da patch.

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

[commit-bot: I haz the power, 2017-09-11 16:23:37]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1505145118113920,
"parent_rev": "98b1b7d59e5b9eab61211d3b13dab87cb3bb18ce", "commit_rev":
"423227306112c4011a538bf87ae7f2534f39f724"}

[commit-bot: I haz the power, 2017-09-11 16:23:45]

Description was changed from

==========
Use RaceChecker instead of ThreadChecker in remote_bitrate_estimator.

BUG=webrtc:7826
==========

to

==========
Use RaceChecker instead of ThreadChecker in remote_bitrate_estimator.

BUG=webrtc:7826

Review-Url: https://codereview.webrtc.org/3006173002
Cr-Commit-Position: refs/heads/master@{#19778}
Committed:
https://chromium.googlesource.com/external/webrtc/+/423227306112c4011a538bf87...
==========

[commit-bot: I haz the power, 2017-09-11 16:23:46]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/external/webrtc/+/423227306112c4011a538bf87...