Commit 6ebf87b0 authored by Henrik Boström's avatar Henrik Boström Committed by Commit Bot

WebRtcInternalsPerfBrowserTest: Assert flags are used to avoid timeout.

This is the same as CL
https://chromium-review.googlesource.com/c/chromium/tools/build/+/1798122
but for the codepath executed by
RunsOneWayCall60SecsAndLogsInternalMetricsDefault and other tests.

Bug: 999136
Change-Id: Ib2000128c983f57c65abfba68bdff4bbe3febcd3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1860018Reviewed-by: default avatarMirko Bonadei <mbonadei@chromium.org>
Commit-Queue: Henrik Boström <hbos@chromium.org>
Cr-Commit-Position: refs/heads/master@{#705545}
parent f68d1fb9
...@@ -184,9 +184,16 @@ class WebRtcInternalsPerfBrowserTest : public WebRtcTestBase { ...@@ -184,9 +184,16 @@ class WebRtcInternalsPerfBrowserTest : public WebRtcTestBase {
ASSERT_TRUE(test::HasReferenceFilesInCheckout()); ASSERT_TRUE(test::HasReferenceFilesInCheckout());
ASSERT_TRUE(embedded_test_server()->Start()); ASSERT_TRUE(embedded_test_server()->Start());
ASSERT_GE(TestTimeouts::test_launcher_timeout().InSeconds(), 100)
<< "This is a long-running test; you must specify "
"--test-launcher-timeout to have a value of at least 100000.";
ASSERT_GE(TestTimeouts::action_max_timeout().InSeconds(), 100) ASSERT_GE(TestTimeouts::action_max_timeout().InSeconds(), 100)
<< "This is a long-running test; you must specify " << "This is a long-running test; you must specify "
"--ui-test-action-max-timeout to have a value of at least 100000."; "--ui-test-action-max-timeout to have a value of at least 100000.";
ASSERT_LT(TestTimeouts::action_max_timeout(),
TestTimeouts::test_launcher_timeout())
<< "action_max_timeout needs to be strictly-less-than "
"test_launcher_timeout";
content::WebContents* left_tab = content::WebContents* left_tab =
OpenTestPageAndGetUserMediaInNewTab(kMainWebrtcTestHtmlPage); OpenTestPageAndGetUserMediaInNewTab(kMainWebrtcTestHtmlPage);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment