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

Unified Diff: webrtc/modules/rtp_rtcp/source/rtp_packet.cc

Issue 3012983002: Remove RtpPacketToSend::GetHeader as almost unused. (Closed)
Patch Set: -forward declare of RTPHeader 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
« no previous file with comments | « webrtc/modules/rtp_rtcp/source/rtp_packet.h ('k') | webrtc/modules/rtp_rtcp/source/rtp_packet_received.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/modules/rtp_rtcp/source/rtp_packet.cc
diff --git a/webrtc/modules/rtp_rtcp/source/rtp_packet.cc b/webrtc/modules/rtp_rtcp/source/rtp_packet.cc
index 2c9d0aea5aa53772acfc32f7c4c8769c59fee554..7df60053fb8e206e6d9ed0a6af12b1ecffacc66d 100644
--- a/webrtc/modules/rtp_rtcp/source/rtp_packet.cc
+++ b/webrtc/modules/rtp_rtcp/source/rtp_packet.cc
@@ -16,7 +16,6 @@
#include "webrtc/common_types.h"
#include "webrtc/modules/rtp_rtcp/include/rtp_header_extension_map.h"
#include "webrtc/modules/rtp_rtcp/source/byte_io.h"
-#include "webrtc/modules/rtp_rtcp/source/rtp_header_extensions.h"
#include "webrtc/rtc_base/checks.h"
#include "webrtc/rtc_base/logging.h"
#include "webrtc/rtc_base/random.h"
@@ -145,43 +144,6 @@ std::vector<uint32_t> Packet::Csrcs() const {
return csrcs;
}
-void Packet::GetHeader(RTPHeader* header) const {
- header->markerBit = Marker();
- header->payloadType = PayloadType();
- header->sequenceNumber = SequenceNumber();
- header->timestamp = Timestamp();
- header->ssrc = Ssrc();
- std::vector<uint32_t> csrcs = Csrcs();
- header->numCSRCs = csrcs.size();
- for (size_t i = 0; i < csrcs.size(); ++i) {
- header->arrOfCSRCs[i] = csrcs[i];
- }
- header->paddingLength = padding_size();
- header->headerLength = headers_size();
- header->payload_type_frequency = 0;
- header->extension.hasTransmissionTimeOffset =
- GetExtension<TransmissionOffset>(
- &header->extension.transmissionTimeOffset);
- header->extension.hasAbsoluteSendTime =
- GetExtension<AbsoluteSendTime>(&header->extension.absoluteSendTime);
- header->extension.hasTransportSequenceNumber =
- GetExtension<TransportSequenceNumber>(
- &header->extension.transportSequenceNumber);
- header->extension.hasAudioLevel = GetExtension<AudioLevel>(
- &header->extension.voiceActivity, &header->extension.audioLevel);
- header->extension.hasVideoRotation =
- GetExtension<VideoOrientation>(&header->extension.videoRotation);
- header->extension.hasVideoContentType =
- GetExtension<VideoContentTypeExtension>(
- &header->extension.videoContentType);
- header->extension.has_video_timing =
- GetExtension<VideoTimingExtension>(&header->extension.video_timing);
- GetExtension<RtpStreamId>(&header->extension.stream_id);
- GetExtension<RepairedRtpStreamId>(&header->extension.repaired_stream_id);
- GetExtension<RtpMid>(&header->extension.mid);
- GetExtension<PlayoutDelayLimits>(&header->extension.playout_delay);
-}
-
size_t Packet::headers_size() const {
return payload_offset_;
}
« no previous file with comments | « webrtc/modules/rtp_rtcp/source/rtp_packet.h ('k') | webrtc/modules/rtp_rtcp/source/rtp_packet_received.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698