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

Unified Diff: webrtc/api/videocapturertracksource.h

Issue 2312603003: Remove stop method from VideoTrackSourceInterface. (Closed)
Patch Set: 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « webrtc/api/mediastreaminterface.h ('k') | webrtc/api/videosourceproxy.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/api/videocapturertracksource.h
diff --git a/webrtc/api/videocapturertracksource.h b/webrtc/api/videocapturertracksource.h
index e7bbfd22876c5797ff0bc90fe8579c2ba23f89f9..b0a2810b7656c15b206c621ab1c19be27bfda82a 100644
--- a/webrtc/api/videocapturertracksource.h
+++ b/webrtc/api/videocapturertracksource.h
@@ -57,8 +57,6 @@ class VideoCapturerTrackSource : public VideoTrackSource,
bool GetStats(Stats* stats) override;
- void Stop() override;
-
protected:
VideoCapturerTrackSource(rtc::Thread* worker_thread,
cricket::VideoCapturer* capturer,
@@ -67,6 +65,8 @@ class VideoCapturerTrackSource : public VideoTrackSource,
void Initialize(const webrtc::MediaConstraintsInterface* constraints);
private:
+ void Stop();
magjed_webrtc 2016/09/05 14:25:42 This function is only called in one place. You can
+
void OnStateChange(cricket::VideoCapturer* capturer,
cricket::CaptureState capture_state);
« no previous file with comments | « webrtc/api/mediastreaminterface.h ('k') | webrtc/api/videosourceproxy.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698