Files required by a service now listed in manifest.

content/browser/child_process_launcher_helper_android.cc

 // Copyright 2017 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_helper.h"
 
 #include <memory>
 
+#include "base/android/apk_assets.h"
 #include "base/i18n/icu_util.h"
 #include "base/logging.h"
 #include "base/metrics/field_trial.h"
 #include "content/browser/android/child_process_launcher_android.h"
 #include "content/browser/child_process_launcher_helper_posix.h"
 #include "content/browser/file_descriptor_info_impl.h"
 #include "content/browser/web_contents/web_contents_impl.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/render_process_host.h"
 #include "content/public/common/content_descriptors.h"
@@ skipping 63 matching lines @@
 
 std::unique_ptr<FileDescriptorInfo>
 ChildProcessLauncherHelper::GetFilesToMap() {
   DCHECK_CURRENTLY_ON(BrowserThread::PROCESS_LAUNCHER);
 
   // Android WebView runs in single process, ensure that we never get here when
   // running in single process mode.
   CHECK(!command_line()->HasSwitch(switches::kSingleProcess));
 
   std::unique_ptr<FileDescriptorInfo> files_to_register =
-      CreateDefaultPosixFilesToMap(*command_line(), child_process_id(),
-                                   mojo_client_handle());
-
-#if defined(V8_USE_EXTERNAL_STARTUP_DATA)

[boliu, 2017/02/09 23:59:54]

it's not clear to me where this chunk of code went? it's not needed anymore?

[Jay Civelli, 2017/02/10 06:32:34]

Yes, it's not needed.
Now these files are specified in the service's manifest.
The goal is to have any non "primordial" file (not common to all/most services)
specified in the manifest.

-  base::MemoryMappedFile::Region region;
-  auto maybe_register = [&region, &files_to_register](int key, int fd) {
-    if (fd != -1)
-      files_to_register->ShareWithRegion(key, fd, region);
-  };
-  maybe_register(
-      kV8NativesDataDescriptor,
-      gin::V8Initializer::GetOpenNativesFileForChildProcesses(&region));
-  maybe_register(
-      kV8SnapshotDataDescriptor32,
-      gin::V8Initializer::GetOpenSnapshotFileForChildProcesses(&region, true));
-  maybe_register(
-      kV8SnapshotDataDescriptor64,
-      gin::V8Initializer::GetOpenSnapshotFileForChildProcesses(&region, false));
-
-  command_line()->AppendSwitch(::switches::kV8NativesPassedByFD);
-  command_line()->AppendSwitch(::switches::kV8SnapshotPassedByFD);
-#endif  // defined(V8_USE_EXTERNAL_STARTUP_DATA)
+      CreateDefaultPosixFilesToMap(child_process_id(), mojo_client_handle(),
+                                   true /* include_service_required_files */,
+                                   GetProcessType(), command_line());
 
 #if ICU_UTIL_DATA_IMPL == ICU_UTIL_DATA_FILE
-  int fd = base::i18n::GetIcuDataFileHandle(&region);
-  files_to_register->ShareWithRegion(kAndroidICUDataDescriptor, fd, region);
+  base::MemoryMappedFile::Region icu_region;
+  int fd = base::i18n::GetIcuDataFileHandle(&icu_region);
+  files_to_register->ShareWithRegion(kAndroidICUDataDescriptor, fd, icu_region);
 #endif  // ICU_UTIL_DATA_IMPL == ICU_UTIL_DATA_FILE
 
   return files_to_register;
 }
 
 void ChildProcessLauncherHelper::BeforeLaunchOnLauncherThread(
     const FileDescriptorInfo& files_to_register,
     base::LaunchOptions* options) {
 }
 
@@ skipping 44 matching lines @@
           << process.process.Handle();
   StopChildProcess(process.process.Handle());
 }
 
 // static
 void ChildProcessLauncherHelper::SetProcessBackgroundedOnLauncherThread(
       base::Process process, bool background) {
   SetChildProcessInForeground(process.Handle(), !background);
 }
 
+// static
+void ChildProcessLauncherHelper::SetRegisteredFilesForService(
+    const std::string& service_name,
+    std::unique_ptr<catalog::RequiredFileMap> required_files) {
+  SetFilesToShareForServicePosix(service_name, std::move(required_files));
+}
+
+// static
+base::File OpenFileToShare(const base::FilePath& path,
+                           base::MemoryMappedFile::Region* region) {
+  base::File file(base::android::OpenApkAsset(path.value(), region));

[boliu, 2017/02/09 23:59:54]

nit: one line return so RVO works?

[Jay Civelli, 2017/02/10 06:32:34]

Done.

+  return file;
+}
+
 }  // namespace internal
 }  // namespace content