From 60cbf70944cf845781c14ee176cec26c61aa502a Mon Sep 17 00:00:00 2001 From: Artem Titov Date: Fri, 24 Jul 2020 11:15:08 +0200 Subject: [PATCH] Remove deprecated ctor from DefaultVideoQualityAnalyzer Bug: webrtc:11743 Change-Id: Ic4817227499ac7455e0088d90306844b11d67836 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/180124 Reviewed-by: Andrey Logvin Commit-Queue: Artem Titov Cr-Commit-Position: refs/heads/master@{#31788} --- .../e2e/analyzer/video/default_video_quality_analyzer.cc | 9 --------- .../e2e/analyzer/video/default_video_quality_analyzer.h | 5 ----- 2 files changed, 14 deletions(-) diff --git a/test/pc/e2e/analyzer/video/default_video_quality_analyzer.cc b/test/pc/e2e/analyzer/video/default_video_quality_analyzer.cc index 000d1654a6..e9d2dabe1a 100644 --- a/test/pc/e2e/analyzer/video/default_video_quality_analyzer.cc +++ b/test/pc/e2e/analyzer/video/default_video_quality_analyzer.cc @@ -124,15 +124,6 @@ DefaultVideoQualityAnalyzer::DefaultVideoQualityAnalyzer( webrtc::Clock* clock, DefaultVideoQualityAnalyzerOptions options) : options_(options), clock_(clock) {} -DefaultVideoQualityAnalyzer::DefaultVideoQualityAnalyzer( - bool heavy_metrics_computation_enabled, - size_t max_frames_in_flight_per_stream_count) - : clock_(Clock::GetRealTimeClock()) { - options_.heavy_metrics_computation_enabled = - heavy_metrics_computation_enabled; - options_.max_frames_in_flight_per_stream_count = - max_frames_in_flight_per_stream_count; -} DefaultVideoQualityAnalyzer::~DefaultVideoQualityAnalyzer() { Stop(); } diff --git a/test/pc/e2e/analyzer/video/default_video_quality_analyzer.h b/test/pc/e2e/analyzer/video/default_video_quality_analyzer.h index 51ded74f6b..ed40d095b1 100644 --- a/test/pc/e2e/analyzer/video/default_video_quality_analyzer.h +++ b/test/pc/e2e/analyzer/video/default_video_quality_analyzer.h @@ -183,11 +183,6 @@ class DefaultVideoQualityAnalyzer : public VideoQualityAnalyzerInterface { webrtc::Clock* clock, DefaultVideoQualityAnalyzerOptions options = DefaultVideoQualityAnalyzerOptions()); - // Keep for backward compatibility during migration. Will be removed soon. - explicit DefaultVideoQualityAnalyzer( - bool heavy_metrics_computation_enabled = true, - size_t max_frames_in_flight_per_stream_count = - kDefaultMaxFramesInFlightPerStream); ~DefaultVideoQualityAnalyzer() override; void Start(std::string test_case_name,