From c66d45a809ca0494eae8530fb1b6a8cc42fd0083 Mon Sep 17 00:00:00 2001 From: Artem Titov Date: Fri, 22 Apr 2022 23:24:16 +0200 Subject: [PATCH] [DVQA] Return a copy of the frames counters. Return a copy of the frames counters to be able to get DVQA stats while it is running. Bug: b/213863770 Change-Id: I04755117cf363e1f160caa11dbe52fb30324fdf5 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/259867 Reviewed-by: Andrey Logvin Commit-Queue: Artem Titov Cr-Commit-Position: refs/heads/main@{#36638} --- test/pc/e2e/analyzer/video/default_video_quality_analyzer.cc | 2 +- test/pc/e2e/analyzer/video/default_video_quality_analyzer.h | 2 +- 2 files changed, 2 insertions(+), 2 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 da49d67379..ecc92f28b9 100644 --- a/test/pc/e2e/analyzer/video/default_video_quality_analyzer.cc +++ b/test/pc/e2e/analyzer/video/default_video_quality_analyzer.cc @@ -711,7 +711,7 @@ VideoStreamsInfo DefaultVideoQualityAnalyzer::GetKnownStreams() const { std::move(stream_to_receivers)); } -const FrameCounters& DefaultVideoQualityAnalyzer::GetGlobalCounters() const { +FrameCounters DefaultVideoQualityAnalyzer::GetGlobalCounters() const { MutexLock lock(&mutex_); return frame_counters_; } 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 a2a885c8e7..bbda276767 100644 --- a/test/pc/e2e/analyzer/video/default_video_quality_analyzer.h +++ b/test/pc/e2e/analyzer/video/default_video_quality_analyzer.h @@ -83,7 +83,7 @@ class DefaultVideoQualityAnalyzer : public VideoQualityAnalyzerInterface { // Returns set of stream labels, that were met during test call. std::set GetKnownVideoStreams() const; VideoStreamsInfo GetKnownStreams() const; - const FrameCounters& GetGlobalCounters() const; + FrameCounters GetGlobalCounters() const; // Returns frame counter per stream label. Valid stream labels can be obtained // by calling GetKnownVideoStreams() std::map GetPerStreamCounters() const;