summaryrefslogtreecommitdiffstats
path: root/content/browser/media/webrtc_browsertest.cc
diff options
context:
space:
mode:
authorgrunell@chromium.org <grunell@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2014-06-24 15:12:24 +0000
committergrunell@chromium.org <grunell@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2014-06-24 15:12:24 +0000
commita2b3334197b1c2c8b1f249c99e3f15269a4addd9 (patch)
tree59ddf2327ed21cbc6762af8c77508c5e422949bd /content/browser/media/webrtc_browsertest.cc
parent3ead0512f44cbbfb8e4edf0a0e86255d6ac629f6 (diff)
downloadchromium_src-a2b3334197b1c2c8b1f249c99e3f15269a4addd9.zip
chromium_src-a2b3334197b1c2c8b1f249c99e3f15269a4addd9.tar.gz
chromium_src-a2b3334197b1c2c8b1f249c99e3f15269a4addd9.tar.bz2
Use peer pid in AEC dump filename.
This makes it easier for the user to identify which recording(s) belongs to which tab. Review URL: https://codereview.chromium.org/350623004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@279411 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'content/browser/media/webrtc_browsertest.cc')
-rw-r--r--content/browser/media/webrtc_browsertest.cc8
1 files changed, 5 insertions, 3 deletions
diff --git a/content/browser/media/webrtc_browsertest.cc b/content/browser/media/webrtc_browsertest.cc
index f5bb216..b1aa3a6 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/process/process_handle.h"
#include "base/strings/string_number_conversions.h"
#include "base/strings/stringprintf.h"
#include "base/threading/platform_thread.h"
@@ -408,12 +409,13 @@ IN_PROC_BROWSER_TEST_P(WebRtcBrowserTest, MAYBE_CallWithAecDump) {
// 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);
+ base::ProcessId render_process_id =
+ base::GetProcId(it.GetCurrentValue()->GetHandle());
+ EXPECT_NE(base::kNullProcessId, render_process_id);
// Add file extensions that we expect to be added.
static const int kExpectedConsumerId = 0;
- dump_file = dump_file.AddExtension(IntToStringType(render_process_host_id))
+ dump_file = dump_file.AddExtension(IntToStringType(render_process_id))
.AddExtension(IntToStringType(kExpectedConsumerId));
EXPECT_TRUE(base::PathExists(dump_file));