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

Side by Side Diff: talk/app/webrtc/mediastreaminterface.h

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 2012 Google Inc. 3 * Copyright 2012 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 127 matching lines...) Expand 10 before | Expand all | Expand 10 after
138 138
139 class VideoSourceInterface; 139 class VideoSourceInterface;
140 140
141 class VideoTrackInterface : public MediaStreamTrackInterface { 141 class VideoTrackInterface : public MediaStreamTrackInterface {
142 public: 142 public:
143 // Register a renderer that will render all frames received on this track. 143 // Register a renderer that will render all frames received on this track.
144 virtual void AddRenderer(VideoRendererInterface* renderer) = 0; 144 virtual void AddRenderer(VideoRendererInterface* renderer) = 0;
145 // Deregister a renderer. 145 // Deregister a renderer.
146 virtual void RemoveRenderer(VideoRendererInterface* renderer) = 0; 146 virtual void RemoveRenderer(VideoRendererInterface* renderer) = 0;
147 147
148 virtual VideoSourceInterface* GetSource() const = 0; 148 virtual VideoSourceInterface* GetSource() const = 0;
pthatcher1 2016/02/03 16:24:29 As mentioned in the other CL, I think we can just
149 virtual rtc::VideoSinkInterface<cricket::VideoFrame>* GetSink() = 0;
149 150
150 protected: 151 protected:
151 virtual ~VideoTrackInterface() {} 152 virtual ~VideoTrackInterface() {}
152 }; 153 };
153 154
154 // Interface for receiving audio data from a AudioTrack. 155 // Interface for receiving audio data from a AudioTrack.
155 class AudioTrackSinkInterface { 156 class AudioTrackSinkInterface {
156 public: 157 public:
157 virtual void OnData(const void* audio_data, 158 virtual void OnData(const void* audio_data,
158 int bits_per_sample, 159 int bits_per_sample,
(...skipping 108 matching lines...) Expand 10 before | Expand all | Expand 10 after
267 virtual bool RemoveTrack(AudioTrackInterface* track) = 0; 268 virtual bool RemoveTrack(AudioTrackInterface* track) = 0;
268 virtual bool RemoveTrack(VideoTrackInterface* track) = 0; 269 virtual bool RemoveTrack(VideoTrackInterface* track) = 0;
269 270
270 protected: 271 protected:
271 virtual ~MediaStreamInterface() {} 272 virtual ~MediaStreamInterface() {}
272 }; 273 };
273 274
274 } // namespace webrtc 275 } // namespace webrtc
275 276
276 #endif // TALK_APP_WEBRTC_MEDIASTREAMINTERFACE_H_ 277 #endif // TALK_APP_WEBRTC_MEDIASTREAMINTERFACE_H_
OLDNEW
« no previous file with comments | « no previous file | talk/app/webrtc/mediastreamprovider.h » ('j') | talk/app/webrtc/mediastreamprovider.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698