Turn FileDescriptorInfo a collection class

content/browser/child_process_launcher.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/child_process_launcher.h"
 
 #include <utility>  // For std::pair.
 
 #include "base/bind.h"
 #include "base/command_line.h"
@@ skipping 27 matching lines @@
 #include "base/memory/shared_memory.h"
 #include "base/memory/singleton.h"
 #include "content/browser/renderer_host/render_sandbox_host_linux.h"
 #include "content/browser/zygote_host/zygote_host_impl_linux.h"
 #include "content/common/child_process_sandbox_support_impl_linux.h"
 #endif
 
 #if defined(OS_POSIX)
 #include "base/metrics/stats_table.h"
 #include "base/posix/global_descriptors.h"
+#include "content/browser/file_descriptor_info_impl.h"
 #endif
 
 namespace content {
 
 // Having the functionality of ChildProcessLauncher be in an internal
 // ref counted object allows us to automatically terminate the process when the
 // parent class destructs, while still holding on to state that we need.
 class ChildProcessLauncher::Context
     : public base::RefCountedThreadSafe<ChildProcessLauncher::Context> {
  public:
@@ skipping 136 matching lines @@
     if (launch_elevated) {
       base::LaunchOptions options;
       options.start_hidden = true;
       base::LaunchElevatedProcess(*cmd_line, options, &handle);
     } else {
       handle = StartSandboxedProcess(delegate, cmd_line);
     }
 #elif defined(OS_POSIX)
     std::string process_type =
         cmd_line->GetSwitchValueASCII(switches::kProcessType);
-    std::vector<FileDescriptorInfo> files_to_register;
-    files_to_register.push_back(
-        FileDescriptorInfo(kPrimaryIPCChannel,
-                           base::FileDescriptor(ipcfd, false)));
+    scoped_ptr<FileDescriptorInfo> files_to_register(
+        FileDescriptorInfoImpl::Create());
+    files_to_register->Share(kPrimaryIPCChannel, ipcfd);
     base::StatsTable* stats_table = base::StatsTable::current();
     if (stats_table &&
         base::SharedMemory::IsHandleValid(
             stats_table->GetSharedMemoryHandle())) {
-      files_to_register.push_back(
-          FileDescriptorInfo(kStatsTableSharedMemFd,
-                             stats_table->GetSharedMemoryHandle()));
+      base::FileDescriptor fd = stats_table->GetSharedMemoryHandle();
+      DCHECK(fd.auto_close);
+      files_to_register->Transfer(kStatsTableSharedMemFd,
+                                  base::ScopedFD(fd.fd));
     }
 #endif
 
 #if defined(OS_ANDROID)
     // Android WebView runs in single process, ensure that we never get here
     // when running in single process mode.
     CHECK(!cmd_line->HasSwitch(switches::kSingleProcess));
 
-    GetContentClient()->browser()->
-        GetAdditionalMappedFilesForChildProcess(*cmd_line, child_process_id,
-                                                &files_to_register);
+    GetContentClient()->browser()->GetAdditionalMappedFilesForChildProcess(
+        *cmd_line, child_process_id, files_to_register.get());
 
-    StartChildProcess(cmd_line->argv(), child_process_id, files_to_register,
+    StartChildProcess(
+        cmd_line->argv(),
+        child_process_id,
+        files_to_register.Pass(),
         base::Bind(&ChildProcessLauncher::Context::OnChildProcessStarted,
-                   this_object, client_thread_id, begin_launch_time));
+                   this_object,
+                   client_thread_id,
+                   begin_launch_time));
 
 #elif defined(OS_POSIX)
     base::ProcessHandle handle = base::kNullProcessHandle;
     // We need to close the client end of the IPC channel to reliably detect
     // child termination.
     base::ScopedFD ipcfd_closer(ipcfd);
 
 #if !defined(OS_MACOSX)
-    GetContentClient()->browser()->
-        GetAdditionalMappedFilesForChildProcess(*cmd_line, child_process_id,
-                                                &files_to_register);
+    GetContentClient()->browser()->GetAdditionalMappedFilesForChildProcess(
+        *cmd_line, child_process_id, files_to_register.get());
     if (use_zygote) {
-      handle = ZygoteHostImpl::GetInstance()->ForkRequest(cmd_line->argv(),
-                                                          files_to_register,
-                                                          process_type);
+      handle = ZygoteHostImpl::GetInstance()->ForkRequest(
+          cmd_line->argv(), files_to_register.Pass(), process_type);
     } else
     // Fall through to the normal posix case below when we're not zygoting.
 #endif  // !defined(OS_MACOSX)
     {
       // Convert FD mapping to FileHandleMappingVector
-      base::FileHandleMappingVector fds_to_map;
-      for (size_t i = 0; i < files_to_register.size(); ++i) {
-        fds_to_map.push_back(std::make_pair(
-            files_to_register[i].fd.fd,
-            files_to_register[i].id +
-                base::GlobalDescriptors::kBaseDescriptor));
-      }
+      base::FileHandleMappingVector fds_to_map =
+          files_to_register->GetMappingWithIDAdjustment(
+              base::GlobalDescriptors::kBaseDescriptor);
 
 #if !defined(OS_MACOSX)
       if (process_type == switches::kRendererProcess) {
         const int sandbox_fd =
             RenderSandboxHostLinux::GetInstance()->GetRendererSocket();
         fds_to_map.push_back(std::make_pair(
             sandbox_fd,
             GetSandboxFD()));
       }
 #endif  // defined(OS_MACOSX)
@@ skipping 250 matching lines @@
          GetHandle(), background));
 }
 
 void ChildProcessLauncher::SetTerminateChildOnShutdown(
     bool terminate_on_shutdown) {
   if (context_.get())
     context_->set_terminate_child_on_shutdown(terminate_on_shutdown);
 }
 
 }  // namespace content