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

Unified Diff: talk/app/webrtc/objc/public/RTCAVFoundationVideoSource.h

Issue 1766653002: Replace SetCapturer and SetCaptureDevice by SetSource. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Rebase. Created 4 years, 8 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: talk/app/webrtc/objc/public/RTCAVFoundationVideoSource.h
diff --git a/talk/app/webrtc/objc/public/RTCAVFoundationVideoSource.h b/talk/app/webrtc/objc/public/RTCAVFoundationVideoSource.h
index fcc8f1f2c0a256ecfab27b5d8cce4adc1019da89..a73e26863c9d672875e0424bf9c6acb673fa876e 100644
--- a/talk/app/webrtc/objc/public/RTCAVFoundationVideoSource.h
+++ b/talk/app/webrtc/objc/public/RTCAVFoundationVideoSource.h
@@ -36,7 +36,13 @@
// that capturer because cricket::VideoCapturer is not ref counted and we cannot
// guarantee its lifetime. Instead, we expose its properties through the ref
// counted video source interface.
-@interface RTCAVFoundationVideoSource : RTCVideoSource
+@interface RTCAVFoundationVideoSource : RTCVideoSource {
+ // TODO(nisse): This is a crude hack, the pointer really is a
+ // webrtc::AVFoundationVideoCapturer*, but using a proper
+ // declaration with namespaces breaks inclusion from .m files, i.e.
+ // files which are Objective-C but not Objective-C++.
+ void *_capturer;
+}
- (instancetype)initWithFactory:(RTCPeerConnectionFactory*)factory
constraints:(RTCMediaConstraints*)constraints;

Powered by Google App Engine
This is Rietveld 408576698