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

Unified Diff: webrtc/video/rtp_streams_synchronizer.cc

Issue 2385763002: Add stats for frequency offset when converting RTP timestamp to NTP time. (Closed)
Patch Set: Created 4 years, 2 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_streams_synchronizer.cc
diff --git a/webrtc/video/rtp_streams_synchronizer.cc b/webrtc/video/rtp_streams_synchronizer.cc
index 885dad39797fcebeb585e4d3fe4c564d8b3a33e6..b23487d4704a3911e79e5d30d4737204bbaa477c 100644
--- a/webrtc/video/rtp_streams_synchronizer.cc
+++ b/webrtc/video/rtp_streams_synchronizer.cc
@@ -168,7 +168,9 @@ void RtpStreamsSynchronizer::Process() {
}
bool RtpStreamsSynchronizer::GetStreamSyncOffsetInMs(
- const VideoFrame& frame, int64_t* stream_offset_ms) const {
+ const VideoFrame& frame,
+ int64_t* stream_offset_ms,
+ double* estimated_freq_khz) const {
rtc::CritScope lock(&crit_);
if (voe_channel_id_ == -1)
return false;
@@ -186,8 +188,8 @@ bool RtpStreamsSynchronizer::GetStreamSyncOffsetInMs(
}
int64_t latest_video_ntp;
- if (!RtpToNtpMs(frame.timestamp(), video_measurement_.rtcp,
- &latest_video_ntp)) {
+ if (!RtpToNtpMsAndReturnFreq(frame.timestamp(), video_measurement_.rtcp,
+ &latest_video_ntp, estimated_freq_khz)) {
return false;
}

Powered by Google App Engine
This is Rietveld 408576698