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

Unified Diff: webrtc/video/rtp_stream_receiver.cc

Issue 2911193002: Implement timing frames. (Closed)
Patch Set: Fix uninitialized variables memcheck errors Created 3 years, 7 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/video/rtp_stream_receiver.cc
diff --git a/webrtc/video/rtp_stream_receiver.cc b/webrtc/video/rtp_stream_receiver.cc
index 78057771d6e5705b65e37eebdbba59ef24fa8c88..e7fd6b4f146adcc83749aabb61a6924d3044e05e 100644
--- a/webrtc/video/rtp_stream_receiver.cc
+++ b/webrtc/video/rtp_stream_receiver.cc
@@ -10,8 +10,8 @@
#include "webrtc/video/rtp_stream_receiver.h"
-#include <vector>
#include <utility>
+#include <vector>
#include "webrtc/base/checks.h"
#include "webrtc/base/location.h"
@@ -238,6 +238,7 @@ int32_t RtpStreamReceiver::OnReceivedPayloadData(
VCMPacket packet(payload_data, payload_size, rtp_header_with_ntp);
packet.timesNacked =
nack_module_ ? nack_module_->OnReceivedPacket(packet) : -1;
+ packet.receive_time_ms = clock_->TimeInMilliseconds();
// In the case of a video stream without picture ids and no rtx the
// RtpFrameReferenceFinder will need to know about padding to
@@ -520,6 +521,11 @@ void RtpStreamReceiver::NotifyReceiverOfFecPacket(const RTPHeader& header) {
if (header.extension.hasVideoContentType) {
rtp_header.type.Video.content_type = header.extension.videoContentType;
}
+ rtp_header.type.Video.video_timing = {0u, 0u, 0u, 0u, 0u, false};
+ if (header.extension.hasVideoTiming) {
+ rtp_header.type.Video.video_timing = header.extension.videoTiming;
+ rtp_header.type.Video.video_timing.is_timing_frame = true;
+ }
rtp_header.type.Video.playout_delay = header.extension.playout_delay;
OnReceivedPayloadData(nullptr, 0, &rtp_header);

Powered by Google App Engine
This is Rietveld 408576698