diff options
author | Tommi <tommi@webrtc.org> | 2020-05-05 15:35:45 +0200 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2020-05-08 07:24:39 +0000 |
commit | 553c869c58ef9c6b09da2e3835ce3adfcac8d540 (patch) | |
tree | 4cfc6e67a421b07c8668bfc2439b614148855bde /test | |
parent | 04e1bab1b324e08b8dec4bf617e2afb13b6bde3b (diff) | |
download | webrtc-553c869c58ef9c6b09da2e3835ce3adfcac8d540.tar.gz |
Start consolidating management/querying of stats on the Call thread.
Call is instantiated on what we traditionally call the 'worker thread'
in PeerConnection terms. Call statistics are however gathered, processed
and reported in a number of different ways, which results in a lot of
locking, which is also unpredictable due to the those actions themselves
contending with other parts of the system.
Designating the worker thread as the general owner of the stats, helps
us keeps things regular and avoids loading unrelated task queues/threads
with reporting things like histograms or locking up due to a call to
GetStats().
This is a reland of remaining changes from https://webrtc-review.googlesource.com/c/src/+/172847:
This applies the changes from the above CL to the forked files and
switches call.cc over to using the forked implementation.
Bug: webrtc:11489
Change-Id: I93ad560500806ddd0e6df1448b1bcf5a1aae7583
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/174000
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Reviewed-by: Magnus Flodman <mflodman@webrtc.org>
Reviewed-by: Danil Chapovalov <danilchap@webrtc.org>
Commit-Queue: Tommi <tommi@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#31186}
Diffstat (limited to 'test')
-rw-r--r-- | test/call_test.h | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/test/call_test.h b/test/call_test.h index 3f4aa072e7..4b26097b6c 100644 --- a/test/call_test.h +++ b/test/call_test.h @@ -31,6 +31,7 @@ #include "test/fake_vp8_encoder.h" #include "test/frame_generator_capturer.h" #include "test/rtp_rtcp_observer.h" +#include "test/run_loop.h" namespace webrtc { namespace test { @@ -176,6 +177,8 @@ class CallTest : public ::testing::Test { FlexfecReceiveStream::Config* GetFlexFecConfig(); TaskQueueBase* task_queue() { return task_queue_.get(); } + test::RunLoop loop_; + Clock* const clock_; const FieldTrialBasedConfig field_trials_; |