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

Unified Diff: webrtc/video/rtp_streams_synchronizer.h

Issue 2452163004: Stop using VoEVideoSync in Call/VideoReceiveStream. (Closed)
Patch Set: rebase Created 3 years, 11 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.h
diff --git a/webrtc/video/rtp_streams_synchronizer.h b/webrtc/video/rtp_streams_synchronizer.h
index bc24d6f8071762c390b3a2e42cbbcaba1ef5afc2..521b1a80e0b50921b9a135e45e8d4e39bdc38a64 100644
--- a/webrtc/video/rtp_streams_synchronizer.h
+++ b/webrtc/video/rtp_streams_synchronizer.h
@@ -24,9 +24,8 @@
namespace webrtc {
-class Clock;
+class Syncable;
class VideoFrame;
-class VoEVideoSync;
namespace vcm {
class VideoReceiver;
@@ -34,11 +33,9 @@ class VideoReceiver;
class RtpStreamsSynchronizer : public Module {
public:
- RtpStreamsSynchronizer(vcm::VideoReceiver* vcm,
- RtpStreamReceiver* rtp_stream_receiver);
+ explicit RtpStreamsSynchronizer(Syncable* syncable_video);
- void ConfigureSync(int voe_channel_id,
- VoEVideoSync* voe_sync_interface);
+ void ConfigureSync(Syncable* syncable_audio);
// Implements Module.
int64_t TimeUntilNextProcess() override;
@@ -53,16 +50,10 @@ class RtpStreamsSynchronizer : public Module {
double* estimated_freq_khz) const;
private:
- Clock* const clock_;
- vcm::VideoReceiver* const video_receiver_;
- RtpReceiver* const video_rtp_receiver_;
- RtpRtcp* const video_rtp_rtcp_;
+ Syncable* syncable_video_;
rtc::CriticalSection crit_;
- int voe_channel_id_ GUARDED_BY(crit_);
- VoEVideoSync* voe_sync_interface_ GUARDED_BY(crit_);
- RtpReceiver* audio_rtp_receiver_ GUARDED_BY(crit_);
- RtpRtcp* audio_rtp_rtcp_ GUARDED_BY(crit_);
+ Syncable* syncable_audio_ GUARDED_BY(crit_);
std::unique_ptr<StreamSynchronization> sync_ GUARDED_BY(crit_);
StreamSynchronization::Measurements audio_measurement_ GUARDED_BY(crit_);
StreamSynchronization::Measurements video_measurement_ GUARDED_BY(crit_);

Powered by Google App Engine
This is Rietveld 408576698