Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(102)

Unified Diff: webrtc/tools/event_log_visualizer/analyzer.cc

Issue 2353543003: Added logging for audio send/receive stream configs. (Closed)
Patch Set: Added code for parsing the audio send/receive configs. Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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 eab4dcf798623e75c2a1e80d27c865915a9108fb..55700d68a308306966f3b0ed9aca1ce5c6af8e4a 100644
--- a/webrtc/tools/event_log_visualizer/analyzer.cc
+++ b/webrtc/tools/event_log_visualizer/analyzer.cc
@@ -331,12 +331,20 @@ EventLogAnalyzer::EventLogAnalyzer(const ParsedRtcEventLog& log)
}
case ParsedRtcEventLog::AUDIO_RECEIVER_CONFIG_EVENT: {
AudioReceiveStream::Config config;
- // TODO(terelius): Parse the audio configs once we have them.
+ parsed_log_.GetAudioReceiveConfig(i, &config);
+ StreamId stream(config.rtp.remote_ssrc, kIncomingPacket);
+ RegisterHeaderExtensions(config.rtp.extensions,
+ &extension_maps[stream]);
+ audio_ssrcs_.insert(stream);
break;
}
case ParsedRtcEventLog::AUDIO_SENDER_CONFIG_EVENT: {
AudioSendStream::Config config(nullptr);
- // TODO(terelius): Parse the audio configs once we have them.
+ parsed_log_.GetAudioSendConfig(i, &config);
+ StreamId stream(config.rtp.ssrc, kOutgoingPacket);
+ RegisterHeaderExtensions(config.rtp.extensions,
+ &extension_maps[stream]);
+ audio_ssrcs_.insert(stream);
break;
}
case ParsedRtcEventLog::RTP_EVENT: {

Powered by Google App Engine
This is Rietveld 408576698