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

Unified Diff: webrtc/video/end_to_end_tests.cc

Issue 1818023002: Delete class webrtc::VideoRenderer and its header file. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Created 4 years, 9 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/video/end_to_end_tests.cc
diff --git a/webrtc/video/end_to_end_tests.cc b/webrtc/video/end_to_end_tests.cc
index b86b2c9d3013f2f5f516593fa83843fabbc7eefc..b7f5933f9c82c0b6c1e730ae915485bca11b699b 100644
--- a/webrtc/video/end_to_end_tests.cc
+++ b/webrtc/video/end_to_end_tests.cc
@@ -123,7 +123,7 @@ TEST_F(EndToEndTest, RendersSingleDelayedFrame) {
// frames in the queue.
static const int kDelayRenderCallbackMs = 1000;
- class Renderer : public VideoRenderer {
+ class Renderer : public rtc::VideoSinkInterface<VideoFrame> {
public:
Renderer() : event_(false, false) {}
@@ -186,7 +186,7 @@ TEST_F(EndToEndTest, RendersSingleDelayedFrame) {
}
TEST_F(EndToEndTest, TransmitsFirstFrame) {
- class Renderer : public VideoRenderer {
+ class Renderer : public rtc::VideoSinkInterface<VideoFrame> {
public:
Renderer() : event_(false, false) {}
@@ -230,7 +230,8 @@ TEST_F(EndToEndTest, TransmitsFirstFrame) {
}
TEST_F(EndToEndTest, SendsAndReceivesVP9) {
- class VP9Observer : public test::EndToEndTest, public VideoRenderer {
+ class VP9Observer : public test::EndToEndTest,
+ public rtc::VideoSinkInterface<VideoFrame> {
public:
VP9Observer()
: EndToEndTest(2 * kDefaultTimeoutMs),
@@ -281,7 +282,8 @@ TEST_F(EndToEndTest, SendsAndReceivesVP9) {
#if defined(WEBRTC_END_TO_END_H264_TESTS)
TEST_F(EndToEndTest, SendsAndReceivesH264) {
- class H264Observer : public test::EndToEndTest, public VideoRenderer {
+ class H264Observer : public test::EndToEndTest,
+ public rtc::VideoSinkInterface<VideoFrame> {
public:
H264Observer()
: EndToEndTest(2 * kDefaultTimeoutMs),
@@ -452,7 +454,8 @@ TEST_F(EndToEndTest, ReceivesAndRetransmitsNack) {
}
TEST_F(EndToEndTest, CanReceiveFec) {
- class FecRenderObserver : public test::EndToEndTest, public VideoRenderer {
+ class FecRenderObserver : public test::EndToEndTest,
+ public rtc::VideoSinkInterface<VideoFrame> {
public:
FecRenderObserver()
: EndToEndTest(kDefaultTimeoutMs), state_(kFirstPacket) {}
@@ -828,7 +831,7 @@ TEST_F(EndToEndTest, UsesFrameCallbacks) {
static const int kWidth = 320;
static const int kHeight = 240;
- class Renderer : public VideoRenderer {
+ class Renderer : public rtc::VideoSinkInterface<VideoFrame> {
public:
Renderer() : event_(false, false) {}
@@ -928,7 +931,8 @@ TEST_F(EndToEndTest, UsesFrameCallbacks) {
void EndToEndTest::ReceivesPliAndRecovers(int rtp_history_ms) {
static const int kPacketsToDrop = 1;
- class PliObserver : public test::EndToEndTest, public VideoRenderer {
+ class PliObserver : public test::EndToEndTest,
+ public rtc::VideoSinkInterface<VideoFrame> {
public:
explicit PliObserver(int rtp_history_ms)
: EndToEndTest(kLongTimeoutMs),
@@ -1289,7 +1293,7 @@ class MultiStreamTest {
// Each renderer verifies that it receives the expected resolution, and as soon
// as every renderer has received a frame, the test finishes.
TEST_F(EndToEndTest, SendsAndReceivesMultipleStreams) {
- class VideoOutputObserver : public VideoRenderer {
+ class VideoOutputObserver : public rtc::VideoSinkInterface<VideoFrame> {
public:
VideoOutputObserver(const MultiStreamTest::CodecSettings& settings,
uint32_t ssrc,

Powered by Google App Engine
This is Rietveld 408576698