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

Side by Side 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 unified diff | Download patch
« no previous file with comments | « webrtc/video_engine/vie_sync_module.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright (c) 2013 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2013 The WebRTC project authors. All Rights Reserved.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license 4 * Use of this source code is governed by a BSD-style license
5 * that can be found in the LICENSE file in the root of the source 5 * that can be found in the LICENSE file in the root of the source
6 * tree. An additional intellectual property rights grant can be found 6 * tree. An additional intellectual property rights grant can be found
7 * in the file PATENTS. All contributing project authors may 7 * in the file PATENTS. All contributing project authors may
8 * be found in the AUTHORS file in the root of the source tree. 8 * be found in the AUTHORS file in the root of the source tree.
9 */ 9 */
10 10
(...skipping 127 matching lines...) Expand 10 before | Expand all | Expand 10 after
138 138
139 // VideoRenderer will be called for each decoded frame. 'nullptr' disables 139 // VideoRenderer will be called for each decoded frame. 'nullptr' disables
140 // rendering of this stream. 140 // rendering of this stream.
141 VideoRenderer* renderer = nullptr; 141 VideoRenderer* renderer = nullptr;
142 142
143 // Expected delay needed by the renderer, i.e. the frame will be delivered 143 // Expected delay needed by the renderer, i.e. the frame will be delivered
144 // this many milliseconds, if possible, earlier than the ideal render time. 144 // this many milliseconds, if possible, earlier than the ideal render time.
145 // Only valid if 'renderer' is set. 145 // Only valid if 'renderer' is set.
146 int render_delay_ms = 10; 146 int render_delay_ms = 10;
147 147
148 // Audio channel corresponding to this video stream, used for audio/video 148 // Identifier for an A/V synchronization group. Empty string to disable.
149 // synchronization. 'audio_channel_id' is ignored if no VoiceEngine is set 149 // TODO(pbos): Synchronize streams in a sync group, not just video streams
150 // when creating the VideoEngine instance. '-1' disables a/v sync. 150 // to one of the audio streams.
151 int audio_channel_id = -1; 151 std::string sync_group;
152 152
153 // Called for each incoming video frame, i.e. in encoded state. E.g. used 153 // Called for each incoming video frame, i.e. in encoded state. E.g. used
154 // when 154 // when
155 // saving the stream to a file. 'nullptr' disables the callback. 155 // saving the stream to a file. 'nullptr' disables the callback.
156 EncodedFrameObserver* pre_decode_callback = nullptr; 156 EncodedFrameObserver* pre_decode_callback = nullptr;
157 157
158 // Called for each decoded frame. E.g. used when adding effects to the 158 // Called for each decoded frame. E.g. used when adding effects to the
159 // decoded 159 // decoded
160 // stream. 'nullptr' disables the callback. 160 // stream. 'nullptr' disables the callback.
161 I420FrameCallback* pre_render_callback = nullptr; 161 I420FrameCallback* pre_render_callback = nullptr;
162 162
163 // Target delay in milliseconds. A positive value indicates this stream is 163 // Target delay in milliseconds. A positive value indicates this stream is
164 // used for streaming instead of a real-time call. 164 // used for streaming instead of a real-time call.
165 int target_delay_ms = 0; 165 int target_delay_ms = 0;
166 }; 166 };
167 167
168 virtual void Start() = 0; 168 virtual void Start() = 0;
169 virtual void Stop() = 0; 169 virtual void Stop() = 0;
170 170
171 // TODO(pbos): Add info on currently-received codec to Stats. 171 // TODO(pbos): Add info on currently-received codec to Stats.
172 virtual Stats GetStats() const = 0; 172 virtual Stats GetStats() const = 0;
173 173
174 protected: 174 protected:
175 virtual ~VideoReceiveStream() {} 175 virtual ~VideoReceiveStream() {}
176 }; 176 };
177 177
178 } // namespace webrtc 178 } // namespace webrtc
179 179
180 #endif // WEBRTC_VIDEO_RECEIVE_STREAM_H_ 180 #endif // WEBRTC_VIDEO_RECEIVE_STREAM_H_
OLDNEW
« 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