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

Unified Diff: webrtc/test/frame_generator.h

Issue 2257413002: Replace interface VideoCapturerInput with VideoSinkInterface. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Fixed code review comments. 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: webrtc/test/frame_generator.h
diff --git a/webrtc/test/frame_generator.h b/webrtc/test/frame_generator.h
index 7f20c749e8eea9b920ce8b0afde26445eaa10499..220aceed127429249ef763e279edc0aabf5e681d 100644
--- a/webrtc/test/frame_generator.h
+++ b/webrtc/test/frame_generator.h
@@ -7,12 +7,14 @@
* in the file PATENTS. All contributing project authors may
* be found in the AUTHORS file in the root of the source tree.
*/
-#ifndef WEBRTC_COMMON_VIDEO_TEST_FRAME_GENERATOR_H_
-#define WEBRTC_COMMON_VIDEO_TEST_FRAME_GENERATOR_H_
+#ifndef WEBRTC_TEST_FRAME_GENERATOR_H_
+#define WEBRTC_TEST_FRAME_GENERATOR_H_
#include <string>
#include <vector>
+#include "webrtc/base/criticalsection.h"
+#include "webrtc/media/base/videosourceinterface.h"
#include "webrtc/typedefs.h"
#include "webrtc/video_frame.h"
@@ -20,6 +22,20 @@ namespace webrtc {
class Clock;
namespace test {
+class FrameForwarder : public rtc::VideoSourceInterface<VideoFrame> {
stefan-webrtc 2016/09/13 09:26:43 A short comment on the purpose of this class would
perkj_webrtc 2016/09/14 14:20:22 Done.
+ public:
+ FrameForwarder();
+ void IncomingCapturedFrame(const VideoFrame& video_frame);
+
+ private:
+ void AddOrUpdateSink(rtc::VideoSinkInterface<VideoFrame>* sink,
+ const rtc::VideoSinkWants& wants) override;
+ void RemoveSink(rtc::VideoSinkInterface<VideoFrame>* sink) override;
+
+ rtc::CriticalSection crit_;
+ rtc::VideoSinkInterface<VideoFrame>* sink_ GUARDED_BY(crit_);
+};
+
class FrameGenerator {
public:
FrameGenerator() {}
@@ -61,4 +77,4 @@ class FrameGenerator {
} // namespace test
} // namespace webrtc
-#endif // WEBRTC_COMMON_VIDEO_TEST_FRAME_GENERATOR_H_
+#endif // WEBRTC_TEST_FRAME_GENERATOR_H_

Powered by Google App Engine
This is Rietveld 408576698