Bug 1525341 - RTP audio jitter stat is not updating r=dminor a=lizzard
authorNico Grunbaum <na-g@nostrum.com>
Wed, 20 Feb 2019 19:32:42 +0000
changeset 516164 cf5c156b86ceeda2be61619c3cb3aa26503876e5
parent 516163 af029ffaf0ff1fa524d3b0dac01e3e40ecd106b2
child 516165 edb0544758a7243453f9f048cc558b98cfd0e53f
push id1953
push userffxbld-merge
push dateMon, 11 Mar 2019 12:10:20 +0000
treeherdermozilla-release@9c35dcbaa899 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersdminor, lizzard
bugs1525341
milestone66.0
Bug 1525341 - RTP audio jitter stat is not updating r=dminor a=lizzard RTP audio jitter stat is not updating because the playout frequency isn't being set Differential Revision: https://phabricator.services.mozilla.com/D20530
media/webrtc/trunk/webrtc/voice_engine/channel.cc
--- a/media/webrtc/trunk/webrtc/voice_engine/channel.cc
+++ b/media/webrtc/trunk/webrtc/voice_engine/channel.cc
@@ -1272,16 +1272,17 @@ void Channel::OnRtpPacket(const RtpPacke
 
   // Store playout timestamp for the received RTP packet
   UpdatePlayoutTimestamp(false);
 
   header.payload_type_frequency =
       rtp_payload_registry_->GetPayloadTypeFrequency(header.payloadType);
   if (header.payload_type_frequency >= 0) {
     bool in_order = IsPacketInOrder(header);
+    statistics_proxy_->OnSendCodecFrequencyChanged(header.payload_type_frequency);
     rtp_receive_statistics_->IncomingPacket(
         header, packet.size(), IsPacketRetransmitted(header, in_order));
     rtp_payload_registry_->SetIncomingPayloadType(header);
 
     ReceivePacket(packet.data(), packet.size(), header);
   }
 }