Chromium Code Reviews| Index: talk/media/base/videoengine_unittest.h |
| diff --git a/talk/media/base/videoengine_unittest.h b/talk/media/base/videoengine_unittest.h |
| index d7fa00d5582d4ca285b8e81ad160e95c994f8b07..dafc05aae32bc2be1a51c53380aef8d0f61c4fa2 100644 |
| --- a/talk/media/base/videoengine_unittest.h |
| +++ b/talk/media/base/videoengine_unittest.h |
| @@ -456,6 +456,7 @@ class VideoMediaChannelTest : public testing::Test, |
| EXPECT_EQ(codec.id, GetPayloadType(p.get())); |
| } |
| // Tests that we only get a VideoRenderer::SetSize() callback when needed. |
| + // TODO(nisse): Delete this test, since we no longer have SetSize? |
|
pthatcher1
2016/01/11 18:42:08
Yes, just remove this.
nisse-webrtc
2016/01/12 08:03:47
Done.
|
| void SendManyResizeOnce() { |
| cricket::VideoCodec codec(DefaultCodec()); |
| EXPECT_TRUE(SetOneCodec(codec)); |
| @@ -468,14 +469,12 @@ class VideoMediaChannelTest : public testing::Test, |
| EXPECT_FRAME_WAIT(2, codec.width, codec.height, kTimeout); |
| rtc::scoped_ptr<const rtc::Buffer> p(GetRtpPacket(0)); |
| EXPECT_EQ(codec.id, GetPayloadType(p.get())); |
| - EXPECT_EQ(1, renderer_.num_set_sizes()); |
| codec.width /= 2; |
| codec.height /= 2; |
| EXPECT_TRUE(SetOneCodec(codec)); |
| EXPECT_TRUE(WaitAndSendFrame(30)); |
| EXPECT_FRAME_WAIT(3, codec.width, codec.height, kTimeout); |
| - EXPECT_EQ(2, renderer_.num_set_sizes()); |
| } |
| void SendReceiveManyAndGetStats(const cricket::VideoCodec& codec, |
| int duration_sec, int fps) { |