Pass file Regions along with FDs to child processes on Android

content/browser/android/child_process_launcher_android.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/android/child_process_launcher_android.h"
 
 #include "base/android/jni_android.h"
 #include "base/android/jni_array.h"
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 85 matching lines @@
       reinterpret_cast<StartChildProcessCallback*>(client_context);
   if (handle)
     callback->Run(static_cast<base::ProcessHandle>(handle));
   delete callback;
 }
 
 void StartChildProcess(
     const base::CommandLine::StringVector& argv,
     int child_process_id,
     scoped_ptr<content::FileDescriptorInfo> files_to_register,
+    const std::map<int, base::MemoryMappedFile::Region>& regions,
     const StartChildProcessCallback& callback) {
   JNIEnv* env = AttachCurrentThread();
   DCHECK(env);
 
   // Create the Command line String[]
   ScopedJavaLocalRef<jobjectArray> j_argv = ToJavaArrayOfStrings(env, argv);
 
   size_t file_count = files_to_register->GetMappingSize();
   DCHECK(file_count > 0);
 
-  ScopedJavaLocalRef<jintArray> j_file_ids(env, env->NewIntArray(file_count));
+  ScopedJavaLocalRef<jclass> j_file_info_class = base::android::GetClass(
+      env, "org/chromium/content/browser/FileDescriptorInfo");
+  ScopedJavaLocalRef<jobjectArray> j_file_infos(
+      env, env->NewObjectArray(file_count, j_file_info_class.obj(), NULL));
   base::android::CheckException(env);
-  jint* file_ids = env->GetIntArrayElements(j_file_ids.obj(), NULL);
-  base::android::CheckException(env);
-  ScopedJavaLocalRef<jintArray> j_file_fds(env, env->NewIntArray(file_count));
-  base::android::CheckException(env);
-  jint* file_fds = env->GetIntArrayElements(j_file_fds.obj(), NULL);
-  base::android::CheckException(env);
-  ScopedJavaLocalRef<jbooleanArray> j_file_auto_close(
-      env, env->NewBooleanArray(file_count));
-  base::android::CheckException(env);
-  jboolean* file_auto_close =
-      env->GetBooleanArrayElements(j_file_auto_close.obj(), NULL);
-  base::android::CheckException(env);
+
   for (size_t i = 0; i < file_count; ++i) {
-    file_ids[i] = files_to_register->GetIDAt(i);
-    file_fds[i] = files_to_register->GetFDAt(i);
-    PCHECK(0 <= file_fds[i]);
-    file_auto_close[i] = files_to_register->OwnsFD(file_fds[i]);
-    if (file_auto_close[i])
-      ignore_result(files_to_register->ReleaseFD(file_fds[i]).release());
+    int fd = files_to_register->GetFDAt(i);
+    PCHECK(0 <= fd);
+    int id = files_to_register->GetIDAt(i);
+    bool autoClose = files_to_register->OwnsFD(fd);

[Yaron, 2015/05/29 18:29:22]

s/autoClose/auto_close/

[agrieve, 2015/05/29 19:16:40]

Done.

+    int64 offset = 0L;
+    int64 size = 0L;
+    auto found_region_iter = regions.find(id);
+    if (found_region_iter != regions.end()) {
+      offset = found_region_iter->second.offset;
+      size = found_region_iter->second.size;
+    }
+    ScopedJavaLocalRef<jobject> j_file_info =
+        Java_ChildProcessLauncher_makeFdInfo(env, id, fd, autoClose, offset,
+                                             size);
+    PCHECK(j_file_info.obj());
+    env->SetObjectArrayElement(j_file_infos.obj(), i, j_file_info.obj());
+    if (autoClose) {
+      ignore_result(files_to_register->ReleaseFD(fd).release());
+    }
   }
-  env->ReleaseIntArrayElements(j_file_ids.obj(), file_ids, 0);
-  env->ReleaseIntArrayElements(j_file_fds.obj(), file_fds, 0);
-  env->ReleaseBooleanArrayElements(j_file_auto_close.obj(), file_auto_close, 0);
 
-  Java_ChildProcessLauncher_start(env,
-      base::android::GetApplicationContext(),
-      j_argv.obj(),
-      child_process_id,
-      j_file_ids.obj(),
-      j_file_fds.obj(),
-      j_file_auto_close.obj(),
+  Java_ChildProcessLauncher_start(
+      env, base::android::GetApplicationContext(), j_argv.obj(),
+      child_process_id, j_file_infos.obj(),
       reinterpret_cast<intptr_t>(new StartChildProcessCallback(callback)));
 }
 
 void StopChildProcess(base::ProcessHandle handle) {
   JNIEnv* env = AttachCurrentThread();
   DCHECK(env);
   Java_ChildProcessLauncher_stop(env, static_cast<jint>(handle));
 }
 
 bool IsChildProcessOomProtected(base::ProcessHandle handle) {
@@ skipping 67 matching lines @@
 jboolean IsSingleProcess(JNIEnv* env, jclass clazz) {
   return base::CommandLine::ForCurrentProcess()->HasSwitch(
       switches::kSingleProcess);
 }
 
 bool RegisterChildProcessLauncher(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }
 
 }  // namespace content