Commit 41b00c13 authored by Henrik Boström's avatar Henrik Boström Committed by Commit Bot

Re-enable WebRtcRtpBrowserTests no longer flaky.

Enable two tests:
- WebRtcRtpBrowserTest.AddAndRemoveTracksWithoutStream
- WebRtcRtpBrowserTest.AddAndRemoveTracksWithIndividualStreams

Also remove the AppendSwitch for "RTCRtpSender", a flag which has been
removed and enabled by default.

TBR=phoglund@chromium.org

Bug: 788760
Change-Id: I2942aed11dca4bf70e4540840ff796def64a3f1d
Reviewed-on: https://chromium-review.googlesource.com/808384Reviewed-by: default avatarHenrik Boström <hbos@chromium.org>
Commit-Queue: Henrik Boström <hbos@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521670}
parent d3293423
...@@ -22,8 +22,6 @@ class WebRtcRtpBrowserTest : public WebRtcTestBase { ...@@ -22,8 +22,6 @@ class WebRtcRtpBrowserTest : public WebRtcTestBase {
void SetUpCommandLine(base::CommandLine* command_line) override { void SetUpCommandLine(base::CommandLine* command_line) override {
command_line->AppendSwitch(switches::kUseFakeDeviceForMediaStream); command_line->AppendSwitch(switches::kUseFakeDeviceForMediaStream);
command_line->AppendSwitchASCII(switches::kEnableBlinkFeatures,
"RTCRtpSender");
// Required by |CollectGarbage|. // Required by |CollectGarbage|.
command_line->AppendSwitchASCII(switches::kJavaScriptFlags, "--expose-gc"); command_line->AppendSwitchASCII(switches::kJavaScriptFlags, "--expose-gc");
} }
...@@ -78,8 +76,7 @@ IN_PROC_BROWSER_TEST_F(WebRtcRtpBrowserTest, GetReceivers) { ...@@ -78,8 +76,7 @@ IN_PROC_BROWSER_TEST_F(WebRtcRtpBrowserTest, GetReceivers) {
VerifyRtpReceivers(right_tab_, 6); VerifyRtpReceivers(right_tab_, 6);
} }
IN_PROC_BROWSER_TEST_F(WebRtcRtpBrowserTest, IN_PROC_BROWSER_TEST_F(WebRtcRtpBrowserTest, AddAndRemoveTracksWithoutStream) {
DISABLED_AddAndRemoveTracksWithoutStream) {
StartServerAndOpenTabs(); StartServerAndOpenTabs();
SetupPeerconnectionWithoutLocalStream(left_tab_); SetupPeerconnectionWithoutLocalStream(left_tab_);
...@@ -275,7 +272,7 @@ IN_PROC_BROWSER_TEST_F(WebRtcRtpBrowserTest, ...@@ -275,7 +272,7 @@ IN_PROC_BROWSER_TEST_F(WebRtcRtpBrowserTest,
} }
IN_PROC_BROWSER_TEST_F(WebRtcRtpBrowserTest, IN_PROC_BROWSER_TEST_F(WebRtcRtpBrowserTest,
DISABLED_AddAndRemoveTracksWithIndividualStreams) { AddAndRemoveTracksWithIndividualStreams) {
StartServerAndOpenTabs(); StartServerAndOpenTabs();
SetupPeerconnectionWithoutLocalStream(left_tab_); SetupPeerconnectionWithoutLocalStream(left_tab_);
......
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