Simplifying ContentBrowserClient::GetAdditionalMappedFiles...()

content/browser/child_process_launcher.cc

Index: content/browser/child_process_launcher.cc
diff --git a/content/browser/child_process_launcher.cc b/content/browser/child_process_launcher.cc
index e93bb08efd4846340e6c467b0c349efee237bdb2..2ea18bc29d4127092d1414231ce81d20c345f270 100644
--- a/content/browser/child_process_launcher.cc
+++ b/content/browser/child_process_launcher.cc
@@ -176,11 +176,7 @@ void LaunchOnLauncherThread(
 #if defined(OS_POSIX) && !defined(OS_MACOSX)
   std::map<int, base::MemoryMappedFile::Region> regions;

[boliu, 2017/01/05 07:02:08]

seems incomplete this is still here and the regions from files_to_resgister is
ignored?

[Jay Civelli, 2017/01/05 17:12:18]

You are right. This is part of a bigger CL I am breaking down and forgot some
parts. Thanks for catching this.

   GetContentClient()->browser()->GetAdditionalMappedFilesForChildProcess(
-      *cmd_line, child_process_id, files_to_register.get()
-#if defined(OS_ANDROID)
-      , &regions
-#endif
-      );
+      *cmd_line, child_process_id, files_to_register.get());
 #if defined(V8_USE_EXTERNAL_STARTUP_DATA)
   bool snapshot_loaded = false;
 #if defined(OS_ANDROID)