android: Mark two binder calls oneway

content/public/android/javatests/src/org/chromium/content/browser/ChildProcessLauncherTest.java

Index: content/public/android/javatests/src/org/chromium/content/browser/ChildProcessLauncherTest.java
diff --git a/content/public/android/javatests/src/org/chromium/content/browser/ChildProcessLauncherTest.java b/content/public/android/javatests/src/org/chromium/content/browser/ChildProcessLauncherTest.java
index 4de2bfa95f750c1c844af50404aa56c39a59090c..7e08d1ea0d8179ed27747d12859b198cd54960c9 100644
--- a/content/public/android/javatests/src/org/chromium/content/browser/ChildProcessLauncherTest.java
+++ b/content/public/android/javatests/src/org/chromium/content/browser/ChildProcessLauncherTest.java
@@ -103,7 +103,7 @@ public class ChildProcessLauncherTest extends InstrumentationTestCase {
         assertEquals(0, ChildProcessLauncher.connectedServicesCountForTesting());
 
         // Crash the service.
-        assertTrue(connection.crashServiceForTesting());
+        connection.crashServiceForTesting();

[palmer, 2017/04/07 18:18:51]

So it doesn't matter if we verify the crash really happened? Or is it the case
that DeadObjectException wasn't really a good test of that anyway? (I don't know
either way, just wondering.)

[boliu, 2017/04/07 18:29:33]

DeadObjectException wasn't a good test. Yes it indicates the other side is gone,
but we don't do anything with it. We listen to asynchronous
onServiceDisconnected to respond to child process going away (crash or normal
shutdown). Test already has to wait for onServiceDisconnected to happen.

 
         // Verify that the connection gets cleaned-up.
         CriteriaHelper.pollInstrumentationThread(Criteria.equals(0, new Callable<Integer>() {
@@ -153,7 +153,7 @@ public class ChildProcessLauncherTest extends InstrumentationTestCase {
                 });
 
         // Crash the service.
-        assertTrue(connection.crashServiceForTesting());
+        connection.crashServiceForTesting();
 
         // Verify that the connection gets cleaned-up.
         CriteriaHelper.pollInstrumentationThread(Criteria.equals(0, new Callable<Integer>() {
@@ -217,7 +217,7 @@ public class ChildProcessLauncherTest extends InstrumentationTestCase {
                 });
 
         // Crash the service.
-        assertTrue(connection.crashServiceForTesting());
+        connection.crashServiceForTesting();
 
         // Verify that a new service is started for the pending spawn.
         CriteriaHelper.pollInstrumentationThread(Criteria.equals(0, new Callable<Integer>() {