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

Unified Diff: webrtc/call/rtc_event_log.h

Issue 2353543003: Added logging for audio send/receive stream configs. (Closed)
Patch Set: Refactored setting of header extensions into separate function. 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/call/rtc_event_log.h
diff --git a/webrtc/call/rtc_event_log.h b/webrtc/call/rtc_event_log.h
index a3359692eb8488dd2796929aaa278476156ee563..30ae8f28b29cf5c597e80d1559afa0a314a220b2 100644
--- a/webrtc/call/rtc_event_log.h
+++ b/webrtc/call/rtc_event_log.h
@@ -14,6 +14,8 @@
#include <memory>
#include <string>
+#include "webrtc/api/call/audio_receive_stream.h"
+#include "webrtc/api/call/audio_send_stream.h"
#include "webrtc/base/platform_file.h"
#include "webrtc/video_receive_stream.h"
#include "webrtc/video_send_stream.h"
@@ -77,6 +79,14 @@ class RtcEventLog {
virtual void LogVideoSendStreamConfig(
const webrtc::VideoSendStream::Config& config) = 0;
+ // Logs configuration information for webrtc::AudioReceiveStream.
+ virtual void LogAudioReceiveStreamConfig(
+ const webrtc::AudioReceiveStream::Config& config) = 0;
+
+ // Logs configuration information for webrtc::AudioSendStream.
+ virtual void LogAudioSendStreamConfig(
+ const webrtc::AudioSendStream::Config& config) = 0;
+
// Logs the header of an incoming or outgoing RTP packet. packet_length
// is the total length of the packet, including both header and payload.
virtual void LogRtpHeader(PacketDirection direction,
@@ -123,6 +133,10 @@ class RtcEventLogNullImpl final : public RtcEventLog {
const VideoReceiveStream::Config& config) override {}
void LogVideoSendStreamConfig(
const VideoSendStream::Config& config) override {}
+ void LogAudioReceiveStreamConfig(
+ const AudioReceiveStream::Config& config) override {}
+ void LogAudioSendStreamConfig(
+ const AudioSendStream::Config& config) override {}
void LogRtpHeader(PacketDirection direction,
MediaType media_type,
const uint8_t* header,

Powered by Google App Engine
This is Rietveld 408576698