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

Unified Diff: content/renderer/media/webrtc/media_stream_video_webrtc_sink.h

Issue 2356663002: Implement and use VideoTrackSource directly. (Closed)
Patch Set: add test TODO 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
Index: content/renderer/media/webrtc/media_stream_video_webrtc_sink.h
diff --git a/content/renderer/media/webrtc/media_stream_video_webrtc_sink.h b/content/renderer/media/webrtc/media_stream_video_webrtc_sink.h
index 97137319b222792ad4c3610f71cd086098a016d7..fdd6233d4a39ff9dc6c7771d265ac03afc4b79d2 100644
--- a/content/renderer/media/webrtc/media_stream_video_webrtc_sink.h
+++ b/content/renderer/media/webrtc/media_stream_video_webrtc_sink.h
@@ -49,6 +49,12 @@ class MediaStreamVideoWebRtcSink : public MediaStreamVideoSink {
// Used to DCHECK that we are called on the correct thread.
base::ThreadChecker thread_checker_;
+ // |video_source_| and |video_track_source_proxy_| are held as references to
+ // outlive |video_track_| since the interfaces between them don't use
+ // reference counting.
+ class WebRtcVideoSource;
+ scoped_refptr<WebRtcVideoSource> video_source_;
+ scoped_refptr<webrtc::VideoTrackSourceInterface> video_source_proxy_;
scoped_refptr<webrtc::VideoTrackInterface> video_track_;
class WebRtcVideoSourceAdapter;

Powered by Google App Engine
This is Rietveld 408576698