commit a41959e550925f09e6b2cc29c044f088d81e6932
author Steve Anton <steveanton@webrtc.org> Wed Nov 28 11:15:33 2018 -0800
committer Commit Bot <commit-bot@chromium.org> Wed Nov 28 20:22:10 2018 +0000
tree 3683f9fafae22d39483016d0288d9b7989ae5bfc
parent c64078fdc01fa05cc693753ef53b930354729296

[Unified Plan] Fix old GetStats() not associating track id

The method for looking up track ID by SSRC was never updated for
Unified Plan so it only looked at the first audio section and the
first video section.

This CL changes the method to look through all audio and video
media sections rather than just the first.

Bug: chromium:906988
Change-Id: Ie79e6162b2bd24b8ac9e983b5fa7360c96f030da
Reviewed-on: https://webrtc-review.googlesource.com/c/112223
Commit-Queue: Steve Anton <steveanton@webrtc.org>
Reviewed-by: Henrik Boström <hbos@webrtc.org>
Reviewed-by: Seth Hampson <shampson@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#25833}

pc/statscollector.h

diff --git a/pc/statscollector.h b/pc/statscollector.h
index eb810e5..786d5df 100644
--- a/pc/statscollector.h
+++ b/pc/statscollector.h
@@ -132,9 +132,8 @@
 
   // Helper method to get the id for the track identified by ssrc.
   // |direction| tells if the track is for sending or receiving.
-  bool GetTrackIdBySsrc(uint32_t ssrc,
-                        std::string* track_id,
-                        StatsReport::Direction direction);
+  absl::string_view GetTrackIdBySsrc(uint32_t ssrc,
+                                     StatsReport::Direction direction);
 
   // Helper method to update the timestamp of track records.
   void UpdateTrackReports();