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

Unified Diff: content/public/android/java/src/org/chromium/content/browser/ChildProcessLauncherHelper.java

Issue 2774163002: android: Java ChildProcessLauncherHelper instance (Closed)
Patch Set: explicit AddRef Created 3 years, 9 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/public/android/java/src/org/chromium/content/browser/ChildProcessLauncherHelper.java
diff --git a/content/public/android/java/src/org/chromium/content/browser/ChildProcessLauncherHelper.java b/content/public/android/java/src/org/chromium/content/browser/ChildProcessLauncherHelper.java
new file mode 100644
index 0000000000000000000000000000000000000000..588589f8c055566bc05d50a0148839a8cf42a3e6
--- /dev/null
+++ b/content/public/android/java/src/org/chromium/content/browser/ChildProcessLauncherHelper.java
@@ -0,0 +1,96 @@
+// 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.
+
+package org.chromium.content.browser;
+
+import android.content.Context;
+import android.os.ParcelFileDescriptor;
+
+import org.chromium.base.Log;
+import org.chromium.base.annotations.CalledByNative;
+import org.chromium.base.annotations.JNINamespace;
+import org.chromium.base.process_launcher.FileDescriptorInfo;
+
+import java.io.IOException;
+
+/**
+ * This is the java counterpart to ChildProcessLauncherHelper. It is owned by native side and
+ * has an explicit destroy method.
+ * Each public or jni methods should have explicit documentation on what threads they are called.
+ */
+@JNINamespace("content::internal")
+class ChildProcessLauncherHelper {
+ private static final String TAG = "ChildProcLH";
+
+ // Note native pointer is only guaranteed live until nativeOnChildProcessStarted.
+ private long mNativeChildProcessLauncherHelper;
+ private int mPid;
+
+ // Called on launcher thread.
+ @CalledByNative
+ private static FileDescriptorInfo makeFdInfo(
+ int id, int fd, boolean autoClose, long offset, long size) {
+ ParcelFileDescriptor pFd;
+ if (autoClose) {
+ // Adopt the FD, it will be closed when we close the ParcelFileDescriptor.
+ pFd = ParcelFileDescriptor.adoptFd(fd);
+ } else {
+ try {
+ pFd = ParcelFileDescriptor.fromFd(fd);
+ } catch (IOException e) {
+ Log.e(TAG, "Invalid FD provided for process connection, aborting connection.", e);
+ return null;
+ }
+ }
+ return new FileDescriptorInfo(id, pFd, offset, size);
+ }
+
+ // Called on launcher thread.
+ @CalledByNative
+ private static ChildProcessLauncherHelper create(long nativePointer, Context context,
+ int paramId, final String[] commandLine, int childProcessId,
+ FileDescriptorInfo[] filesToBeMapped) {
+ return new ChildProcessLauncherHelper(
+ nativePointer, context, paramId, commandLine, childProcessId, filesToBeMapped);
+ }
+
+ private ChildProcessLauncherHelper(long nativePointer, Context context, int paramId,
+ final String[] commandLine, int childProcessId, FileDescriptorInfo[] filesToBeMapped) {
+ mNativeChildProcessLauncherHelper = nativePointer;
+
+ ChildProcessLauncher.start(context, paramId, commandLine, childProcessId, filesToBeMapped,
+ new ChildProcessLauncher.LaunchCallback() {
+ @Override
+ public void onChildProcessStarted(int pid) {
+ mPid = pid;
+ if (mNativeChildProcessLauncherHelper != 0) {
+ nativeOnChildProcessStarted(mNativeChildProcessLauncherHelper, pid);
+ }
+ mNativeChildProcessLauncherHelper = 0;
+ }
+ });
+ }
+
+ // Called on client (UI or IO) thread.
+ @CalledByNative
+ private boolean isOomProtected() {
+ return ChildProcessLauncher.getBindingManager().isOomProtected(mPid);
+ }
+
+ // Called on launcher thread.
+ @CalledByNative
+ private void setInForeground(int pid, boolean inForeground) {
+ assert mPid == pid;
+ ChildProcessLauncher.getBindingManager().setInForeground(mPid, inForeground);
+ }
+
+ @CalledByNative
+ private static void stop(int pid) {
+ ChildProcessLauncher.stop(pid);
+ }
+
+ // Can be called on a number of threads, including launcher, and binder.
+ private static native void nativeOnChildProcessStarted(
+ long nativeChildProcessLauncherHelper, int pid);
+}

Powered by Google App Engine
This is Rietveld 408576698