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

Unified Diff: webrtc/video_receive_stream.h

Issue 1181653002: Base A/V synchronization on sync_labels. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: rebase Created 5 years, 5 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
« no previous file with comments | « webrtc/video_engine/vie_sync_module.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/video_receive_stream.h
diff --git a/webrtc/video_receive_stream.h b/webrtc/video_receive_stream.h
index a8003c1aa9c53c49aa1ebd8478617d34c51f5dc9..11033812d592abdf0a0bd45fee71d7ee25cc1bc1 100644
--- a/webrtc/video_receive_stream.h
+++ b/webrtc/video_receive_stream.h
@@ -145,10 +145,10 @@ class VideoReceiveStream {
// Only valid if 'renderer' is set.
int render_delay_ms = 10;
- // Audio channel corresponding to this video stream, used for audio/video
- // synchronization. 'audio_channel_id' is ignored if no VoiceEngine is set
- // when creating the VideoEngine instance. '-1' disables a/v sync.
- int audio_channel_id = -1;
+ // Identifier for an A/V synchronization group. Empty string to disable.
+ // TODO(pbos): Synchronize streams in a sync group, not just video streams
+ // to one of the audio streams.
+ std::string sync_group;
// Called for each incoming video frame, i.e. in encoded state. E.g. used
// when
« no previous file with comments | « webrtc/video_engine/vie_sync_module.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698