diff options
author | grunell@chromium.org <grunell@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-06-24 15:12:24 +0000 |
---|---|---|
committer | grunell@chromium.org <grunell@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-06-24 15:12:24 +0000 |
commit | a2b3334197b1c2c8b1f249c99e3f15269a4addd9 (patch) | |
tree | 59ddf2327ed21cbc6762af8c77508c5e422949bd /content/browser/media/webrtc_browsertest.cc | |
parent | 3ead0512f44cbbfb8e4edf0a0e86255d6ac629f6 (diff) | |
download | chromium_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.cc | 8 |
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)); |