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

Unified Diff: webrtc/api/videosource.h

Issue 1759473003: Removed VideoSource dependency to ChannelManager. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Created 4 years, 10 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/api/peerconnectionfactory.cc ('k') | webrtc/api/videosource.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/api/videosource.h
diff --git a/webrtc/api/videosource.h b/webrtc/api/videosource.h
index e3ca0180d51e06a57e30ed21e73dead2c8263503..7ff479a45fa2a20956fdd8035f39398cf32ca2f4 100644
--- a/webrtc/api/videosource.h
+++ b/webrtc/api/videosource.h
@@ -17,6 +17,7 @@
#include "webrtc/api/notifier.h"
#include "webrtc/api/videosourceinterface.h"
#include "webrtc/api/videotrackrenderers.h"
+#include "webrtc/base/asyncinvoker.h"
#include "webrtc/base/scoped_ptr.h"
#include "webrtc/base/sigslot.h"
#include "webrtc/media/base/videosinkinterface.h"
@@ -48,7 +49,7 @@ class VideoSource : public Notifier<VideoSourceInterface>,
// |constraints| can be NULL and in that case the camera is opened using a
// default resolution.
static rtc::scoped_refptr<VideoSource> Create(
- cricket::ChannelManager* channel_manager,
+ rtc::Thread* worker_thread,
cricket::VideoCapturer* capturer,
const webrtc::MediaConstraintsInterface* constraints,
bool remote);
@@ -71,7 +72,7 @@ class VideoSource : public Notifier<VideoSourceInterface>,
virtual void RemoveSink(rtc::VideoSinkInterface<cricket::VideoFrame>* output);
protected:
- VideoSource(cricket::ChannelManager* channel_manager,
+ VideoSource(rtc::Thread* worker_thread,
cricket::VideoCapturer* capturer,
bool remote);
virtual ~VideoSource();
@@ -82,12 +83,13 @@ class VideoSource : public Notifier<VideoSourceInterface>,
cricket::CaptureState capture_state);
void SetState(SourceState new_state);
- cricket::ChannelManager* channel_manager_;
+ rtc::Thread* signaling_thread_;
+ rtc::Thread* worker_thread_;
+ rtc::AsyncInvoker invoker_;
rtc::scoped_ptr<cricket::VideoCapturer> video_capturer_;
+ bool started_;
rtc::scoped_ptr<cricket::VideoRenderer> frame_input_;
- std::list<rtc::VideoSinkInterface<cricket::VideoFrame>*> sinks_;
nisse-webrtc 2016/03/03 07:56:27 The sink list is now only in the capturer, right?
perkj_webrtc 2016/03/03 08:06:55 right
-
cricket::VideoFormat format_;
cricket::VideoOptions options_;
SourceState state_;
« no previous file with comments | « webrtc/api/peerconnectionfactory.cc ('k') | webrtc/api/videosource.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698