commit c07e904a25aa95be79662754d97f7451da8bc2a4
author Eldar Rello <elrello@microsoft.com> Fri Jul 03 08:08:07 2020
committer Commit Bot <commit-bot@chromium.org> Fri Jul 03 09:27:52 2020
tree 76071debe1a42ea623029407cb3d4f76249aa94a
parent 1d50cb61d87bdc5d5d29c009da1abcc06e68b4c2

Fix missing local and remote ids in RtpStreamStats

Bug: chromium:1098266
Change-Id: I536464541c5971ea173bd7ed83d523fa50b43d0b
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/178486
Reviewed-by: Henrik Boström <hbos@webrtc.org>
Commit-Queue: Henrik Boström <hbos@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#31620}

pc/rtc_stats_collector.cc

diff --git a/pc/rtc_stats_collector.cc b/pc/rtc_stats_collector.cc
index f66be30..07e40cb 100644
--- a/pc/rtc_stats_collector.cc
+++ b/pc/rtc_stats_collector.cc
@@ -459,7 +459,7 @@
 ProduceRemoteInboundRtpStreamStatsFromReportBlockData(
     const ReportBlockData& report_block_data,
     cricket::MediaType media_type,
-    std::map<std::string, RTCOutboundRTPStreamStats*> outbound_rtps,
+    const std::map<std::string, RTCOutboundRTPStreamStats*>& outbound_rtps,
     const RTCStatsReport& report) {
   const auto& report_block = report_block_data.report_block();
   // RTCStats' timestamp generally refers to when the metric was sampled, but
@@ -1615,8 +1615,8 @@
        track_media_info_map.voice_media_info()->senders) {
     for (const auto& report_block_data : voice_sender_info.report_block_datas) {
       report->AddStats(ProduceRemoteInboundRtpStreamStatsFromReportBlockData(
-          report_block_data, cricket::MEDIA_TYPE_AUDIO,
-          std::move(audio_outbound_rtps), *report));
+          report_block_data, cricket::MEDIA_TYPE_AUDIO, audio_outbound_rtps,
+          *report));
     }
   }
 }
@@ -1692,8 +1692,8 @@
        track_media_info_map.video_media_info()->senders) {
     for (const auto& report_block_data : video_sender_info.report_block_datas) {
       report->AddStats(ProduceRemoteInboundRtpStreamStatsFromReportBlockData(
-          report_block_data, cricket::MEDIA_TYPE_VIDEO,
-          std::move(video_outbound_rtps), *report));
+          report_block_data, cricket::MEDIA_TYPE_VIDEO, video_outbound_rtps,
+          *report));
     }
   }
 }