Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 // Copyright 2013 The Chromium Authors. All rights reserved. | 1 // Copyright 2013 The Chromium Authors. All rights reserved. |
| 2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
| 3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
| 4 | 4 |
| 5 #include <stddef.h> | 5 #include <stddef.h> |
| 6 | 6 |
| 7 #include <ctime> | 7 #include <ctime> |
| 8 | 8 |
| 9 #include "base/base64.h" | |
|
mcasas
2017/03/22 17:04:55
Probably not needed anymore.
| |
| 9 #include "base/command_line.h" | 10 #include "base/command_line.h" |
| 10 #include "base/files/file_enumerator.h" | 11 #include "base/files/file_enumerator.h" |
| 11 #include "base/files/file_util.h" | 12 #include "base/files/file_util.h" |
| 12 #include "base/files/scoped_temp_dir.h" | 13 #include "base/files/scoped_temp_dir.h" |
| 13 #include "base/macros.h" | 14 #include "base/macros.h" |
| 14 #include "base/process/launch.h" | 15 #include "base/process/launch.h" |
| 15 #include "base/process/process.h" | 16 #include "base/process/process.h" |
| 16 #include "base/scoped_native_library.h" | 17 #include "base/scoped_native_library.h" |
| 17 #include "base/strings/string_number_conversions.h" | 18 #include "base/strings/string_number_conversions.h" |
| 18 #include "base/strings/string_util.h" | 19 #include "base/strings/string_util.h" |
| 19 #include "base/strings/stringprintf.h" | 20 #include "base/strings/stringprintf.h" |
| 20 #include "base/strings/utf_string_conversions.h" | 21 #include "base/strings/utf_string_conversions.h" |
| 22 #include "base/test/test_file_util.h" | |
| 21 #include "build/build_config.h" | 23 #include "build/build_config.h" |
| 22 #include "chrome/browser/media/webrtc/webrtc_browsertest_audio.h" | 24 #include "chrome/browser/media/webrtc/webrtc_browsertest_audio.h" |
| 23 #include "chrome/browser/media/webrtc/webrtc_browsertest_base.h" | 25 #include "chrome/browser/media/webrtc/webrtc_browsertest_base.h" |
| 24 #include "chrome/browser/media/webrtc/webrtc_browsertest_common.h" | 26 #include "chrome/browser/media/webrtc/webrtc_browsertest_common.h" |
| 25 #include "chrome/browser/profiles/profile.h" | 27 #include "chrome/browser/profiles/profile.h" |
| 26 #include "chrome/browser/ui/browser.h" | 28 #include "chrome/browser/ui/browser.h" |
| 27 #include "chrome/browser/ui/browser_tabstrip.h" | 29 #include "chrome/browser/ui/browser_tabstrip.h" |
| 28 #include "chrome/browser/ui/tabs/tab_strip_model.h" | 30 #include "chrome/browser/ui/tabs/tab_strip_model.h" |
| 29 #include "chrome/common/chrome_paths.h" | 31 #include "chrome/common/chrome_paths.h" |
| 30 #include "chrome/common/chrome_switches.h" | 32 #include "chrome/common/chrome_switches.h" |
| 33 #include "chrome/common/pref_names.h" | |
| 31 #include "chrome/test/base/ui_test_utils.h" | 34 #include "chrome/test/base/ui_test_utils.h" |
| 35 #include "components/prefs/pref_service.h" | |
| 32 #include "content/public/common/content_switches.h" | 36 #include "content/public/common/content_switches.h" |
| 33 #include "content/public/test/browser_test_utils.h" | 37 #include "content/public/test/browser_test_utils.h" |
| 34 #include "media/base/audio_parameters.h" | 38 #include "media/base/audio_parameters.h" |
| 35 #include "media/base/media_switches.h" | 39 #include "media/base/media_switches.h" |
| 36 #include "net/test/embedded_test_server/embedded_test_server.h" | 40 #include "net/test/embedded_test_server/embedded_test_server.h" |
| 37 #include "testing/perf/perf_test.h" | 41 #include "testing/perf/perf_test.h" |
| 38 | 42 |
| 39 namespace { | 43 namespace { |
| 40 | 44 |
| 41 static const base::FilePath::CharType kReferenceFile[] = | 45 static const base::FilePath::CharType kReferenceFile[] = |
| 42 FILE_PATH_LITERAL("speech_44kHz_16bit_stereo.wav"); | 46 FILE_PATH_LITERAL("speech_44kHz_16bit_stereo.wav"); |
| 43 | 47 |
| 44 // The javascript will load the reference file relative to its location, | 48 // The javascript will load the reference file relative to its location, |
| 45 // which is in /webrtc on the web server. The files we are looking for are in | 49 // which is in /webrtc on the web server. The files we are looking for are in |
| 46 // webrtc/resources in the chrome/test/data folder. | 50 // webrtc/resources in the chrome/test/data folder. |
| 47 static const char kReferenceFileRelativeUrl[] = | 51 static const char kReferenceFileRelativeUrl[] = |
| 48 "resources/speech_44kHz_16bit_stereo.wav"; | 52 "resources/speech_44kHz_16bit_stereo.wav"; |
| 49 | 53 |
| 50 static const char kWebRtcAudioTestHtmlPage[] = | 54 static const char kWebRtcAudioTestHtmlPage[] = |
| 51 "/webrtc/webrtc_audio_quality_test.html"; | 55 "/webrtc/webrtc_audio_quality_test.html"; |
| 52 | 56 |
| 57 // How long to record the audio in the receiving peerConnection. | |
| 58 static const int kCaptureDurationInSeconds = 25; | |
| 59 | |
| 60 // The name where the recorded WebM audio file will be saved. | |
| 61 static const char kWebmRecordingFilename[] = "recording.webm"; | |
| 62 | |
| 63 // How often to ask the test page whether the audio recording is completed. | |
| 64 const int kPollingIntervalInMs = 1000; | |
| 65 | |
| 53 // For the AGC test, there are 6 speech segments split on silence. If one | 66 // For the AGC test, there are 6 speech segments split on silence. If one |
| 54 // segment is significantly different in length compared to the same segment in | 67 // segment is significantly different in length compared to the same segment in |
| 55 // the reference file, there's something fishy going on. | 68 // the reference file, there's something fishy going on. |
| 56 const int kMaxAgcSegmentDiffMs = | 69 const int kMaxAgcSegmentDiffMs = |
| 57 #if defined(OS_MACOSX) | 70 #if defined(OS_MACOSX) |
| 58 // Something is different on Mac; http://crbug.com/477653. | 71 // Something is different on Mac; http://crbug.com/477653. |
| 59 600; | 72 600; |
| 60 #else | 73 #else |
| 61 200; | 74 200; |
| 62 #endif | 75 #endif |
| 63 | 76 |
| 64 #if defined(OS_LINUX) || defined(OS_WIN) || defined(OS_MACOSX) | 77 #if defined(OS_LINUX) || defined(OS_WIN) || defined(OS_MACOSX) |
| 65 #define MAYBE_WebRtcAudioQualityBrowserTest WebRtcAudioQualityBrowserTest | 78 #define MAYBE_WebRtcAudioQualityBrowserTest WebRtcAudioQualityBrowserTest |
| 66 #else | 79 #else |
| 67 // Not implemented on Android, ChromeOS etc. | 80 // Not implemented on Android, ChromeOS etc. |
| 68 #define MAYBE_WebRtcAudioQualityBrowserTest DISABLED_WebRtcAudioQualityBrowserTe st | 81 #define MAYBE_WebRtcAudioQualityBrowserTest DISABLED_WebRtcAudioQualityBrowserTe st |
| 69 #endif | 82 #endif |
| 70 | 83 |
| 71 } // namespace | 84 } // namespace |
| 72 | 85 |
| 73 // Test we can set up a WebRTC call and play audio through it. | 86 // Test we can set up a WebRTC call and play audio through it. |
| 74 // | 87 // |
| 75 // If you're not a googler and want to run this test, you need to provide a | 88 // If you're not a googler and want to run this test, you need to provide a |
| 76 // pesq binary for your platform (and sox.exe on windows). Read more on how | 89 // pesq binary for your platform (and sox.exe on windows). Read more on how |
| 77 // resources are managed in chrome/test/data/webrtc/resources/README. | 90 // resources are managed in chrome/test/data/webrtc/resources/README. |
| 78 // | 91 // |
| 79 // This test will only work on machines that have been configured to record | |
| 80 // their own input. | |
| 81 // | |
| 82 // On Linux: | 92 // On Linux: |
| 83 // 1. # sudo apt-get install pavucontrol sox | 93 // 1. # sudo apt-get install sox |
| 84 // 2. For the user who will run the test: # pavucontrol | |
| 85 // 3. In a separate terminal, # arecord dummy | |
| 86 // 4. In pavucontrol, go to the recording tab. | |
| 87 // 5. For the ALSA plugin [aplay]: ALSA Capture from, change from <x> to | |
| 88 // <Monitor of x>, where x is whatever your primary sound device is called. | |
| 89 // 6. Try launching chrome as the target user on the target machine, try | |
| 90 // playing, say, a YouTube video, and record with # arecord -f dat tmp.dat. | |
| 91 // Verify the recording with aplay (should have recorded what you played | |
| 92 // from chrome). | |
| 93 // | |
| 94 // Note: the volume for ALL your input devices will be forced to 100% by | |
| 95 // running this test on Linux. | |
| 96 // | 94 // |
| 97 // On Mac: | 95 // On Mac: |
| 98 // TODO(phoglund): download sox from gs instead. | 96 // TODO(phoglund): download sox from gs instead. |
| 99 // 1. Get SoundFlower: http://rogueamoeba.com/freebies/soundflower/download.php | 97 // 1. Get SoundFlower: http://rogueamoeba.com/freebies/soundflower/download.php |
| 100 // 2. Install it + reboot. | 98 // 2. Install it + reboot. |
| 101 // 3. Install MacPorts (http://www.macports.org/). | 99 // 3. Install MacPorts (http://www.macports.org/). |
| 102 // 4. Install sox: sudo port install sox. | 100 // 4. Install sox: sudo port install sox. |
| 103 // 5. (For Chrome bots) Ensure sox and rec are reachable from the env the test | 101 // 5. (For Chrome bots) Ensure sox is reachable from the env the test |
| 104 // executes in (sox and rec tends to install in /opt/, which generally isn't | 102 // executes in (sox tends to install in /opt/, which generally isn't in the |
| 105 // in the Chrome bots' env). For instance, run | 103 // Chrome bots' env). For instance, run |
| 106 // sudo ln -s /opt/local/bin/rec /usr/local/bin/rec | |
| 107 // sudo ln -s /opt/local/bin/sox /usr/local/bin/sox | 104 // sudo ln -s /opt/local/bin/sox /usr/local/bin/sox |
| 108 // 6. In Sound Preferences, set both input and output to Soundflower (2ch). | |
| 109 // Note: You will no longer hear audio on this machine, and it will no | |
| 110 // longer use any built-in mics. | |
| 111 // 7. Try launching chrome as the target user on the target machine, try | |
| 112 // playing, say, a YouTube video, and record with 'rec test.wav trim 0 5'. | |
| 113 // Stop the video in chrome and try playing back the file; you should hear | |
| 114 // a recording of the video (note; if you play back on the target machine | |
| 115 // you must revert the changes in step 3 first). | |
| 116 // | |
| 117 // On Windows 7: | |
| 118 // 1. Control panel > Sound > Manage audio devices. | |
| 119 // 2. In the recording tab, right-click in an empty space in the pane with the | |
| 120 // devices. Tick 'show disabled devices'. | |
| 121 // 3. You should see a 'stereo mix' device - this is what your speakers output. | |
| 122 // If you don't have one, your driver doesn't support stereo mix devices. | |
| 123 // Some drivers use different names for the mix device though (like "Wave"). | |
| 124 // Right click > Properties. | |
| 125 // 4. Ensure "listen to this device" is unchecked, otherwise you get echo. | |
| 126 // 5. Ensure the mix device is the default recording device. | |
| 127 // 6. Launch chrome and try playing a video with sound. You should see | |
| 128 // in the volume meter for the mix device. Configure the mix device to have | |
| 129 // 50 / 100 in level. Also go into the playback tab, right-click Speakers, | |
| 130 // and set that level to 50 / 100. Otherwise you will get distortion in | |
| 131 // the recording. | |
| 132 class MAYBE_WebRtcAudioQualityBrowserTest : public WebRtcTestBase { | 105 class MAYBE_WebRtcAudioQualityBrowserTest : public WebRtcTestBase { |
| 133 public: | 106 public: |
| 134 MAYBE_WebRtcAudioQualityBrowserTest() {} | 107 MAYBE_WebRtcAudioQualityBrowserTest() {} |
| 135 void SetUpInProcessBrowserTestFixture() override { | 108 void SetUpInProcessBrowserTestFixture() override { |
| 136 DetectErrorsInJavaScript(); // Look for errors in our rather complex js. | 109 DetectErrorsInJavaScript(); // Look for errors in our rather complex js. |
| 137 } | 110 } |
| 138 | 111 |
| 112 void SetUpOnMainThread() override { | |
| 113 base::FilePath tmp_dir; | |
| 114 EXPECT_TRUE(base::GetTempDir(&tmp_dir)); | |
| 115 webm_recorded_output_filename_ = tmp_dir.Append(kWebmRecordingFilename); | |
| 116 | |
| 117 browser()->profile()->GetPrefs()->SetFilePath( | |
| 118 prefs::kDownloadDefaultDirectory, tmp_dir); | |
| 119 browser()->profile()->GetPrefs()->SetBoolean(prefs::kPromptForDownload, | |
| 120 false); | |
| 121 } | |
| 122 | |
| 139 void SetUpCommandLine(base::CommandLine* command_line) override { | 123 void SetUpCommandLine(base::CommandLine* command_line) override { |
| 140 EXPECT_FALSE(command_line->HasSwitch( | 124 EXPECT_FALSE(command_line->HasSwitch( |
| 141 switches::kUseFakeUIForMediaStream)); | 125 switches::kUseFakeUIForMediaStream)); |
| 142 | 126 |
| 143 // The WebAudio-based tests don't care what devices are available to | 127 // The WebAudio-based tests don't care what devices are available to |
| 144 // getUserMedia, and the getUserMedia-based tests will play back a file | 128 // getUserMedia, and the getUserMedia-based tests will play back a file |
| 145 // through the fake device using using --use-file-for-fake-audio-capture. | 129 // through the fake device using using --use-file-for-fake-audio-capture. |
| 146 command_line->AppendSwitch(switches::kUseFakeDeviceForMediaStream); | 130 command_line->AppendSwitch(switches::kUseFakeDeviceForMediaStream); |
| 147 | 131 |
| 148 // Add loopback interface such that there is always connectivity. | 132 // Add loopback interface such that there is always connectivity. |
| (...skipping 35 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 184 content::WebContents* tab_contents) { | 168 content::WebContents* tab_contents) { |
| 185 EXPECT_EQ("ok-muted", ExecuteJavascript( | 169 EXPECT_EQ("ok-muted", ExecuteJavascript( |
| 186 "setMediaElementMuted('" + element_id + "', true)", tab_contents)); | 170 "setMediaElementMuted('" + element_id + "', true)", tab_contents)); |
| 187 } | 171 } |
| 188 | 172 |
| 189 protected: | 173 protected: |
| 190 void TestAutoGainControl(const base::FilePath::StringType& reference_filename, | 174 void TestAutoGainControl(const base::FilePath::StringType& reference_filename, |
| 191 const std::string& constraints, | 175 const std::string& constraints, |
| 192 const std::string& perf_modifier); | 176 const std::string& perf_modifier); |
| 193 void SetupAndRecordAudioCall(const base::FilePath& reference_file, | 177 void SetupAndRecordAudioCall(const base::FilePath& reference_file, |
| 194 const base::FilePath& recording, | 178 const base::FilePath& recorded_output_path, |
| 195 const std::string& constraints, | 179 const std::string& constraints); |
| 196 const base::TimeDelta recording_time); | |
| 197 void TestWithFakeDeviceGetUserMedia(const std::string& constraints, | 180 void TestWithFakeDeviceGetUserMedia(const std::string& constraints, |
| 198 const std::string& perf_modifier); | 181 const std::string& perf_modifier); |
| 182 | |
| 183 base::FilePath webm_recorded_output_filename_; | |
| 199 }; | 184 }; |
| 200 | 185 |
| 201 namespace { | 186 namespace { |
| 202 | 187 |
| 203 class AudioRecorder { | |
| 204 public: | |
| 205 AudioRecorder() {} | |
| 206 ~AudioRecorder() {} | |
| 207 | |
| 208 // Starts the recording program for the specified duration. Returns true | |
| 209 // on success. We record in 16-bit 44.1 kHz Stereo (mostly because that's | |
| 210 // what SoundRecorder.exe will give us and we can't change that). | |
| 211 bool StartRecording(base::TimeDelta recording_time, | |
| 212 const base::FilePath& output_file) { | |
| 213 EXPECT_FALSE(recording_application_.IsValid()) | |
| 214 << "Tried to record, but is already recording."; | |
| 215 | |
| 216 int duration_sec = static_cast<int>(recording_time.InSeconds()); | |
| 217 base::CommandLine command_line(base::CommandLine::NO_PROGRAM); | |
| 218 | |
| 219 #if defined(OS_WIN) | |
| 220 // This disable is required to run SoundRecorder.exe on 64-bit Windows | |
| 221 // from a 32-bit binary. We need to load the wow64 disable function from | |
| 222 // the DLL since it doesn't exist on Windows XP. | |
| 223 base::ScopedNativeLibrary kernel32_lib(base::FilePath(L"kernel32")); | |
| 224 if (kernel32_lib.is_valid()) { | |
| 225 typedef BOOL (WINAPI* Wow64DisableWow64FSRedirection)(PVOID*); | |
| 226 Wow64DisableWow64FSRedirection wow_64_disable_wow_64_fs_redirection; | |
| 227 wow_64_disable_wow_64_fs_redirection = | |
| 228 reinterpret_cast<Wow64DisableWow64FSRedirection>( | |
| 229 kernel32_lib.GetFunctionPointer( | |
| 230 "Wow64DisableWow64FsRedirection")); | |
| 231 if (wow_64_disable_wow_64_fs_redirection != NULL) { | |
| 232 PVOID* ignored = NULL; | |
| 233 wow_64_disable_wow_64_fs_redirection(ignored); | |
| 234 } | |
| 235 } | |
| 236 | |
| 237 char duration_in_hms[128] = {0}; | |
| 238 struct tm duration_tm = {0}; | |
| 239 duration_tm.tm_sec = duration_sec; | |
| 240 EXPECT_NE(0u, strftime(duration_in_hms, arraysize(duration_in_hms), | |
| 241 "%H:%M:%S", &duration_tm)); | |
| 242 | |
| 243 command_line.SetProgram( | |
| 244 base::FilePath(FILE_PATH_LITERAL("SoundRecorder.exe"))); | |
| 245 command_line.AppendArg("/FILE"); | |
| 246 command_line.AppendArgPath(output_file); | |
| 247 command_line.AppendArg("/DURATION"); | |
| 248 command_line.AppendArg(duration_in_hms); | |
| 249 #elif defined(OS_MACOSX) | |
| 250 command_line.SetProgram(base::FilePath("rec")); | |
| 251 command_line.AppendArg("-b"); | |
| 252 command_line.AppendArg("16"); | |
| 253 command_line.AppendArg("-q"); | |
| 254 command_line.AppendArgPath(output_file); | |
| 255 command_line.AppendArg("trim"); | |
| 256 command_line.AppendArg("0"); | |
| 257 command_line.AppendArg(base::IntToString(duration_sec)); | |
| 258 #else | |
| 259 command_line.SetProgram(base::FilePath("arecord")); | |
| 260 command_line.AppendArg("-d"); | |
| 261 command_line.AppendArg(base::IntToString(duration_sec)); | |
| 262 command_line.AppendArg("-f"); | |
| 263 command_line.AppendArg("cd"); | |
| 264 command_line.AppendArg("-c"); | |
| 265 command_line.AppendArg("2"); | |
| 266 command_line.AppendArgPath(output_file); | |
| 267 #endif | |
| 268 | |
| 269 DVLOG(0) << "Running " << command_line.GetCommandLineString(); | |
| 270 recording_application_ = | |
| 271 base::LaunchProcess(command_line, base::LaunchOptions()); | |
| 272 return recording_application_.IsValid(); | |
| 273 } | |
| 274 | |
| 275 // Joins the recording program. Returns true on success. | |
| 276 bool WaitForRecordingToEnd() { | |
| 277 int exit_code = -1; | |
| 278 recording_application_.WaitForExit(&exit_code); | |
| 279 return exit_code == 0; | |
| 280 } | |
| 281 private: | |
| 282 base::Process recording_application_; | |
| 283 }; | |
| 284 | |
| 285 bool ForceMicrophoneVolumeTo100Percent() { | |
| 286 #if defined(OS_WIN) | |
| 287 // Note: the force binary isn't in tools since it's one of our own. | |
| 288 base::CommandLine command_line(test::GetReferenceFilesDir().Append( | |
| 289 FILE_PATH_LITERAL("force_mic_volume_max.exe"))); | |
| 290 DVLOG(0) << "Running " << command_line.GetCommandLineString(); | |
| 291 std::string result; | |
| 292 if (!base::GetAppOutput(command_line, &result)) { | |
| 293 LOG(ERROR) << "Failed to set source volume: output was " << result; | |
| 294 return false; | |
| 295 } | |
| 296 #elif defined(OS_MACOSX) | |
| 297 base::CommandLine command_line( | |
| 298 base::FilePath(FILE_PATH_LITERAL("osascript"))); | |
| 299 command_line.AppendArg("-e"); | |
| 300 command_line.AppendArg("set volume input volume 100"); | |
| 301 command_line.AppendArg("-e"); | |
| 302 command_line.AppendArg("set volume output volume 85"); | |
| 303 | |
| 304 std::string result; | |
| 305 if (!base::GetAppOutput(command_line, &result)) { | |
| 306 LOG(ERROR) << "Failed to set source volume: output was " << result; | |
| 307 return false; | |
| 308 } | |
| 309 #else | |
| 310 // Just force the volume of, say the first 5 devices. A machine will rarely | |
| 311 // have more input sources than that. This is way easier than finding the | |
| 312 // input device we happen to be using. | |
| 313 for (int device_index = 0; device_index < 5; ++device_index) { | |
| 314 std::string result; | |
| 315 const std::string kHundredPercentVolume = "65536"; | |
| 316 base::CommandLine command_line(base::FilePath(FILE_PATH_LITERAL("pacmd"))); | |
| 317 command_line.AppendArg("set-source-volume"); | |
| 318 command_line.AppendArg(base::IntToString(device_index)); | |
| 319 command_line.AppendArg(kHundredPercentVolume); | |
| 320 DVLOG(0) << "Running " << command_line.GetCommandLineString(); | |
| 321 if (!base::GetAppOutput(command_line, &result)) { | |
| 322 LOG(ERROR) << "Failed to set source volume: output was " << result; | |
| 323 return false; | |
| 324 } | |
| 325 } | |
| 326 #endif | |
| 327 return true; | |
| 328 } | |
| 329 | |
| 330 // Sox is the "Swiss army knife" of audio processing. We mainly use it for | 188 // Sox is the "Swiss army knife" of audio processing. We mainly use it for |
| 331 // silence trimming. See http://sox.sourceforge.net. | 189 // silence trimming. See http://sox.sourceforge.net. |
| 332 base::CommandLine MakeSoxCommandLine() { | 190 base::CommandLine MakeSoxCommandLine() { |
| 333 #if defined(OS_WIN) | 191 #if defined(OS_WIN) |
| 334 base::FilePath sox_path = test::GetToolForPlatform("sox"); | 192 base::FilePath sox_path = test::GetToolForPlatform("sox"); |
| 335 if (!base::PathExists(sox_path)) { | 193 if (!base::PathExists(sox_path)) { |
| 336 LOG(ERROR) << "Missing sox.exe binary in " << sox_path.value() | 194 LOG(ERROR) << "Missing sox.exe binary in " << sox_path.value() |
| 337 << "; you may have to provide this binary yourself."; | 195 << "; you may have to provide this binary yourself."; |
| 338 return base::CommandLine(base::CommandLine::NO_PROGRAM); | 196 return base::CommandLine(base::CommandLine::NO_PROGRAM); |
| 339 } | 197 } |
| (...skipping 40 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 380 command_line.AppendArg(kTreshold); | 238 command_line.AppendArg(kTreshold); |
| 381 command_line.AppendArg("reverse"); | 239 command_line.AppendArg("reverse"); |
| 382 | 240 |
| 383 DVLOG(0) << "Running " << command_line.GetCommandLineString(); | 241 DVLOG(0) << "Running " << command_line.GetCommandLineString(); |
| 384 std::string result; | 242 std::string result; |
| 385 bool ok = base::GetAppOutput(command_line, &result); | 243 bool ok = base::GetAppOutput(command_line, &result); |
| 386 DVLOG(0) << "Output was:\n\n" << result; | 244 DVLOG(0) << "Output was:\n\n" << result; |
| 387 return ok; | 245 return ok; |
| 388 } | 246 } |
| 389 | 247 |
| 248 // Runs ffmpeg on the captured webm video and writes it to a .wav file. | |
| 249 bool RunWebmToWavConverter(const base::FilePath& webm_recorded_output_path, | |
| 250 const base::FilePath& wav_recorded_output_path) { | |
| 251 const base::FilePath path_to_ffmpeg = test::GetToolForPlatform("ffmpeg"); | |
| 252 if (!base::PathExists(path_to_ffmpeg)) { | |
| 253 LOG(ERROR) << "Missing ffmpeg: should be in " << path_to_ffmpeg.value(); | |
| 254 return false; | |
| 255 } | |
| 256 | |
| 257 // Set up ffmpeg to output at a certain bitrate (-ab). This is hopefully set | |
| 258 // high enough to avoid degrading audio quality too much. | |
| 259 base::CommandLine ffmpeg_command(path_to_ffmpeg); | |
| 260 ffmpeg_command.AppendArg("-i"); | |
| 261 ffmpeg_command.AppendArgPath(webm_recorded_output_path); | |
| 262 ffmpeg_command.AppendArg("-ab"); | |
| 263 ffmpeg_command.AppendArg("300k"); | |
| 264 ffmpeg_command.AppendArg("-y"); | |
| 265 ffmpeg_command.AppendArgPath(wav_recorded_output_path); | |
| 266 | |
| 267 // We produce an output file that will later be used as an input to the | |
| 268 // barcode decoder and frame analyzer tools. | |
| 269 DVLOG(0) << "Running " << ffmpeg_command.GetCommandLineString(); | |
| 270 std::string result; | |
| 271 bool ok = base::GetAppOutputAndError(ffmpeg_command, &result); | |
| 272 DVLOG(0) << "Output was:\n\n" << result; | |
| 273 return ok; | |
| 274 } | |
| 275 | |
| 390 // Looks for 0.2 second audio segments surrounded by silences under 0.3% audio | 276 // Looks for 0.2 second audio segments surrounded by silences under 0.3% audio |
| 391 // power and splits the input file on those silences. Output files are written | 277 // power and splits the input file on those silences. Output files are written |
| 392 // according to the output file template (e.g. /tmp/out.wav writes | 278 // according to the output file template (e.g. /tmp/out.wav writes |
| 393 // /tmp/out001.wav, /tmp/out002.wav, etc if there are two silence-padded | 279 // /tmp/out001.wav, /tmp/out002.wav, etc if there are two silence-padded |
| 394 // regions in the file). The silences between speech segments must be at | 280 // regions in the file). The silences between speech segments must be at |
| 395 // least 500 ms for this to be reliable. | 281 // least 500 ms for this to be reliable. |
| 396 bool SplitFileOnSilence(const base::FilePath& input_file, | 282 bool SplitFileOnSilence(const base::FilePath& input_file, |
| 397 const base::FilePath& output_file_template) { | 283 const base::FilePath& output_file_template) { |
| 398 base::CommandLine command_line = MakeSoxCommandLine(); | 284 base::CommandLine command_line = MakeSoxCommandLine(); |
| 399 if (command_line.GetProgram().empty()) | 285 if (command_line.GetProgram().empty()) |
| (...skipping 180 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 580 float difference_in_decibel = AnalyzeOneSegment(ref_segments[i], | 466 float difference_in_decibel = AnalyzeOneSegment(ref_segments[i], |
| 581 actual_segments[i], | 467 actual_segments[i], |
| 582 i); | 468 i); |
| 583 std::string trace_name = MakeTraceName(reference_file, i); | 469 std::string trace_name = MakeTraceName(reference_file, i); |
| 584 perf_test::PrintResult("agc_energy_diff", perf_modifier, trace_name, | 470 perf_test::PrintResult("agc_energy_diff", perf_modifier, trace_name, |
| 585 difference_in_decibel, "dB", false); | 471 difference_in_decibel, "dB", false); |
| 586 } | 472 } |
| 587 } | 473 } |
| 588 | 474 |
| 589 void ComputeAndPrintPesqResults(const base::FilePath& reference_file, | 475 void ComputeAndPrintPesqResults(const base::FilePath& reference_file, |
| 590 const base::FilePath& recording, | 476 const base::FilePath& recorded_output_path, |
| 591 const std::string& perf_modifier) { | 477 const std::string& perf_modifier) { |
| 592 base::FilePath trimmed_reference = CreateTemporaryWaveFile(); | 478 base::FilePath trimmed_reference = CreateTemporaryWaveFile(); |
| 593 base::FilePath trimmed_recording = CreateTemporaryWaveFile(); | 479 base::FilePath trimmed_recording = CreateTemporaryWaveFile(); |
| 594 | 480 |
| 595 ASSERT_TRUE(RemoveSilence(reference_file, trimmed_reference)); | 481 ASSERT_TRUE(RemoveSilence(reference_file, trimmed_reference)); |
| 596 ASSERT_TRUE(RemoveSilence(recording, trimmed_recording)); | 482 ASSERT_TRUE(RemoveSilence(recorded_output_path, trimmed_recording)); |
| 597 | 483 |
| 598 std::string raw_mos; | 484 std::string raw_mos; |
| 599 std::string mos_lqo; | 485 std::string mos_lqo; |
| 600 bool succeeded = RunPesq(trimmed_reference, trimmed_recording, 16000, | 486 bool succeeded = RunPesq(trimmed_reference, trimmed_recording, 16000, |
| 601 &raw_mos, &mos_lqo); | 487 &raw_mos, &mos_lqo); |
| 602 EXPECT_TRUE(succeeded) << "Failed to run PESQ."; | 488 EXPECT_TRUE(succeeded) << "Failed to run PESQ."; |
| 603 if (succeeded) { | 489 if (succeeded) { |
| 604 perf_test::PrintResult( | 490 perf_test::PrintResult( |
| 605 "audio_pesq", perf_modifier, "raw_mos", raw_mos, "score", true); | 491 "audio_pesq", perf_modifier, "raw_mos", raw_mos, "score", true); |
| 606 perf_test::PrintResult( | 492 perf_test::PrintResult( |
| 607 "audio_pesq", perf_modifier, "mos_lqo", mos_lqo, "score", true); | 493 "audio_pesq", perf_modifier, "mos_lqo", mos_lqo, "score", true); |
| 608 } | 494 } |
| 609 | 495 |
| 610 DeleteFileUnlessTestFailed(trimmed_reference, false); | 496 DeleteFileUnlessTestFailed(trimmed_reference, false); |
| 611 DeleteFileUnlessTestFailed(trimmed_recording, false); | 497 DeleteFileUnlessTestFailed(trimmed_recording, false); |
| 612 } | 498 } |
| 613 | 499 |
| 614 } // namespace | 500 } // namespace |
| 615 | 501 |
| 616 // Sets up a two-way WebRTC call and records its output to |recording|, using | 502 // Sets up a two-way WebRTC call and records its output to |
| 617 // getUserMedia. | 503 // |recorded_output_path|, using getUserMedia. |
| 618 // | 504 // |
| 619 // |reference_file| should have at least five seconds of silence in the | 505 // |reference_file| should have at least five seconds of silence in the |
| 620 // beginning: otherwise all the reference audio will not be picked up by the | 506 // beginning: otherwise all the reference audio will not be picked up by the |
| 621 // recording. Note that the reference file will start playing as soon as the | 507 // recording. Note that the reference file will start playing as soon as the |
| 622 // audio device is up following the getUserMedia call in the left tab. The time | 508 // audio device is up following the getUserMedia call in the left tab. The time |
| 623 // it takes to negotiate a call isn't deterministic, but five seconds should be | 509 // it takes to negotiate a call isn't deterministic, but five seconds should be |
| 624 // plenty of time. Similarly, the recording time should be enough to catch the | 510 // plenty of time. Similarly, the recording time should be enough to catch the |
| 625 // whole reference file. If you then silence-trim the reference file and actual | 511 // whole reference file. If you then silence-trim the reference file and actual |
| 626 // file, you should end up with two time-synchronized files. | 512 // file, you should end up with two time-synchronized files. |
| 627 void MAYBE_WebRtcAudioQualityBrowserTest::SetupAndRecordAudioCall( | 513 void MAYBE_WebRtcAudioQualityBrowserTest::SetupAndRecordAudioCall( |
| 628 const base::FilePath& reference_file, | 514 const base::FilePath& reference_file, |
| 629 const base::FilePath& recording, | 515 const base::FilePath& recorded_output_path, |
| 630 const std::string& constraints, | 516 const std::string& constraints) { |
| 631 const base::TimeDelta recording_time) { | |
| 632 ASSERT_TRUE(embedded_test_server()->Start()); | 517 ASSERT_TRUE(embedded_test_server()->Start()); |
| 633 ASSERT_TRUE(test::HasReferenceFilesInCheckout()); | 518 ASSERT_TRUE(test::HasReferenceFilesInCheckout()); |
| 634 ASSERT_TRUE(ForceMicrophoneVolumeTo100Percent()); | |
| 635 | 519 |
| 636 ConfigureFakeDeviceToPlayFile(reference_file); | 520 ConfigureFakeDeviceToPlayFile(reference_file); |
| 637 | 521 |
| 638 // Create a two-way call. Mute one of the receivers though; that way it will | 522 // Create a two-way call. Mute one of the receivers though; that way it will |
| 639 // be receiving audio bytes, but we will not be playing out of both elements. | 523 // be receiving audio bytes, but we will not be playing out of both elements. |
| 640 GURL test_page = embedded_test_server()->GetURL(kWebRtcAudioTestHtmlPage); | 524 GURL test_page = embedded_test_server()->GetURL(kWebRtcAudioTestHtmlPage); |
| 641 content::WebContents* left_tab = | 525 content::WebContents* left_tab = |
| 642 OpenPageAndGetUserMediaInNewTabWithConstraints(test_page, constraints); | 526 OpenPageAndGetUserMediaInNewTabWithConstraints(test_page, constraints); |
| 643 SetupPeerconnectionWithLocalStream(left_tab); | 527 SetupPeerconnectionWithLocalStream(left_tab); |
| 644 MuteMediaElement("remote-view", left_tab); | 528 MuteMediaElement("remote-view", left_tab); |
| 645 | 529 |
| 646 content::WebContents* right_tab = | 530 content::WebContents* right_tab = |
| 647 OpenPageAndGetUserMediaInNewTabWithConstraints(test_page, constraints); | 531 OpenPageAndGetUserMediaInNewTabWithConstraints(test_page, constraints); |
| 648 SetupPeerconnectionWithLocalStream(right_tab); | 532 SetupPeerconnectionWithLocalStream(right_tab); |
| 649 | 533 |
| 650 AudioRecorder recorder; | |
| 651 ASSERT_TRUE(recorder.StartRecording(recording_time, recording)); | |
| 652 | |
| 653 NegotiateCall(left_tab, right_tab); | 534 NegotiateCall(left_tab, right_tab); |
| 654 | 535 |
| 655 ASSERT_TRUE(recorder.WaitForRecordingToEnd()); | 536 EXPECT_EQ( |
| 656 DVLOG(0) << "Done recording to " << recording.value() << std::endl; | 537 "ok-capturing", |
| 538 ExecuteJavascript( | |
| 539 base::StringPrintf("startAudioCapture(%d, \"%s\");", | |
| 540 kCaptureDurationInSeconds, kWebmRecordingFilename), | |
| 541 right_tab)); | |
| 542 | |
| 543 EXPECT_TRUE(test::PollingWaitUntil("getCaptureStatus();", "done-capturing", | |
| 544 right_tab, kPollingIntervalInMs)); | |
| 657 | 545 |
| 658 HangUp(left_tab); | 546 HangUp(left_tab); |
| 547 | |
| 548 RunWebmToWavConverter(webm_recorded_output_filename_, recorded_output_path); | |
| 549 EXPECT_TRUE(base::DieFileDie(webm_recorded_output_filename_, false)); | |
| 550 | |
| 551 DVLOG(0) << "Done recording to " << recorded_output_path.MaybeAsASCII(); | |
| 659 } | 552 } |
| 660 | 553 |
| 661 void MAYBE_WebRtcAudioQualityBrowserTest::TestWithFakeDeviceGetUserMedia( | 554 void MAYBE_WebRtcAudioQualityBrowserTest::TestWithFakeDeviceGetUserMedia( |
| 662 const std::string& constraints, | 555 const std::string& constraints, |
| 663 const std::string& perf_modifier) { | 556 const std::string& perf_modifier) { |
| 664 if (OnWin8()) { | 557 if (OnWin8()) { |
| 665 // http://crbug.com/379798. | 558 // http://crbug.com/379798. |
| 666 LOG(ERROR) << "This test is not implemented for Windows XP/Win8."; | 559 LOG(ERROR) << "This test is not implemented for Windows XP/Win8."; |
| 667 return; | 560 return; |
| 668 } | 561 } |
| 669 | 562 |
| 670 base::FilePath reference_file = | 563 base::FilePath reference_file = |
| 671 test::GetReferenceFilesDir().Append(kReferenceFile); | 564 test::GetReferenceFilesDir().Append(kReferenceFile); |
| 672 base::FilePath recording = CreateTemporaryWaveFile(); | 565 base::FilePath recorded_output_path = CreateTemporaryWaveFile(); |
| 673 | 566 |
| 674 ASSERT_NO_FATAL_FAILURE(SetupAndRecordAudioCall( | 567 ASSERT_NO_FATAL_FAILURE(SetupAndRecordAudioCall( |
| 675 reference_file, recording, constraints, | 568 reference_file, recorded_output_path, constraints)); |
| 676 base::TimeDelta::FromSeconds(30))); | |
| 677 | 569 |
| 678 ComputeAndPrintPesqResults(reference_file, recording, perf_modifier); | 570 ComputeAndPrintPesqResults(reference_file, recorded_output_path, |
| 679 DeleteFileUnlessTestFailed(recording, false); | 571 perf_modifier); |
| 572 DeleteFileUnlessTestFailed(recorded_output_path, false); | |
| 680 } | 573 } |
| 681 | 574 |
| 682 IN_PROC_BROWSER_TEST_F(MAYBE_WebRtcAudioQualityBrowserTest, | 575 IN_PROC_BROWSER_TEST_F(MAYBE_WebRtcAudioQualityBrowserTest, |
| 683 MANUAL_TestCallQualityWithAudioFromFakeDevice) { | 576 MANUAL_TestCallQualityWithAudioFromFakeDevice) { |
| 684 TestWithFakeDeviceGetUserMedia(kAudioOnlyCallConstraints, "_getusermedia"); | 577 TestWithFakeDeviceGetUserMedia(kAudioOnlyCallConstraints, "_getusermedia"); |
| 685 } | 578 } |
| 686 | 579 |
| 687 IN_PROC_BROWSER_TEST_F(MAYBE_WebRtcAudioQualityBrowserTest, | 580 IN_PROC_BROWSER_TEST_F(MAYBE_WebRtcAudioQualityBrowserTest, |
| 688 MANUAL_TestCallQualityWithAudioFromWebAudio) { | 581 MANUAL_TestCallQualityWithAudioFromWebAudio) { |
| 689 if (OnWin8()) { | 582 if (OnWin8()) { |
| 690 // http://crbug.com/379798. | 583 // http://crbug.com/379798. |
| 691 LOG(ERROR) << "This test is not implemented for Windows XP/Win8."; | 584 LOG(ERROR) << "This test is not implemented for Windows XP/Win8."; |
| 692 return; | 585 return; |
| 693 } | 586 } |
| 694 ASSERT_TRUE(test::HasReferenceFilesInCheckout()); | 587 ASSERT_TRUE(test::HasReferenceFilesInCheckout()); |
| 695 ASSERT_TRUE(embedded_test_server()->Start()); | 588 ASSERT_TRUE(embedded_test_server()->Start()); |
| 696 | 589 |
| 697 ASSERT_TRUE(ForceMicrophoneVolumeTo100Percent()); | |
| 698 | |
| 699 content::WebContents* left_tab = | 590 content::WebContents* left_tab = |
| 700 OpenPageWithoutGetUserMedia(kWebRtcAudioTestHtmlPage); | 591 OpenPageWithoutGetUserMedia(kWebRtcAudioTestHtmlPage); |
| 701 content::WebContents* right_tab = | 592 content::WebContents* right_tab = |
| 702 OpenPageWithoutGetUserMedia(kWebRtcAudioTestHtmlPage); | 593 OpenPageWithoutGetUserMedia(kWebRtcAudioTestHtmlPage); |
| 703 | 594 |
| 704 AddAudioFileToWebAudio(kReferenceFileRelativeUrl, left_tab); | 595 AddAudioFileToWebAudio(kReferenceFileRelativeUrl, left_tab); |
| 705 | 596 |
| 706 NegotiateCall(left_tab, right_tab); | 597 NegotiateCall(left_tab, right_tab); |
| 707 | 598 |
| 708 base::FilePath recording = CreateTemporaryWaveFile(); | 599 const base::FilePath recorded_output_path = CreateTemporaryWaveFile(); |
| 709 | |
| 710 // Note: the sound clip is 21.6 seconds: record for 25 seconds to get some | |
| 711 // safety margins on each side. | |
| 712 AudioRecorder recorder; | |
| 713 ASSERT_TRUE(recorder.StartRecording(base::TimeDelta::FromSeconds(25), | |
| 714 recording)); | |
| 715 | 600 |
| 716 PlayAudioFileThroughWebAudio(left_tab); | 601 PlayAudioFileThroughWebAudio(left_tab); |
| 717 | 602 |
| 718 ASSERT_TRUE(recorder.WaitForRecordingToEnd()); | 603 EXPECT_EQ( |
| 719 DVLOG(0) << "Done recording to " << recording.value() << std::endl; | 604 "ok-capturing", |
| 605 ExecuteJavascript( | |
| 606 base::StringPrintf("startAudioCapture(%d, \"%s\");", | |
| 607 kCaptureDurationInSeconds, kWebmRecordingFilename), | |
| 608 right_tab)); | |
| 609 | |
| 610 EXPECT_TRUE(test::PollingWaitUntil("getCaptureStatus();", "done-capturing", | |
| 611 right_tab, kPollingIntervalInMs)); | |
| 720 | 612 |
| 721 HangUp(left_tab); | 613 HangUp(left_tab); |
| 722 | 614 |
| 615 RunWebmToWavConverter(webm_recorded_output_filename_, recorded_output_path); | |
| 616 EXPECT_TRUE(base::DieFileDie(webm_recorded_output_filename_, false)); | |
| 617 | |
| 618 DVLOG(0) << "Done recording to " << recorded_output_path.MaybeAsASCII(); | |
| 619 | |
| 723 // Compare with the reference file on disk (this is the same file we played | 620 // Compare with the reference file on disk (this is the same file we played |
| 724 // through WebAudio earlier). | 621 // through WebAudio earlier). |
| 725 base::FilePath reference_file = | 622 base::FilePath reference_file = |
| 726 test::GetReferenceFilesDir().Append(kReferenceFile); | 623 test::GetReferenceFilesDir().Append(kReferenceFile); |
| 727 ComputeAndPrintPesqResults(reference_file, recording, "_webaudio"); | 624 ComputeAndPrintPesqResults(reference_file, recorded_output_path, "_webaudio"); |
| 728 } | 625 } |
| 729 | 626 |
| 730 /** | 627 /** |
| 731 * The auto gain control test plays a file into the fake microphone. Then it | 628 * The auto gain control test plays a file into the fake microphone. Then it |
| 732 * sets up a one-way WebRTC call with audio only and records Chrome's output on | 629 * sets up a one-way WebRTC call with audio only and records Chrome's output on |
| 733 * the receiving side using the audio loopback provided by the quality test | 630 * the receiving side using the audio loopback provided by the quality test |
| 734 * (see the class comments for more details). | 631 * (see the class comments for more details). |
| 735 * | 632 * |
| 736 * Then both the recording and reference file are split on silence. This creates | 633 * Then both the recording and reference file are split on silence. This creates |
| 737 * a number of segments with speech in them. The reason for this is to provide | 634 * a number of segments with speech in them. The reason for this is to provide |
| (...skipping 23 matching lines...) Expand all Loading... | |
| 761 const base::FilePath::StringType& reference_filename, | 658 const base::FilePath::StringType& reference_filename, |
| 762 const std::string& constraints, | 659 const std::string& constraints, |
| 763 const std::string& perf_modifier) { | 660 const std::string& perf_modifier) { |
| 764 if (OnWin8()) { | 661 if (OnWin8()) { |
| 765 // http://crbug.com/379798. | 662 // http://crbug.com/379798. |
| 766 LOG(ERROR) << "This test is not implemented for Windows XP/Win8."; | 663 LOG(ERROR) << "This test is not implemented for Windows XP/Win8."; |
| 767 return; | 664 return; |
| 768 } | 665 } |
| 769 base::FilePath reference_file = | 666 base::FilePath reference_file = |
| 770 test::GetReferenceFilesDir().Append(reference_filename); | 667 test::GetReferenceFilesDir().Append(reference_filename); |
| 771 base::FilePath recording = CreateTemporaryWaveFile(); | 668 base::FilePath recorded_output_path = CreateTemporaryWaveFile(); |
| 772 | 669 |
| 773 ASSERT_NO_FATAL_FAILURE(SetupAndRecordAudioCall( | 670 ASSERT_NO_FATAL_FAILURE(SetupAndRecordAudioCall( |
| 774 reference_file, recording, constraints, | 671 reference_file, recorded_output_path, constraints)); |
| 775 base::TimeDelta::FromSeconds(30))); | |
| 776 | 672 |
| 777 base::ScopedTempDir split_ref_files; | 673 base::ScopedTempDir split_ref_files; |
| 778 ASSERT_TRUE(split_ref_files.CreateUniqueTempDir()); | 674 ASSERT_TRUE(split_ref_files.CreateUniqueTempDir()); |
| 779 ASSERT_NO_FATAL_FAILURE( | 675 ASSERT_NO_FATAL_FAILURE( |
| 780 SplitFileOnSilenceIntoDir(reference_file, split_ref_files.GetPath())); | 676 SplitFileOnSilenceIntoDir(reference_file, split_ref_files.GetPath())); |
| 781 std::vector<base::FilePath> ref_segments = | 677 std::vector<base::FilePath> ref_segments = |
| 782 ListWavFilesInDir(split_ref_files.GetPath()); | 678 ListWavFilesInDir(split_ref_files.GetPath()); |
| 783 | 679 |
| 784 base::ScopedTempDir split_actual_files; | 680 base::ScopedTempDir split_actual_files; |
| 785 ASSERT_TRUE(split_actual_files.CreateUniqueTempDir()); | 681 ASSERT_TRUE(split_actual_files.CreateUniqueTempDir()); |
| 786 ASSERT_NO_FATAL_FAILURE( | 682 ASSERT_NO_FATAL_FAILURE(SplitFileOnSilenceIntoDir( |
| 787 SplitFileOnSilenceIntoDir(recording, split_actual_files.GetPath())); | 683 recorded_output_path, split_actual_files.GetPath())); |
| 788 | 684 |
| 789 // Keep the recording and split files if the analysis fails. | 685 // Keep the recording and split files if the analysis fails. |
| 790 base::FilePath actual_files_dir = split_actual_files.Take(); | 686 base::FilePath actual_files_dir = split_actual_files.Take(); |
| 791 std::vector<base::FilePath> actual_segments = | 687 std::vector<base::FilePath> actual_segments = |
| 792 ListWavFilesInDir(actual_files_dir); | 688 ListWavFilesInDir(actual_files_dir); |
| 793 | 689 |
| 794 AnalyzeSegmentsAndPrintResult( | 690 AnalyzeSegmentsAndPrintResult( |
| 795 ref_segments, actual_segments, reference_file, perf_modifier); | 691 ref_segments, actual_segments, reference_file, perf_modifier); |
| 796 | 692 |
| 797 DeleteFileUnlessTestFailed(recording, false); | 693 DeleteFileUnlessTestFailed(recorded_output_path, false); |
| 798 DeleteFileUnlessTestFailed(actual_files_dir, true); | 694 DeleteFileUnlessTestFailed(actual_files_dir, true); |
| 799 } | 695 } |
| 800 | 696 |
| 801 // The AGC should apply non-zero gain here. | 697 // The AGC should apply non-zero gain here. |
| 802 IN_PROC_BROWSER_TEST_F(MAYBE_WebRtcAudioQualityBrowserTest, | 698 IN_PROC_BROWSER_TEST_F(MAYBE_WebRtcAudioQualityBrowserTest, |
| 803 MANUAL_TestAutoGainControlOnLowAudio) { | 699 MANUAL_TestAutoGainControlOnLowAudio) { |
| 804 ASSERT_NO_FATAL_FAILURE(TestAutoGainControl( | 700 ASSERT_NO_FATAL_FAILURE(TestAutoGainControl( |
| 805 kReferenceFile, kAudioOnlyCallConstraints, "_with_agc")); | 701 kReferenceFile, kAudioOnlyCallConstraints, "_with_agc")); |
| 806 } | 702 } |
| 807 | 703 |
| 808 // Since the AGC is off here there should be no gain at all. | 704 // Since the AGC is off here there should be no gain at all. |
| 809 IN_PROC_BROWSER_TEST_F(MAYBE_WebRtcAudioQualityBrowserTest, | 705 IN_PROC_BROWSER_TEST_F(MAYBE_WebRtcAudioQualityBrowserTest, |
| 810 MANUAL_TestAutoGainIsOffWithAudioProcessingOff) { | 706 MANUAL_TestAutoGainIsOffWithAudioProcessingOff) { |
| 811 const char* kAudioCallWithoutAudioProcessing = | 707 const char* kAudioCallWithoutAudioProcessing = |
| 812 "{audio: { mandatory: { echoCancellation: false } } }"; | 708 "{audio: { mandatory: { echoCancellation: false } } }"; |
| 813 ASSERT_NO_FATAL_FAILURE(TestAutoGainControl( | 709 ASSERT_NO_FATAL_FAILURE(TestAutoGainControl( |
| 814 kReferenceFile, kAudioCallWithoutAudioProcessing, "_no_agc")); | 710 kReferenceFile, kAudioCallWithoutAudioProcessing, "_no_agc")); |
| 815 } | 711 } |
| OLD | NEW |