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

Unified Diff: content/browser/media/webrtc_getusermedia_browsertest.cc

Issue 195363002: VideoCapturerDelegate: Retrieve supported/in-use format(s) for constraint negotiation (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: perkj@s comments Created 6 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: content/browser/media/webrtc_getusermedia_browsertest.cc
diff --git a/content/browser/media/webrtc_getusermedia_browsertest.cc b/content/browser/media/webrtc_getusermedia_browsertest.cc
index f6699ca36d5f5e80304d2b28ba5f158befb995a9..26f1514a58f23331aaa35f5e5ed2708586187211 100644
--- a/content/browser/media/webrtc_getusermedia_browsertest.cc
+++ b/content/browser/media/webrtc_getusermedia_browsertest.cc
@@ -380,7 +380,7 @@ IN_PROC_BROWSER_TEST_P(WebRtcGetUserMediaBrowserTest,
GURL url(embedded_test_server()->GetURL("/media/getusermedia.html"));
std::string constraints_4_3 = GenerateGetUserMediaCall(
- kGetUserMediaAndAnalyseAndStop, 640, 640, 480, 480, 30, 30);
+ kGetUserMediaAndAnalyseAndStop, 640, 640, 480, 480, 30, 10);
// TODO(mcasas): add more aspect ratios, in particular 16:10 crbug.com/275594.
@@ -400,7 +400,7 @@ IN_PROC_BROWSER_TEST_P(WebRtcGetUserMediaBrowserTest,
GURL url(embedded_test_server()->GetURL("/media/getusermedia.html"));
std::string constraints_16_9 = GenerateGetUserMediaCall(
- kGetUserMediaAndAnalyseAndStop, 640, 640, 360, 360, 30, 30);
+ kGetUserMediaAndAnalyseAndStop, 640, 640, 360, 360, 30, 10);
NavigateToURL(shell(), url);
ASSERT_EQ("16:9 letterbox",
@@ -450,13 +450,13 @@ IN_PROC_BROWSER_TEST_P(WebRtcConstraintsBrowserTest, GetUserMediaConstraints) {
}
static const UserMediaSizes kAllUserMediaSizes[] = {
- {320, 320, 180, 180, 30, 30},
- {320, 320, 240, 240, 30, 30},
- {640, 640, 360, 360, 30, 30},
- {640, 640, 480, 480, 30, 30},
- {960, 960, 720, 720, 30, 30},
- {1280, 1280, 720, 720, 30, 30},
- {1920, 1920, 1080, 1080, 30, 30}};
+ {320, 320, 180, 180, 30, 10},
+ {320, 320, 240, 240, 30, 10},
+ {640, 640, 360, 360, 30, 10},
+ {640, 640, 480, 480, 30, 10},
+ {960, 960, 720, 720, 30, 10},
+ {1280, 1280, 720, 720, 30, 10},
+ {1920, 1920, 1080, 1080, 30, 10}};
INSTANTIATE_TEST_CASE_P(UserMedia,
WebRtcConstraintsBrowserTest,

Powered by Google App Engine
This is Rietveld 408576698