Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(317)

Unified Diff: content/browser/child_process_launcher.cc

Issue 11189068: Changing minidump process generation to be in-process on Android. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Synced again. Created 8 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « content/browser/child_process_launcher.h ('k') | content/browser/renderer_host/render_process_host_impl.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/child_process_launcher.cc
diff --git a/content/browser/child_process_launcher.cc b/content/browser/child_process_launcher.cc
index 82a0f2b7cef8b7adf88422cc03bac6d0b9b01fe4..d0f0570ca2ea80374e7b5103cc164478e8ee0495 100644
--- a/content/browser/child_process_launcher.cc
+++ b/content/browser/child_process_launcher.cc
@@ -76,6 +76,7 @@ class ChildProcessLauncher::Context
int ipcfd,
#endif
CommandLine* cmd_line,
+ int child_process_id,
Client* client) {
client_ = client;
@@ -93,6 +94,7 @@ class ChildProcessLauncher::Context
&Context::LaunchInternal,
make_scoped_refptr(this),
client_thread_id_,
+ child_process_id,
#if defined(OS_WIN)
exposed_dir,
#elif defined(OS_ANDROID)
@@ -149,6 +151,7 @@ class ChildProcessLauncher::Context
// |this_object| is NOT thread safe. Only use it to post a task back.
scoped_refptr<Context> this_object,
BrowserThread::ID client_thread_id,
+ int child_process_id,
#if defined(OS_WIN)
const FilePath& exposed_dir,
#elif defined(OS_ANDROID)
@@ -173,7 +176,8 @@ class ChildProcessLauncher::Context
base::FileDescriptor(ipcfd, false)));
GetContentClient()->browser()->
- GetAdditionalMappedFilesForChildProcess(*cmd_line, &files_to_register);
+ GetAdditionalMappedFilesForChildProcess(*cmd_line, child_process_id,
+ &files_to_register);
StartSandboxedProcess(cmd_line->argv(), files_to_register,
base::Bind(&ChildProcessLauncher::Context::OnSandboxedProcessStarted,
@@ -194,7 +198,8 @@ class ChildProcessLauncher::Context
#if !defined(OS_MACOSX)
GetContentClient()->browser()->
- GetAdditionalMappedFilesForChildProcess(*cmd_line, &files_to_register);
+ GetAdditionalMappedFilesForChildProcess(*cmd_line, child_process_id,
+ &files_to_register);
if (use_zygote) {
handle = ZygoteHostImpl::GetInstance()->ForkRequest(cmd_line->argv(),
files_to_register,
@@ -205,12 +210,11 @@ class ChildProcessLauncher::Context
{
// Convert FD mapping to FileHandleMappingVector
base::FileHandleMappingVector fds_to_map;
- for (std::vector<FileDescriptorInfo>::const_iterator
- i = files_to_register.begin(); i != files_to_register.end(); ++i) {
- const FileDescriptorInfo& fd_info = *i;
+ for (size_t i = 0; i < files_to_register.size(); ++i) {
fds_to_map.push_back(std::make_pair(
- fd_info.fd.fd,
- fd_info.id + base::GlobalDescriptors::kBaseDescriptor));
+ files_to_register[i].fd.fd,
+ files_to_register[i].id +
+ base::GlobalDescriptors::kBaseDescriptor));
}
#if !defined(OS_MACOSX)
@@ -379,6 +383,7 @@ ChildProcessLauncher::ChildProcessLauncher(
int ipcfd,
#endif
CommandLine* cmd_line,
+ int child_process_id,
Client* client) {
context_ = new Context();
context_->Launch(
@@ -392,6 +397,7 @@ ChildProcessLauncher::ChildProcessLauncher(
ipcfd,
#endif
cmd_line,
+ child_process_id,
client);
}
« no previous file with comments | « content/browser/child_process_launcher.h ('k') | content/browser/renderer_host/render_process_host_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698