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

Unified Diff: content/browser/child_process_launcher_helper_posix.cc

Issue 2684433003: Files required by a service now listed in manifest. (Closed)
Patch Set: Fix build. Created 3 years, 10 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
Index: content/browser/child_process_launcher_helper_posix.cc
diff --git a/content/browser/child_process_launcher_helper_posix.cc b/content/browser/child_process_launcher_helper_posix.cc
index 9c3583ea6333501c26475c5c82177de78999cd5c..dca99b27a3f820b0b3ce1df9964bd1562cc929e4 100644
--- a/content/browser/child_process_launcher_helper_posix.cc
+++ b/content/browser/child_process_launcher_helper_posix.cc
@@ -6,19 +6,76 @@
#include "base/command_line.h"
#include "base/metrics/field_trial.h"
+#include "base/posix/global_descriptors.h"
+#include "base/strings/string_number_conversions.h"
+#include "base/strings/string_util.h"
#include "content/browser/file_descriptor_info_impl.h"
#include "content/public/browser/content_browser_client.h"
#include "content/public/common/content_client.h"
#include "content/public/common/content_descriptors.h"
+#include "content/public/common/content_switches.h"
#include "mojo/edk/embedder/platform_handle.h"
+#include "services/catalog/public/cpp/manifest_parsing_util.h"
+#include "services/service_manager/public/cpp/shared_file_util.h"
namespace content {
namespace internal {
+namespace {
+
+using RequiredFilesByServiceMap =
+ std::map<std::string, catalog::RequiredFileMap>;
+
+RequiredFilesByServiceMap& GetRequiredFilesByServiceMap() {
+ static auto required_files_by_service = new RequiredFilesByServiceMap();
+ return *required_files_by_service;
+}
+
+std::map<std::string, std::string>& GetServiceNameByProcessTypeMap() {
+ static auto service_name_resolver = new std::map<std::string, std::string>(
+ {// The service names are defined in the JSON manifests, so we don't have
+ // a constant accessible for them.
+ // TODO(jcivelli): remove this map once the service name is accessible
+ // from the command line crbug.com/687250
+ {switches::kGpuProcess, "content_gpu"},
+ {switches::kPpapiPluginProcess, "content_plugin"},
+ {switches::kRendererProcess, "content_renderer"},
+ {switches::kUtilityProcess, "content_utility"},
+ {"ppapi-broker", "ppapi_broker"},
+ {"nacl-loader", "nacl_loader"},
+ {"nacl-loader-nonsfi", "nacl_loader_nonsfi"}});
+ return *service_name_resolver;
+}
+
+base::PlatformFile OpenFileIfNecessary(const base::FilePath& path,
+ base::MemoryMappedFile::Region* region) {
+ static auto opened_files =
+ new std::map<base::FilePath, std::pair<base::PlatformFile,
+ base::MemoryMappedFile::Region>>;
+
+ const auto& iter = opened_files->find(path);
+ if (iter != opened_files->end()) {
+ *region = iter->second.second;
+ return iter->second.first;
+ }
+ base::File file = OpenFileToShare(path, region);
+ if (!file.IsValid()) {
+ return base::kInvalidPlatformFile;
+ }
+ // g_opened_files becomes the owner of the file descriptor.
+ base::PlatformFile fd = file.TakePlatformFile();
+ (*opened_files)[path] = std::make_pair(fd, *region);
+ return fd;
+}
+
+} // namespace
+
std::unique_ptr<FileDescriptorInfo> CreateDefaultPosixFilesToMap(
- const base::CommandLine& command_line,
int child_process_id,
- const mojo::edk::PlatformHandle& mojo_client_handle) {
+ const mojo::edk::PlatformHandle& mojo_client_handle,
+ bool include_service_required_files,
+ const std::string& process_type,
+ base::CommandLine* command_line) {
std::unique_ptr<FileDescriptorInfo> files_to_register(
FileDescriptorInfoImpl::Create());
@@ -33,11 +90,56 @@ std::unique_ptr<FileDescriptorInfo> CreateDefaultPosixFilesToMap(
// GetAdditionalMappedFilesForChildProcess a no op on Mac.
#if !defined(OS_MACOSX)
GetContentClient()->browser()->GetAdditionalMappedFilesForChildProcess(
- command_line, child_process_id, files_to_register.get());
+ *command_line, child_process_id, files_to_register.get());
#endif
+ if (!include_service_required_files)
+ return files_to_register;
+
+ // Also include the files specified in the services' manifests.
+ auto service_name_iter = GetServiceNameByProcessTypeMap().find(process_type);
+ DCHECK(service_name_iter != GetServiceNameByProcessTypeMap().end())
+ << "No service found for process type " << process_type;
+ const std::string& service_name = service_name_iter->second;
+ auto files_iter = GetRequiredFilesByServiceMap().find(service_name);
+ if (files_iter != GetRequiredFilesByServiceMap().end()) {
+ const catalog::RequiredFileMap& required_files_map = files_iter->second;
+ base::GlobalDescriptors::Key key = kContentDynamicDescriptorStart;
+ service_manager::SharedFileSwitchValueBuilder file_switch_value_builder;
+ for (const auto& key_path_iter : required_files_map) {
+ base::MemoryMappedFile::Region region;
+ base::PlatformFile file =
+ OpenFileIfNecessary(key_path_iter.second, &region);
+ if (file == base::kInvalidPlatformFile) {
+ DLOG(ERROR) << "Ignoring invalid file " << key_path_iter.second.value();
+ continue;
+ }
+ file_switch_value_builder.AddEntry(key_path_iter.first, key);
+ files_to_register->ShareWithRegion(key, file, region);
+ key++;
+ DCHECK(key < kContentDynamicDescriptorMax);
+ }
+ command_line->AppendSwitchASCII(switches::kSharedFiles,
+ file_switch_value_builder.switch_value());
+ }
+
return files_to_register;
}
+void SetFilesToShareForServicePosix(const std::string& service_name,
+ catalog::RequiredFileMap required_files) {
+ if (required_files.empty())
+ return;
+
+ if (!base::StartsWith(service_name, "content_",
+ base::CompareCase::INSENSITIVE_ASCII)) {
+ // Not a content child service, ignore.
+ return;
+ }
+
+ DCHECK(GetRequiredFilesByServiceMap().count(service_name) == 0);
+ GetRequiredFilesByServiceMap()[service_name] = std::move(required_files);
+}
+
} // namespace internal
} // namespace content
« no previous file with comments | « content/browser/child_process_launcher_helper_posix.h ('k') | content/browser/child_process_launcher_helper_win.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698