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

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

Issue 2742383004: Delete support for receiving RTCP RPSI and SLI messages. (Closed)
Patch Set: Rebased. Created 3 years, 9 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
« no previous file with comments | « no previous file | webrtc/modules/rtp_rtcp/BUILD.gn » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/logging/rtc_event_log/rtc_event_log2text.cc
diff --git a/webrtc/logging/rtc_event_log/rtc_event_log2text.cc b/webrtc/logging/rtc_event_log/rtc_event_log2text.cc
index cddd91c53c79f8856f6bbbb985efd41f21b03f44..b2d537e3b3b0cb2a67158e89038a9126385e9825 100644
--- a/webrtc/logging/rtc_event_log/rtc_event_log2text.cc
+++ b/webrtc/logging/rtc_event_log/rtc_event_log2text.cc
@@ -26,10 +26,8 @@
#include "webrtc/modules/rtp_rtcp/source/rtcp_packet/rapid_resync_request.h"
#include "webrtc/modules/rtp_rtcp/source/rtcp_packet/receiver_report.h"
#include "webrtc/modules/rtp_rtcp/source/rtcp_packet/remb.h"
-#include "webrtc/modules/rtp_rtcp/source/rtcp_packet/rpsi.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/rtcp_packet/sli.h"
#include "webrtc/modules/rtp_rtcp/source/rtcp_packet/tmmbn.h"
#include "webrtc/modules/rtp_rtcp/source/rtcp_packet/tmmbr.h"
#include "webrtc/modules/rtp_rtcp/source/rtcp_packet/transport_feedback.h"
@@ -266,28 +264,6 @@ void PrintPsFeedback(const webrtc::rtcp::CommonHeader& rtcp_block,
<< "\tssrc=" << pli.sender_ssrc() << std::endl;
break;
}
- case webrtc::rtcp::Sli::kFeedbackMessageType: {
- webrtc::rtcp::Sli sli;
- if (!sli.Parse(rtcp_block))
- return;
- if (ExcludePacket(direction, media_type, sli.sender_ssrc()))
- return;
- std::cout << log_timestamp << "\t"
- << "RTCP_SLI" << StreamInfo(direction, media_type)
- << "\tssrc=" << sli.sender_ssrc() << std::endl;
- break;
- }
- case webrtc::rtcp::Rpsi::kFeedbackMessageType: {
- webrtc::rtcp::Rpsi rpsi;
- if (!rpsi.Parse(rtcp_block))
- return;
- if (ExcludePacket(direction, media_type, rpsi.sender_ssrc()))
- return;
- std::cout << log_timestamp << "\t"
- << "RTCP_RPSI" << StreamInfo(direction, media_type)
- << "\tssrc=" << rpsi.sender_ssrc() << std::endl;
- break;
- }
case webrtc::rtcp::Fir::kFeedbackMessageType: {
webrtc::rtcp::Fir fir;
if (!fir.Parse(rtcp_block))
« no previous file with comments | « no previous file | webrtc/modules/rtp_rtcp/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698