From ee23383c5e6936880af10d239402a4203ebff15e Mon Sep 17 00:00:00 2001 From: Artem Titov Date: Tue, 8 Sep 2020 12:23:05 +0200 Subject: [PATCH] Use time controller based task queue factory in PC level tests. It is required to properly support real and simulated time. Bug: webrtc:11743 Change-Id: If6dd59691d966378f8ff897c82dee05c1899e9e0 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/183602 Reviewed-by: Andrey Logvin Commit-Queue: Artem Titov Cr-Commit-Position: refs/heads/master@{#32057} --- test/pc/e2e/BUILD.gn | 1 - test/pc/e2e/peer_connection_quality_test.cc | 3 +-- 2 files changed, 1 insertion(+), 3 deletions(-) diff --git a/test/pc/e2e/BUILD.gn b/test/pc/e2e/BUILD.gn index 62ed0b8a91..fbcaa35db8 100644 --- a/test/pc/e2e/BUILD.gn +++ b/test/pc/e2e/BUILD.gn @@ -395,7 +395,6 @@ if (!build_with_chromium) { "../../../api:video_quality_analyzer_api", "../../../api/rtc_event_log", "../../../api/task_queue", - "../../../api/task_queue:default_task_queue_factory", "../../../api/units:time_delta", "../../../api/units:timestamp", "../../../pc:pc_test_utils", diff --git a/test/pc/e2e/peer_connection_quality_test.cc b/test/pc/e2e/peer_connection_quality_test.cc index 4516780a2c..26039f8b5e 100644 --- a/test/pc/e2e/peer_connection_quality_test.cc +++ b/test/pc/e2e/peer_connection_quality_test.cc @@ -21,7 +21,6 @@ #include "api/rtc_event_log/rtc_event_log.h" #include "api/rtc_event_log_output_file.h" #include "api/scoped_refptr.h" -#include "api/task_queue/default_task_queue_factory.h" #include "api/test/time_controller.h" #include "api/test/video_quality_analyzer_interface.h" #include "pc/sdp_utils.h" @@ -109,7 +108,7 @@ PeerConnectionE2EQualityTest::PeerConnectionE2EQualityTest( std::unique_ptr audio_quality_analyzer, std::unique_ptr video_quality_analyzer) : time_controller_(time_controller), - task_queue_factory_(CreateDefaultTaskQueueFactory()), + task_queue_factory_(time_controller_.CreateTaskQueueFactory()), test_case_name_(std::move(test_case_name)), executor_(std::make_unique( time_controller_.GetClock())) {