Index: content/browser/media/webrtc_browsertest.cc |
diff --git a/content/browser/media/webrtc_browsertest.cc b/content/browser/media/webrtc_browsertest.cc |
index 729d64a112f79d5363f6c89e83dc337158442585..8c97c51533c52937189b6681fc550a21189e44c5 100644 |
--- a/content/browser/media/webrtc_browsertest.cc |
+++ b/content/browser/media/webrtc_browsertest.cc |
@@ -4,6 +4,7 @@ |
#include "base/command_line.h" |
#include "base/file_util.h" |
+#include "base/strings/string_number_conversions.h" |
#include "base/strings/stringprintf.h" |
#include "base/threading/platform_thread.h" |
#include "base/values.h" |
@@ -410,6 +411,17 @@ IN_PROC_BROWSER_TEST_P(WebRtcBrowserTest, MAYBE_CallWithAecDump) { |
DisableOpusIfOnAndroid(); |
ExecuteJavascriptAndWaitForOk("call({video: true, audio: true});"); |
+ // Get the ID for the render process host. There should only be one. |
+ RenderProcessHost::iterator it( |
+ content::RenderProcessHost::AllHostsIterator()); |
+ int render_process_host_id = it.GetCurrentValue()->GetID(); |
+ EXPECT_GE(render_process_host_id, 0); |
+ |
+ // Add file extensions that we expect to be added. |
+ static const int kExpectedConsumerId = 0; |
+ dump_file = dump_file.AddExtension(base::IntToString(render_process_host_id)) |
+ .AddExtension(base::IntToString(kExpectedConsumerId)); |
+ |
EXPECT_TRUE(base::PathExists(dump_file)); |
int64 file_size = 0; |
EXPECT_TRUE(base::GetFileSize(dump_file, &file_size)); |
@@ -418,6 +430,9 @@ IN_PROC_BROWSER_TEST_P(WebRtcBrowserTest, MAYBE_CallWithAecDump) { |
base::DeleteFile(dump_file, false); |
} |
+// TODO(grunell): Add test for multiple dumps when re-use of |
+// MediaStreamAudioProcessor in AudioCapturer has been removed. |
+ |
#if defined(OS_LINUX) && !defined(OS_CHROMEOS) && defined(ARCH_CPU_ARM_FAMILY) |
// Timing out on ARM linux bot: http://crbug.com/238490 |
#define MAYBE_CallWithAecDumpEnabledThenDisabled DISABLED_CallWithAecDumpEnabledThenDisabled |