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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: webrtc/video/stream_synchronization.cc

Issue 1756193005: Add histogram stats for AV sync stream offset: (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Created 4 years, 9 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
« webrtc/call/call_perf_tests.cc ('K') | « webrtc/video/receive_statistics_proxy.cc ('k') | webrtc/video/video_receive_stream.cc » ('j') | webrtc/video/video_receive_stream.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/video/stream_synchronization.cc
diff --git a/webrtc/video/stream_synchronization.cc b/webrtc/video/stream_synchronization.cc
index cb37d80ef5ab3106af16c3aff53593663e72d471..3727f8fdb53e5934c6a884e507ac79f52161e8da 100644
--- a/webrtc/video/stream_synchronization.cc
+++ b/webrtc/video/stream_synchronization.cc
@@ -60,10 +60,6 @@ bool StreamSynchronization::ComputeRelativeDelay(
const Measurements& video_measurement,
int* relative_delay_ms) {
assert(relative_delay_ms);
- if (audio_measurement.rtcp.size() < 2 || video_measurement.rtcp.size() < 2) {
- // We need two RTCP SR reports per stream to do synchronization.
- return false;
- }
int64_t audio_last_capture_time_ms;
if (!RtpToNtpMs(audio_measurement.latest_timestamp,
audio_measurement.rtcp,
« webrtc/call/call_perf_tests.cc ('K') | « webrtc/video/receive_statistics_proxy.cc ('k') | webrtc/video/video_receive_stream.cc » ('j') | webrtc/video/video_receive_stream.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698