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

Unified Diff: webrtc/media/engine/webrtcvideoengine2_unittest.cc

Issue 2709733002: Merge to 57: Screen content simulcast layers should not be downscaled. (Closed)
Patch Set: Created 3 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « webrtc/media/engine/simulcast.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/media/engine/webrtcvideoengine2_unittest.cc
diff --git a/webrtc/media/engine/webrtcvideoengine2_unittest.cc b/webrtc/media/engine/webrtcvideoengine2_unittest.cc
index 24c5f1311ef02f4263861fbd131bc1a3ffd09d53..d08ce56f5f144e686452b8dfba7c3f7855557ab3 100644
--- a/webrtc/media/engine/webrtcvideoengine2_unittest.cc
+++ b/webrtc/media/engine/webrtcvideoengine2_unittest.cc
@@ -3988,6 +3988,13 @@ class WebRtcVideoChannel2SimulcastTest : public testing::Test {
expected_streams = GetSimulcastConfig(
num_configured_streams, capture_width, capture_height, 0,
kDefaultQpMax, kDefaultVideoMaxFramerate, screenshare);
+ if (screenshare) {
+ for (const webrtc::VideoStream& stream : expected_streams) {
+ // Never scale screen content.
+ EXPECT_EQ(stream.width, capture_width);
+ EXPECT_EQ(stream.height, capture_height);
+ }
+ }
} else {
webrtc::VideoStream stream;
stream.width = capture_width;
« no previous file with comments | « webrtc/media/engine/simulcast.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698