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

Side by Side Diff: talk/app/webrtc/videotrack.cc

Issue 1664773002: Added VideoTrackInterface::GetSink method, for use by VideoRtpReceiver. Get rid of FrameInput. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Delete FrameInput method and FrameInputWrapper class. 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 unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * libjingle 2 * libjingle
3 * Copyright 2011 Google Inc. 3 * Copyright 2011 Google Inc.
4 * 4 *
5 * Redistribution and use in source and binary forms, with or without 5 * Redistribution and use in source and binary forms, with or without
6 * modification, are permitted provided that the following conditions are met: 6 * modification, are permitted provided that the following conditions are met:
7 * 7 *
8 * 1. Redistributions of source code must retain the above copyright notice, 8 * 1. Redistributions of source code must retain the above copyright notice,
9 * this list of conditions and the following disclaimer. 9 * this list of conditions and the following disclaimer.
10 * 2. Redistributions in binary form must reproduce the above copyright notice, 10 * 2. Redistributions in binary form must reproduce the above copyright notice,
(...skipping 40 matching lines...) Expand 10 before | Expand all | Expand 10 after
51 } 51 }
52 52
53 void VideoTrack::AddRenderer(VideoRendererInterface* renderer) { 53 void VideoTrack::AddRenderer(VideoRendererInterface* renderer) {
54 renderers_.AddRenderer(renderer); 54 renderers_.AddRenderer(renderer);
55 } 55 }
56 56
57 void VideoTrack::RemoveRenderer(VideoRendererInterface* renderer) { 57 void VideoTrack::RemoveRenderer(VideoRendererInterface* renderer) {
58 renderers_.RemoveRenderer(renderer); 58 renderers_.RemoveRenderer(renderer);
59 } 59 }
60 60
61 rtc::VideoSinkInterface<cricket::VideoFrame>* VideoTrack::GetSink() {
62 RTC_DCHECK(video_source_->remote());
63 return &renderers_;
pthatcher1 2016/02/03 16:24:29 As mentioned in the other CL, you could just imple
64 }
65
61 bool VideoTrack::set_enabled(bool enable) { 66 bool VideoTrack::set_enabled(bool enable) {
62 renderers_.SetEnabled(enable); 67 renderers_.SetEnabled(enable);
63 return MediaStreamTrack<VideoTrackInterface>::set_enabled(enable); 68 return MediaStreamTrack<VideoTrackInterface>::set_enabled(enable);
64 } 69 }
65 70
66 rtc::scoped_refptr<VideoTrack> VideoTrack::Create( 71 rtc::scoped_refptr<VideoTrack> VideoTrack::Create(
67 const std::string& id, VideoSourceInterface* source) { 72 const std::string& id, VideoSourceInterface* source) {
68 rtc::RefCountedObject<VideoTrack>* track = 73 rtc::RefCountedObject<VideoTrack>* track =
69 new rtc::RefCountedObject<VideoTrack>(id, source); 74 new rtc::RefCountedObject<VideoTrack>(id, source);
70 return track; 75 return track;
71 } 76 }
72 77
73 } // namespace webrtc 78 } // namespace webrtc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698