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

Side by Side Diff: webrtc/modules/video_render/video_render_impl.cc

Issue 1813173002: Deletes the class VideoRendererCallback. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Rebase, and tweak of DEPS files. 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 unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (c) 2012 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2012 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 139 matching lines...) Expand 10 before | Expand all | Expand 10 after
150 kTraceVideoRenderer, 150 kTraceVideoRenderer,
151 _id, 151 _id,
152 "%s: stream doesn't exist", 152 "%s: stream doesn't exist",
153 __FUNCTION__); 153 __FUNCTION__);
154 return 0; 154 return 0;
155 } 155 }
156 assert(it->second != NULL); 156 assert(it->second != NULL);
157 return it->second->IncomingRate(); 157 return it->second->IncomingRate();
158 } 158 }
159 159
160 VideoRenderCallback* 160 rtc::VideoSinkInterface<VideoFrame>*
161 ModuleVideoRenderImpl::AddIncomingRenderStream(const uint32_t streamId, 161 ModuleVideoRenderImpl::AddIncomingRenderStream(const uint32_t streamId,
162 const uint32_t zOrder, 162 const uint32_t zOrder,
163 const float left, 163 const float left,
164 const float top, 164 const float top,
165 const float right, 165 const float right,
166 const float bottom) 166 const float bottom)
167 { 167 {
168 CriticalSectionScoped cs(&_moduleCrit); 168 CriticalSectionScoped cs(&_moduleCrit);
169 169
170 if (!_ptrRenderer) 170 if (!_ptrRenderer)
171 { 171 {
172 WEBRTC_TRACE(kTraceError, kTraceVideoRenderer, _id, 172 WEBRTC_TRACE(kTraceError, kTraceVideoRenderer, _id,
173 "%s: No renderer", __FUNCTION__); 173 "%s: No renderer", __FUNCTION__);
174 return NULL; 174 return NULL;
175 } 175 }
176 176
177 if (_streamRenderMap.find(streamId) != _streamRenderMap.end()) { 177 if (_streamRenderMap.find(streamId) != _streamRenderMap.end()) {
178 // The stream already exists... 178 // The stream already exists...
179 WEBRTC_TRACE(kTraceError, kTraceVideoRenderer, _id, 179 WEBRTC_TRACE(kTraceError, kTraceVideoRenderer, _id,
180 "%s: stream already exists", __FUNCTION__); 180 "%s: stream already exists", __FUNCTION__);
181 return NULL; 181 return NULL;
182 } 182 }
183 183
184 VideoRenderCallback* ptrRenderCallback = 184 rtc::VideoSinkInterface<VideoFrame>* ptrRenderCallback =
185 _ptrRenderer->AddIncomingRenderStream(streamId, zOrder, left, top, 185 _ptrRenderer->AddIncomingRenderStream(streamId, zOrder, left, top,
186 right, bottom); 186 right, bottom);
187 if (ptrRenderCallback == NULL) 187 if (ptrRenderCallback == NULL)
188 { 188 {
189 WEBRTC_TRACE(kTraceError, kTraceVideoRenderer, _id, 189 WEBRTC_TRACE(kTraceError, kTraceVideoRenderer, _id,
190 "%s: Can't create incoming stream in renderer", 190 "%s: Can't create incoming stream in renderer",
191 __FUNCTION__); 191 __FUNCTION__);
192 return NULL; 192 return NULL;
193 } 193 }
194 194
195 // Create platform independant code 195 // Create platform independant code
196 IncomingVideoStream* ptrIncomingStream = 196 IncomingVideoStream* ptrIncomingStream =
197 new IncomingVideoStream(streamId, false); 197 new IncomingVideoStream(false);
198 ptrIncomingStream->SetRenderCallback(ptrRenderCallback); 198 ptrIncomingStream->SetRenderCallback(ptrRenderCallback);
199 VideoRenderCallback* moduleCallback = ptrIncomingStream->ModuleCallback(); 199 rtc::VideoSinkInterface<VideoFrame>* moduleCallback = ptrIncomingStream->Mod uleCallback();
perkj_webrtc 2016/03/31 05:11:59 line length
nisse-webrtc 2016/03/31 07:07:20 Done.
200 200
201 // Store the stream 201 // Store the stream
202 _streamRenderMap[streamId] = ptrIncomingStream; 202 _streamRenderMap[streamId] = ptrIncomingStream;
203 203
204 return moduleCallback; 204 return moduleCallback;
205 } 205 }
206 206
207 int32_t ModuleVideoRenderImpl::DeleteIncomingRenderStream( 207 int32_t ModuleVideoRenderImpl::DeleteIncomingRenderStream(
208 const uint32_t s treamId) 208 const uint32_t s treamId)
209 { 209 {
(...skipping 18 matching lines...) Expand all
228 228
229 _ptrRenderer->DeleteIncomingRenderStream(streamId); 229 _ptrRenderer->DeleteIncomingRenderStream(streamId);
230 230
231 _streamRenderMap.erase(item); 231 _streamRenderMap.erase(item);
232 232
233 return 0; 233 return 0;
234 } 234 }
235 235
236 int32_t ModuleVideoRenderImpl::AddExternalRenderCallback( 236 int32_t ModuleVideoRenderImpl::AddExternalRenderCallback(
237 const uint32_t streamId, 237 const uint32_t streamId,
238 VideoRenderCallback* renderObject) { 238 rtc::VideoSinkInterface<VideoFrame>* renderObject) {
239 CriticalSectionScoped cs(&_moduleCrit); 239 CriticalSectionScoped cs(&_moduleCrit);
240 240
241 IncomingVideoStreamMap::iterator item = _streamRenderMap.find(streamId); 241 IncomingVideoStreamMap::iterator item = _streamRenderMap.find(streamId);
242 242
243 if (item == _streamRenderMap.end()) 243 if (item == _streamRenderMap.end())
244 { 244 {
245 WEBRTC_TRACE(kTraceError, kTraceVideoRenderer, _id, 245 WEBRTC_TRACE(kTraceError, kTraceVideoRenderer, _id,
246 "%s: stream doesn't exist", __FUNCTION__); 246 "%s: stream doesn't exist", __FUNCTION__);
247 return -1; 247 return -1;
248 } 248 }
(...skipping 37 matching lines...) Expand 10 before | Expand all | Expand 10 after
286 286
287 bool ModuleVideoRenderImpl::HasIncomingRenderStream( 287 bool ModuleVideoRenderImpl::HasIncomingRenderStream(
288 const uint32_t streamId) const { 288 const uint32_t streamId) const {
289 CriticalSectionScoped cs(&_moduleCrit); 289 CriticalSectionScoped cs(&_moduleCrit);
290 290
291 return _streamRenderMap.find(streamId) != _streamRenderMap.end(); 291 return _streamRenderMap.find(streamId) != _streamRenderMap.end();
292 } 292 }
293 293
294 int32_t ModuleVideoRenderImpl::RegisterRawFrameCallback( 294 int32_t ModuleVideoRenderImpl::RegisterRawFrameCallback(
295 const uint32_t streamId, 295 const uint32_t streamId,
296 VideoRenderCallback* callbackObj) { 296 rtc::VideoSinkInterface<VideoFrame>* callbackObj) {
297 return -1; 297 return -1;
298 } 298 }
299 299
300 int32_t ModuleVideoRenderImpl::StartRender(const uint32_t streamId) 300 int32_t ModuleVideoRenderImpl::StartRender(const uint32_t streamId)
301 { 301 {
302 CriticalSectionScoped cs(&_moduleCrit); 302 CriticalSectionScoped cs(&_moduleCrit);
303 303
304 if (!_ptrRenderer) 304 if (!_ptrRenderer)
305 { 305 {
306 WEBRTC_TRACE(kTraceError, kTraceVideoRenderer, _id, 306 WEBRTC_TRACE(kTraceError, kTraceVideoRenderer, _id,
(...skipping 286 matching lines...) Expand 10 before | Expand all | Expand 10 after
593 WEBRTC_TRACE(kTraceError, kTraceVideoRenderer, _id, 593 WEBRTC_TRACE(kTraceError, kTraceVideoRenderer, _id,
594 "%s: stream doesn't exist", __FUNCTION__); 594 "%s: stream doesn't exist", __FUNCTION__);
595 return -1; 595 return -1;
596 } 596 }
597 assert(item->second != NULL); 597 assert(item->second != NULL);
598 item->second->SetTimeoutImage(videoFrame, timeout); 598 item->second->SetTimeoutImage(videoFrame, timeout);
599 return 0; 599 return 0;
600 } 600 }
601 601
602 } // namespace webrtc 602 } // namespace webrtc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698