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

Unified Diff: webrtc/logging/rtc_event_log/rtc_event_log.cc

Issue 3012273002: Ignore this CL - here as a baseline only (originally Bjorn's CL)
Patch Set: Created 3 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/logging/rtc_event_log/rtc_event_log.cc
diff --git a/webrtc/logging/rtc_event_log/rtc_event_log.cc b/webrtc/logging/rtc_event_log/rtc_event_log.cc
index 6cb9dd96184d18d68b348012537e32b795057af3..3c2b00a5c3ae33bb8158946f9f66560c9f805073 100644
--- a/webrtc/logging/rtc_event_log/rtc_event_log.cc
+++ b/webrtc/logging/rtc_event_log/rtc_event_log.cc
@@ -33,6 +33,8 @@
#include "webrtc/modules/rtp_rtcp/source/rtcp_packet/rtpfb.h"
#include "webrtc/modules/rtp_rtcp/source/rtcp_packet/sdes.h"
#include "webrtc/modules/rtp_rtcp/source/rtcp_packet/sender_report.h"
+#include "webrtc/modules/rtp_rtcp/source/rtp_packet_received.h"
+#include "webrtc/modules/rtp_rtcp/source/rtp_packet_to_send.h"
#include "webrtc/rtc_base/checks.h"
#include "webrtc/rtc_base/constructormagic.h"
#include "webrtc/rtc_base/event.h"
@@ -113,16 +115,27 @@ class RtcEventLogImpl final : public RtcEventLog {
void LogVideoSendStreamConfig(const rtclog::StreamConfig& config) override;
void LogAudioReceiveStreamConfig(const rtclog::StreamConfig& config) override;
void LogAudioSendStreamConfig(const rtclog::StreamConfig& config) override;
+ // TODO(terelius): This can be removed as soon as the interface has been
+ // updated.
void LogRtpHeader(PacketDirection direction,
const uint8_t* header,
size_t packet_length) override;
+ // TODO(terelius): This can be made private, non-virtual as soon as the
+ // interface has been updated.
void LogRtpHeader(PacketDirection direction,
const uint8_t* header,
size_t packet_length,
int probe_cluster_id) override;
+ void LogIncomingRtpHeader(const RtpPacketReceived& packet) override;
+ void LogOutgoingRtpHeader(const RtpPacketToSend& packet,
+ int probe_cluster_id) override;
+ // TODO(terelius): This can be made private, non-virtual as soon as the
+ // interface has been updated.
void LogRtcpPacket(PacketDirection direction,
const uint8_t* packet,
size_t length) override;
+ void LogIncomingRtcpPacket(rtc::ArrayView<const uint8_t> packet) override;
+ void LogOutgoingRtcpPacket(rtc::ArrayView<const uint8_t> packet) override;
void LogAudioPlayout(uint32_t ssrc) override;
void LogLossBasedBweUpdate(int32_t bitrate_bps,
uint8_t fraction_loss,
@@ -426,6 +439,16 @@ void RtcEventLogImpl::LogAudioSendStreamConfig(
StoreEvent(std::move(event));
}
+void RtcEventLogImpl::LogIncomingRtpHeader(const RtpPacketReceived& packet) {
+ LogRtpHeader(kIncomingPacket, packet.data(), packet.size(),
+ PacedPacketInfo::kNotAProbe);
+}
+
+void RtcEventLogImpl::LogOutgoingRtpHeader(const RtpPacketToSend& packet,
+ int probe_cluster_id) {
+ LogRtpHeader(kOutgoingPacket, packet.data(), packet.size(), probe_cluster_id);
+}
+
void RtcEventLogImpl::LogRtpHeader(PacketDirection direction,
const uint8_t* header,
size_t packet_length) {
@@ -463,6 +486,16 @@ void RtcEventLogImpl::LogRtpHeader(PacketDirection direction,
StoreEvent(std::move(rtp_event));
}
+void RtcEventLogImpl::LogIncomingRtcpPacket(
+ rtc::ArrayView<const uint8_t> packet) {
+ LogRtcpPacket(kIncomingPacket, packet.data(), packet.size());
+}
+
+void RtcEventLogImpl::LogOutgoingRtcpPacket(
+ rtc::ArrayView<const uint8_t> packet) {
+ LogRtcpPacket(kOutgoingPacket, packet.data(), packet.size());
+}
+
void RtcEventLogImpl::LogRtcpPacket(PacketDirection direction,
const uint8_t* packet,
size_t length) {
« no previous file with comments | « webrtc/logging/rtc_event_log/rtc_event_log.h ('k') | webrtc/logging/rtc_event_log/rtc_event_log_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698