Track SSRCs configured for each media type in event log visualization tool.

webrtc/tools/event_log_visualizer/analyzer.cc

Index: webrtc/tools/event_log_visualizer/analyzer.cc
diff --git a/webrtc/tools/event_log_visualizer/analyzer.cc b/webrtc/tools/event_log_visualizer/analyzer.cc
index c97a30fc0ebc8dc8ddee49372d3491f62ccdcb02..b61fdb445782865bf24568776ba11638b40e7cc6 100644
--- a/webrtc/tools/event_log_visualizer/analyzer.cc
+++ b/webrtc/tools/event_log_visualizer/analyzer.cc
@@ -100,7 +100,7 @@ constexpr float kTopMargin = 0.05f;
 
 }  // namespace
 
-bool EventLogAnalyzer::StreamId::operator<(const StreamId& other) const {
+bool StreamId::operator<(const StreamId& other) const {
   if (ssrc_ < other.ssrc_) {
     return true;
   }
@@ -112,7 +112,7 @@ bool EventLogAnalyzer::StreamId::operator<(const StreamId& other) const {
   return false;
 }
 
-bool EventLogAnalyzer::StreamId::operator==(const StreamId& other) const {
+bool StreamId::operator==(const StreamId& other) const {
   return ssrc_ == other.ssrc_ && direction_ == other.direction_;
 }
 
@@ -151,10 +151,13 @@ EventLogAnalyzer::EventLogAnalyzer(const ParsedRtcEventLog& log)
         StreamId stream(config.rtp.remote_ssrc, kIncomingPacket);
         RegisterHeaderExtensions(config.rtp.extensions,
                                  &extension_maps[stream]);
+        video_ssrcs_.insert(stream);
         for (auto kv : config.rtp.rtx) {
           StreamId rtx_stream(kv.second.ssrc, kIncomingPacket);
           RegisterHeaderExtensions(config.rtp.extensions,
                                    &extension_maps[rtx_stream]);
+          video_ssrcs_.insert(rtx_stream);
+          rtx_ssrcs_.insert(rtx_stream);
         }
         break;
       }
@@ -165,11 +168,14 @@ EventLogAnalyzer::EventLogAnalyzer(const ParsedRtcEventLog& log)
           StreamId stream(ssrc, kOutgoingPacket);
           RegisterHeaderExtensions(config.rtp.extensions,
                                    &extension_maps[stream]);
+          video_ssrcs_.insert(stream);
         }
         for (auto ssrc : config.rtp.rtx.ssrcs) {
-          StreamId stream(ssrc, kOutgoingPacket);
+          StreamId rtx_stream(ssrc, kOutgoingPacket);
           RegisterHeaderExtensions(config.rtp.extensions,
-                                   &extension_maps[stream]);
+                                   &extension_maps[rtx_stream]);
+          video_ssrcs_.insert(rtx_stream);
+          rtx_ssrcs_.insert(rtx_stream);
         }
         break;
       }
@@ -303,6 +309,18 @@ class BitrateObserver : public CongestionController::Observer,
   bool bitrate_updated_;
 };
 
+bool EventLogAnalyzer::IsRtxSsrc(StreamId stream_id) {
+  return (rtx_ssrcs_.count(stream_id) == 1);

[stefan-webrtc, 2016/08/04 14:22:55]

I think you can remove ().

I'd also prefer using find() instead of count().

[terelius, 2016/08/05 12:24:57]

Removed the parenthesis. I prefer count() since that value is less powerful than
an iterator into the map.

+}
+
+bool EventLogAnalyzer::IsVideoSsrc(StreamId stream_id) {
+  return (video_ssrcs_.count(stream_id) == 1);
+}
+
+bool EventLogAnalyzer::IsAudioSsrc(StreamId stream_id) {
+  return (audio_ssrcs_.count(stream_id) == 1);
+}
+
 void EventLogAnalyzer::CreatePacketGraph(PacketDirection desired_direction,
                                          Plot* plot) {
   std::map<uint32_t, TimeSeries> time_series;