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

Unified Diff: webrtc/media/base/videobroadcaster_unittest.cc

Issue 2764133002: Revert of Add framerate to VideoSinkWants and ability to signal on overuse (Closed)
Patch Set: Created 3 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
« no previous file with comments | « webrtc/media/base/videobroadcaster.cc ('k') | webrtc/media/base/videocapturer.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/media/base/videobroadcaster_unittest.cc
diff --git a/webrtc/media/base/videobroadcaster_unittest.cc b/webrtc/media/base/videobroadcaster_unittest.cc
index ad9ccb7dbcc86bc3b60e0a47d95fce535c2c1c1e..5274868204589856369a9a1a46a9b41c68c37b7c 100644
--- a/webrtc/media/base/videobroadcaster_unittest.cc
+++ b/webrtc/media/base/videobroadcaster_unittest.cc
@@ -87,24 +87,23 @@
TEST(VideoBroadcasterTest, AppliesMinOfSinkWantsMaxPixelCount) {
VideoBroadcaster broadcaster;
- EXPECT_EQ(std::numeric_limits<int>::max(),
- broadcaster.wants().max_pixel_count);
+ EXPECT_TRUE(!broadcaster.wants().max_pixel_count);
FakeVideoRenderer sink1;
VideoSinkWants wants1;
- wants1.max_pixel_count = 1280 * 720;
+ wants1.max_pixel_count = rtc::Optional<int>(1280 * 720);
broadcaster.AddOrUpdateSink(&sink1, wants1);
- EXPECT_EQ(1280 * 720, broadcaster.wants().max_pixel_count);
+ EXPECT_EQ(1280 * 720, *broadcaster.wants().max_pixel_count);
FakeVideoRenderer sink2;
VideoSinkWants wants2;
- wants2.max_pixel_count = 640 * 360;
+ wants2.max_pixel_count = rtc::Optional<int>(640 * 360);
broadcaster.AddOrUpdateSink(&sink2, wants2);
- EXPECT_EQ(640 * 360, broadcaster.wants().max_pixel_count);
+ EXPECT_EQ(640 * 360, *broadcaster.wants().max_pixel_count);
broadcaster.RemoveSink(&sink2);
- EXPECT_EQ(1280 * 720, broadcaster.wants().max_pixel_count);
+ EXPECT_EQ(1280 * 720, *broadcaster.wants().max_pixel_count);
}
TEST(VideoBroadcasterTest, AppliesMinOfSinkWantsMaxAndTargetPixelCount) {
@@ -126,28 +125,6 @@
broadcaster.RemoveSink(&sink2);
EXPECT_EQ(1280 * 720, *broadcaster.wants().target_pixel_count);
-}
-
-TEST(VideoBroadcasterTest, AppliesMinOfSinkWantsMaxFramerate) {
- VideoBroadcaster broadcaster;
- EXPECT_EQ(std::numeric_limits<int>::max(),
- broadcaster.wants().max_framerate_fps);
-
- FakeVideoRenderer sink1;
- VideoSinkWants wants1;
- wants1.max_framerate_fps = 30;
-
- broadcaster.AddOrUpdateSink(&sink1, wants1);
- EXPECT_EQ(30, broadcaster.wants().max_framerate_fps);
-
- FakeVideoRenderer sink2;
- VideoSinkWants wants2;
- wants2.max_framerate_fps = 15;
- broadcaster.AddOrUpdateSink(&sink2, wants2);
- EXPECT_EQ(15, broadcaster.wants().max_framerate_fps);
-
- broadcaster.RemoveSink(&sink2);
- EXPECT_EQ(30, broadcaster.wants().max_framerate_fps);
}
TEST(VideoBroadcasterTest, SinkWantsBlackFrames) {
« no previous file with comments | « webrtc/media/base/videobroadcaster.cc ('k') | webrtc/media/base/videocapturer.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698