Multiprocess test client: Android child process launcher rework.

base/test/android/java/src/org/chromium/base/ITestClient.aidl

Index: base/test/android/java/src/org/chromium/base/ITestClient.aidl
diff --git a/base/test/android/java/src/org/chromium/base/ITestClient.aidl b/base/test/android/java/src/org/chromium/base/ITestClient.aidl
new file mode 100644
index 0000000000000000000000000000000000000000..5132472c50c9788da030033ee190556f935b5be0
--- /dev/null
+++ b/base/test/android/java/src/org/chromium/base/ITestClient.aidl
@@ -0,0 +1,18 @@
+// Copyright 2016 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.base;
+
+import org.chromium.base.FileDescriptorInfo;
+import org.chromium.base.MainReturnCodeResult;
+
+/**
+ * This interface is used to control child processes.
+ */
+interface ITestClient {
+  int launch(in String[] args, in FileDescriptorInfo[] fdsToMap);

[nyquist, 2016/12/13 20:40:30]

Could you explain what the return value is here? I thought at first it was some
error-code, but then it turns out it's the PID.

I guess at that point it might look weird not to document the other methods as
well.

[Jay Civelli, 2016/12/14 02:20:10]

Sorry, I had forgotten to comment that file entirely. Should be better now.

+  MainReturnCodeResult waitForMainToReturn(int timeoutMs);
+  boolean forceStopSynchronous(int exitCode);

[nyquist, 2016/12/13 20:40:30]

What does the return value signify here? It seems like we're always returning
true. Will this change going forward?

[Jay Civelli, 2016/12/14 02:20:10]

I explained in the comment.

+  void forceStop(int exitCode);
+}