Return a copy, becase GetPercentile in SamplesStatsCounter isn't const

Bug: webrtc:10138
Change-Id: I2ec2ce4765e514bfd065f094f5905233e5f4f9cd
Reviewed-on: https://webrtc-review.googlesource.com/c/123043
Reviewed-by: Peter Slatala <psla@webrtc.org>
Commit-Queue: Artem Titov <titovartem@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#26697}
This commit is contained in:
Artem Titov 2019-02-14 11:06:34 +01:00 committed by Commit Bot
parent cf8405e172
commit ceba6ae2a7
2 changed files with 5 additions and 5 deletions

View file

@ -316,13 +316,13 @@ DefaultVideoQualityAnalyzer::GetPerStreamCounters() const {
return stream_frame_counters_; return stream_frame_counters_;
} }
const std::map<std::string, StreamStats>& std::map<std::string, StreamStats> DefaultVideoQualityAnalyzer::GetStats()
DefaultVideoQualityAnalyzer::GetStats() const { const {
rtc::CritScope cri(&comparison_lock_); rtc::CritScope cri(&comparison_lock_);
return stream_stats_; return stream_stats_;
} }
const AnalyzerStats& DefaultVideoQualityAnalyzer::GetAnalyzerStats() const { AnalyzerStats DefaultVideoQualityAnalyzer::GetAnalyzerStats() const {
rtc::CritScope crit(&comparison_lock_); rtc::CritScope crit(&comparison_lock_);
return analyzer_stats_; return analyzer_stats_;
} }

View file

@ -145,8 +145,8 @@ class DefaultVideoQualityAnalyzer : public VideoQualityAnalyzerInterface {
const std::map<std::string, FrameCounters>& GetPerStreamCounters() const; const std::map<std::string, FrameCounters>& GetPerStreamCounters() const;
// Returns video quality stats per stream label. Valid stream labels can be // Returns video quality stats per stream label. Valid stream labels can be
// obtained by calling GetKnownVideoStreams() // obtained by calling GetKnownVideoStreams()
const std::map<std::string, StreamStats>& GetStats() const; std::map<std::string, StreamStats> GetStats() const;
const AnalyzerStats& GetAnalyzerStats() const; AnalyzerStats GetAnalyzerStats() const;
private: private:
struct FrameStats { struct FrameStats {