Index: webrtc/voice_engine/channel.cc |
diff --git a/webrtc/voice_engine/channel.cc b/webrtc/voice_engine/channel.cc |
index 1957d211943c2cfc675f8e9c4c100679104828b0..2b71b7110980ed841ffa284c48f12dbc1055fa80 100644 |
--- a/webrtc/voice_engine/channel.cc |
+++ b/webrtc/voice_engine/channel.cc |
@@ -3173,11 +3173,7 @@ void Channel::GetDecodingCallStatistics(AudioDecodingCallStats* stats) const { |
bool Channel::GetDelayEstimate(int* jitter_buffer_delay_ms, |
int* playout_buffer_delay_ms) const { |
rtc::CritScope lock(&video_sync_lock_); |
- if (_average_jitter_buffer_delay_us == 0) { |
- return false; |
- } |
- *jitter_buffer_delay_ms = |
- (_average_jitter_buffer_delay_us + 500) / 1000 + _recPacketDelayMs; |
+ *jitter_buffer_delay_ms = audio_coding_->FilteredCurrentDelayMs(); |
*playout_buffer_delay_ms = playout_delay_ms_; |
return true; |
} |
@@ -3390,6 +3386,9 @@ void Channel::UpdatePlayoutTimestamp(bool rtcp) { |
} |
// Called for incoming RTP packets after successful RTP header parsing. |
+// TODO(henrik.lundin): Clean out this method. With the introduction of |
+// AudioCoding::FilteredCurrentDelayMs() most (if not all) of this method can |
+// be deleted, along with a few member variables. (WebRTC issue 6237.) |
the sun
2016/08/23 06:45:07
Oh, that would be so nice!
|
void Channel::UpdatePacketDelay(uint32_t rtp_timestamp, |
uint16_t sequence_number) { |
WEBRTC_TRACE(kTraceStream, kTraceVoice, VoEId(_instanceId, _channelId), |