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

Side by Side Diff: webrtc/media/devices/videorendererfactory.h

Issue 2299633002: Remove all reference to carbon api (Closed)
Patch Set: Fix chromium build errors Created 4 years, 3 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/media/devices/carbonvideorenderer.cc ('k') | webrtc/media/media.gyp » ('j') | 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) 2010 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2010 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
11 // 11 //
12 // A factory to create a GUI video renderer. 12 // A factory to create a GUI video renderer.
13 13
14 #ifndef WEBRTC_MEDIA_DEVICES_VIDEORENDERERFACTORY_H_ 14 #ifndef WEBRTC_MEDIA_DEVICES_VIDEORENDERERFACTORY_H_
15 #define WEBRTC_MEDIA_DEVICES_VIDEORENDERERFACTORY_H_ 15 #define WEBRTC_MEDIA_DEVICES_VIDEORENDERERFACTORY_H_
16 16
17 #include "webrtc/media/base/videosinkinterface.h" 17 #include "webrtc/media/base/videosinkinterface.h"
18 #if defined(WEBRTC_LINUX) && defined(HAVE_GTK) 18 #if defined(WEBRTC_LINUX) && defined(HAVE_GTK)
19 #include "webrtc/media/devices/gtkvideorenderer.h" 19 #include "webrtc/media/devices/gtkvideorenderer.h"
20 #elif defined(WEBRTC_MAC) && !defined(WEBRTC_IOS) && !defined(CARBON_DEPRECATED)
21 #include "webrtc/media/devices/carbonvideorenderer.h"
22 #elif defined(WIN32) 20 #elif defined(WIN32)
23 #include "webrtc/media/devices/gdivideorenderer.h" 21 #include "webrtc/media/devices/gdivideorenderer.h"
24 #endif 22 #endif
25 23
26 namespace cricket { 24 namespace cricket {
27 25
28 class VideoRendererFactory { 26 class VideoRendererFactory {
29 public: 27 public:
30 static rtc::VideoSinkInterface<cricket::VideoFrame>* CreateGuiVideoRenderer( 28 static rtc::VideoSinkInterface<cricket::VideoFrame>* CreateGuiVideoRenderer(
31 int x, 29 int x,
32 int y) { 30 int y) {
33 #if defined(WEBRTC_LINUX) && defined(HAVE_GTK) 31 #if defined(WEBRTC_LINUX) && defined(HAVE_GTK)
34 return new GtkVideoRenderer(x, y); 32 return new GtkVideoRenderer(x, y);
35 #elif defined(WEBRTC_MAC) && !defined(WEBRTC_IOS) && \
36 !defined(CARBON_DEPRECATED)
37 CarbonVideoRenderer* renderer = new CarbonVideoRenderer(x, y);
38 // Needs to be initialized on the main thread.
39 if (renderer->Initialize()) {
40 return renderer;
41 } else {
42 delete renderer;
43 return NULL;
44 }
45 #elif defined(WIN32) 33 #elif defined(WIN32)
46 return new GdiVideoRenderer(x, y); 34 return new GdiVideoRenderer(x, y);
47 #else 35 #else
48 return NULL; 36 return NULL;
49 #endif 37 #endif
50 } 38 }
51 }; 39 };
52 40
53 } // namespace cricket 41 } // namespace cricket
54 42
55 #endif // WEBRTC_MEDIA_DEVICES_VIDEORENDERERFACTORY_H_ 43 #endif // WEBRTC_MEDIA_DEVICES_VIDEORENDERERFACTORY_H_
OLDNEW
« no previous file with comments | « webrtc/media/devices/carbonvideorenderer.cc ('k') | webrtc/media/media.gyp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698