admin/webrtc_m130
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Remove redundant mapping.

Browse Source 
It looks like the map `content_specific_stats_` is just carried over into `aggregated_stats` without doing any important processing. This seems to be redundant and hence, removing it in this CL.

Bug: None
Change-Id: Ia4a5de03999ecd33d387014928cba322bb11ee93
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/360745
Commit-Queue: Emil Vardar (xWF) <vardar@google.com>
Reviewed-by: Erik Språng <sprang@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#42870}
This commit is contained in:
Emil Vardar 2024-08-27 13:48:31 +00:00 committed by WebRTC LUCI CQ
parent 54559d354e
commit 572280fe6a
1 changed files with 0 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
video/receive_statistics_proxy.cc
View File

@ -234,18 +234,7 @@ void ReceiveStatisticsProxy::UpdateHistograms(
if (delay_ms)
RTC_HISTOGRAM_COUNTS_10000("WebRTC.Video.OnewayDelayInMs", *delay_ms);
// Aggregate content_specific_stats_ by removing experiment or simulcast
// information;
std::map<VideoContentType, ContentSpecificStats> aggregated_stats;
for (const auto& it : content_specific_stats_) {
// Calculate simulcast specific metrics (".S0" ... ".S2" suffixes).
VideoContentType content_type = it.first;
// Calculate aggregated metrics (no suffixes. Aggregated on everything).
content_type = it.first;
aggregated_stats[content_type].Add(it.second);
}
for (const auto& it : aggregated_stats) {
// For the metric Foo we report the following slices:
// WebRTC.Video.Foo,
// WebRTC.Video.Screenshare.Foo,