Update Packet::GetHeader() to copy playout_delay

SergeyUlanov · SergeyUlanov · commit ab7f856fa72a · 2017-05-22T19:48:25.000Z
Packet::GetHeader() wasn't copying playout_delay. As result playout_delay was ignored when flexfec is enabled. Patch by Rob McCool <rmccool@google.com> BUG=webrtc:7590 Review-Url: https://codereview.webrtc.org/2899553003 Cr-Commit-Position: refs/heads/master@{#18218} (cherry picked from commit b2c001a) BUG=chromium:722490 Review-Url: https://codereview.webrtc.org/2898813002 . Cr-Commit-Position: refs/branch-heads/59@{#13} Cr-Branched-From: 10d095d-refs/heads/master@{#17657}
diff --git a/webrtc/modules/rtp_rtcp/source/rtp_packet.cc b/webrtc/modules/rtp_rtcp/source/rtp_packet.cc
@@ -169,6 +169,7 @@ void Packet::GetHeader(RTPHeader* header) const {
       &header->extension.voiceActivity, &header->extension.audioLevel);
   header->extension.hasVideoRotation =
       GetExtension<VideoOrientation>(&header->extension.videoRotation);
+  GetExtension<PlayoutDelayLimits>(&header->extension.playout_delay);
 }
 
 size_t Packet::headers_size() const {

Original file line number	Diff line number	Diff line change
`@@ -169,6 +169,7 @@ void Packet::GetHeader(RTPHeader* header) const {`
`169`	`169`	`&header->extension.voiceActivity, &header->extension.audioLevel);`
`170`	`170`	`header->extension.hasVideoRotation =`
`171`	`171`	`GetExtension<VideoOrientation>(&header->extension.videoRotation);`
	`172`	`+ GetExtension<PlayoutDelayLimits>(&header->extension.playout_delay);`
`172`	`173`	`}`
`173`	`174`
`174`	`175`	`size_t Packet::headers_size() const {`