Commit e76e85a3 authored by Guido Urdaneta's avatar Guido Urdaneta Committed by Commit Bot

Use fake audio in RenderProcessHostTest.

This CL moves the command-line switch from
CaptureStreamRenderProcessHostTest to its parent, RenderProcessHostTest.
Some RenderProcessHostTests use audio and are still flaking.

Bug: 986021
Change-Id: Ic1327890183433ef9dd737d7c8d649560e5a981f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2107540
Auto-Submit: Guido Urdaneta <guidou@chromium.org>
Reviewed-by: default avatarDale Curtis <dalecurtis@chromium.org>
Reviewed-by: default avatarJohn Abd-El-Malek <jam@chromium.org>
Commit-Queue: John Abd-El-Malek <jam@chromium.org>
Cr-Commit-Position: refs/heads/master@{#751121}
parent 99ad41ff
......@@ -186,6 +186,11 @@ class RenderProcessHostTest : public ContentBrowserTest,
}
void SetUpCommandLine(base::CommandLine* command_line) override {
#if defined(OS_LINUX)
// Due to problems with PulseAudio failing to start, use a fake audio
// stream. https://crbug.com/1047655#c70
command_line->AppendSwitch(switches::kDisableAudioOutput);
#endif
command_line->AppendSwitchASCII(
switches::kAutoplayPolicy,
switches::autoplay::kNoUserGestureRequiredPolicy);
......@@ -917,11 +922,6 @@ class CaptureStreamRenderProcessHostTest : public RenderProcessHostTest {
}
void SetUpCommandLine(base::CommandLine* command_line) override {
#if defined(OS_LINUX)
// Due to problems with PulseAudio failing to start, use a fake audio
// stream. https://crbug.com/1047655#c70
command_line->AppendSwitch(switches::kDisableAudioOutput);
#endif
// These flags are necessary to emulate camera input for getUserMedia()
// tests.
command_line->AppendSwitch(switches::kUseFakeUIForMediaStream);
......
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