Index: content/app/content_main_runner.cc |
diff --git a/content/app/content_main_runner.cc b/content/app/content_main_runner.cc |
index 934bac48a0bdbf6fff89c24b0d2d8719094f2eca..3585db17b5adf77793712738cb46253f11c81727 100644 |
--- a/content/app/content_main_runner.cc |
+++ b/content/app/content_main_runner.cc |
@@ -21,6 +21,7 @@ |
#include "base/debug/debugger.h" |
#include "base/debug/stack_trace.h" |
#include "base/feature_list.h" |
+#include "base/file_descriptor_store.h" |
#include "base/files/file_path.h" |
#include "base/i18n/icu_util.h" |
#include "base/lazy_instance.h" |
@@ -56,6 +57,7 @@ |
#include "content/public/browser/content_browser_client.h" |
#include "content/public/common/content_client.h" |
#include "content/public/common/content_constants.h" |
+#include "content/public/common/content_descriptor_keys.h" |
#include "content/public/common/content_paths.h" |
#include "content/public/common/content_switches.h" |
#include "content/public/common/main_function_params.h" |
@@ -68,6 +70,7 @@ |
#include "ipc/ipc_descriptors.h" |
#include "media/base/media.h" |
#include "ppapi/features/features.h" |
+#include "services/service_manager/public/cpp/shared_file_util.h" |
#include "ui/base/ui_base_paths.h" |
#include "ui/base/ui_base_switches.h" |
@@ -188,9 +191,9 @@ base::LazyInstance<ContentUtilityClient> |
#if defined(V8_USE_EXTERNAL_STARTUP_DATA) && defined(OS_ANDROID) |
#if defined __LP64__ |
-#define kV8SnapshotDataDescriptor kV8SnapshotDataDescriptor64 |
+#define kV8SnapshotDataDescriptor kV8Snapshot64DataDescriptor |
#else |
-#define kV8SnapshotDataDescriptor kV8SnapshotDataDescriptor32 |
+#define kV8SnapshotDataDescriptor kV8Snapshot32DataDescriptor |
#endif |
#endif |
@@ -247,6 +250,22 @@ void CommonSubprocessInit() { |
#endif |
} |
+void PopulateFDsFromCommandLine() { |
+ const std::string& shared_file_param = |
+ base::CommandLine::ForCurrentProcess()->GetSwitchValueASCII( |
+ switches::kSharedFiles); |
+ if (shared_file_param.empty()) |
+ return; |
+ |
+ std::map<int, std::string> shared_file_descriptors; |
+ service_manager::ParseSharedFileSwitchValue(shared_file_param, |
+ &shared_file_descriptors); |
+ for (const auto& descriptor : shared_file_descriptors) { |
+ base::FileDescriptorStore::GetInstance()->Set(descriptor.second, |
+ descriptor.first); |
+ } |
+} |
+ |
class ContentClientInitializer { |
public: |
static void Set(const std::string& process_type, |
@@ -537,8 +556,8 @@ class ContentMainRunnerImpl : public ContentMainRunner { |
InitializeMac(); |
#endif |
- // On Android, the command line is initialized when library is loaded and |
- // we have already started our TRACE_EVENT0. |
+ // On Android, the command line is initialized and the FDs set when the |
+ // library is loaded and we have already started our TRACE_EVENT0. |
#if !defined(OS_ANDROID) |
// argc/argv are ignored on Windows and Android; see command_line.h for |
// details. |
@@ -552,6 +571,8 @@ class ContentMainRunnerImpl : public ContentMainRunner { |
base::CommandLine::Init(argc, argv); |
+ PopulateFDsFromCommandLine(); |
+ |
base::EnableTerminationOnHeapCorruption(); |
// TODO(yiyaoliu, vadimt): Remove this once crbug.com/453640 is fixed. |
@@ -709,32 +730,23 @@ class ContentMainRunnerImpl : public ContentMainRunner { |
#if defined(V8_USE_EXTERNAL_STARTUP_DATA) |
#if defined(OS_POSIX) && !defined(OS_MACOSX) |
-#if !defined(OS_ANDROID) |
- // kV8NativesDataDescriptor and kV8SnapshotDataDescriptor could be shared |
- // with child processes via file descriptors. On Android they are set in |
- // ChildProcessService::InternalInitChildProcess, otherwise set them here. |
- if (command_line.HasSwitch(switches::kV8NativesPassedByFD)) { |
- g_fds->Set( |
- kV8NativesDataDescriptor, |
- kV8NativesDataDescriptor + base::GlobalDescriptors::kBaseDescriptor); |
- } |
- if (command_line.HasSwitch(switches::kV8SnapshotPassedByFD)) { |
- g_fds->Set( |
- kV8SnapshotDataDescriptor, |
- kV8SnapshotDataDescriptor + base::GlobalDescriptors::kBaseDescriptor); |
- } |
-#endif // !OS_ANDROID |
- int v8_natives_fd = g_fds->MaybeGet(kV8NativesDataDescriptor); |
- int v8_snapshot_fd = g_fds->MaybeGet(kV8SnapshotDataDescriptor); |
+ base::FileDescriptorStore* file_descriptor_store = |
+ base::FileDescriptorStore::GetInstance(); |
+ int v8_snapshot_fd = |
+ file_descriptor_store->MaybeGet(kV8SnapshotDataDescriptor); |
if (v8_snapshot_fd != -1) { |
- auto v8_snapshot_region = g_fds->GetRegion(kV8SnapshotDataDescriptor); |
+ auto v8_snapshot_region = |
+ file_descriptor_store->GetRegion(kV8SnapshotDataDescriptor); |
gin::V8Initializer::LoadV8SnapshotFromFD( |
v8_snapshot_fd, v8_snapshot_region.offset, v8_snapshot_region.size); |
} else { |
gin::V8Initializer::LoadV8Snapshot(); |
} |
+ int v8_natives_fd = |
+ file_descriptor_store->MaybeGet(kV8NativesDataDescriptor); |
if (v8_natives_fd != -1) { |
- auto v8_natives_region = g_fds->GetRegion(kV8NativesDataDescriptor); |
+ auto v8_natives_region = |
+ file_descriptor_store->GetRegion(kV8NativesDataDescriptor); |
gin::V8Initializer::LoadV8NativesFromFD( |
v8_natives_fd, v8_natives_region.offset, v8_natives_region.size); |
} else { |