Revert of android: Async setupConnection binder call

Revert of android: Async setupConnection binder call (patchset #4 id:60001 of https://codereview.chromium.org/2810433002/ )

Reason for revert:
Appears to have broken ChildProcessLauncherTest_testBindServiceFromMultipleProcesses/0 on https://build.chromium.org/p/chromium.linux/builders/Android%20Tests%20%28dbg%29/builds/41605

junit.framework.AssertionFailedError
	at org.chromium.content.browser.ChildProcessLauncherTest.testBindServiceFromMultipleProcesses(ChildProcessLauncherTest.java:448)
	at java.lang.reflect.Method.invokeNative(Native Method)
	at android.test.InstrumentationTestCase.runMethod(InstrumentationTestCase.java:214)
	at android.test.InstrumentationTestCase.runTest(InstrumentationTestCase.java:199)
	at org.chromium.base.test.BaseTestResult.run(BaseTestResult.java:129)
	at android.test.AndroidTestRunner.runTest(AndroidTestRunner.java:191)
	at android.test.AndroidTestRunner.runTest(AndroidTestRunner.java:176)
	at android.test.InstrumentationTestRunner.onStart(InstrumentationTestRunner.java:554)
	at android.app.Instrumentation$InstrumentationThread.run(Instrumentation.java:1701)

Original issue's description:
> android: Async setupConnection binder call
> 
> Make setupConnection oneway/async. Add a ICallbackInt interface to
> return the PID of the child process. The reply is then posted back to
> launcher thread. Connection setup is already mostly async so only
> involves moving related code to the reply callback.
> 
> One caveat is that getCallingPid no longer works and just returns 0
> for oneway calls. The pid check is not strictly necessary, so dropped
> the from setupConnection instead. 
> 
> BUG=689758, 690118
> 
> Review-Url: https://codereview.chromium.org/2810433002
> Cr-Commit-Position: refs/heads/master@{#463332}
> Committed: https://chromium.googlesource.com/chromium/src/+/39c7bd71dc2f9c9f692dd346e4e4e525af25ede1

TBR=rsesek@chromium.org,jcivelli@chromium.org,torne@chromium.org,boliu@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=689758, 690118
Review-Url: https://codereview.chromium.org/2806343003
Cr-Commit-Position: refs/heads/master@{#463387}
Committed: https://chromium.googlesource.com/chromium/src/+/2cc9e495637d506c65ae9802408043f7fbaf27d2

[please use gerrit instead, 2017-04-10 20:51:15]

The CQ bit was checked by rouslan@chromium.org

[please use gerrit instead, 2017-04-10 20:51:16]

Created Revert of android: Async setupConnection binder call

[commit-bot: I haz the power, 2017-04-10 20:51:53]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-10 20:52:38]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1491857475739670, "parent_rev":
"b87f01f54ba47ace1ddf35476a9166ef67341ea8", "commit_rev":
"2cc9e495637d506c65ae9802408043f7fbaf27d2"}

[commit-bot: I haz the power, 2017-04-10 20:53:32]

Description was changed from

==========
Revert of android: Async setupConnection binder call (patchset #4 id:60001 of
https://codereview.chromium.org/2810433002/ )

Reason for revert:
Appears to have broken
ChildProcessLauncherTest_testBindServiceFromMultipleProcesses/0 on
https://build.chromium.org/p/chromium.linux/builders/Android%20Tests%20%28dbg...

junit.framework.AssertionFailedError
	at
org.chromium.content.browser.ChildProcessLauncherTest.testBindServiceFromMultipleProcesses(ChildProcessLauncherTest.java:448)
	at java.lang.reflect.Method.invokeNative(Native Method)
	at
android.test.InstrumentationTestCase.runMethod(InstrumentationTestCase.java:214)
	at
android.test.InstrumentationTestCase.runTest(InstrumentationTestCase.java:199)
	at org.chromium.base.test.BaseTestResult.run(BaseTestResult.java:129)
	at android.test.AndroidTestRunner.runTest(AndroidTestRunner.java:191)
	at android.test.AndroidTestRunner.runTest(AndroidTestRunner.java:176)
	at
android.test.InstrumentationTestRunner.onStart(InstrumentationTestRunner.java:554)
	at
android.app.Instrumentation$InstrumentationThread.run(Instrumentation.java:1701)

Original issue's description:
> android: Async setupConnection binder call
> 
> Make setupConnection oneway/async. Add a ICallbackInt interface to
> return the PID of the child process. The reply is then posted back to
> launcher thread. Connection setup is already mostly async so only
> involves moving related code to the reply callback.
> 
> One caveat is that getCallingPid no longer works and just returns 0
> for oneway calls. The pid check is not strictly necessary, so dropped
> the from setupConnection instead. 
> 
> BUG=689758, 690118
> 
> Review-Url: https://codereview.chromium.org/2810433002
> Cr-Commit-Position: refs/heads/master@{#463332}
> Committed:
https://chromium.googlesource.com/chromium/src/+/39c7bd71dc2f9c9f692dd346e4e4...

TBR=rsesek@chromium.org,jcivelli@chromium.org,torne@chromium.org,boliu@chromi...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=689758, 690118
==========

to

==========
Revert of android: Async setupConnection binder call (patchset #4 id:60001 of
https://codereview.chromium.org/2810433002/ )

Reason for revert:
Appears to have broken
ChildProcessLauncherTest_testBindServiceFromMultipleProcesses/0 on
https://build.chromium.org/p/chromium.linux/builders/Android%20Tests%20%28dbg...

junit.framework.AssertionFailedError
	at
org.chromium.content.browser.ChildProcessLauncherTest.testBindServiceFromMultipleProcesses(ChildProcessLauncherTest.java:448)
	at java.lang.reflect.Method.invokeNative(Native Method)
	at
android.test.InstrumentationTestCase.runMethod(InstrumentationTestCase.java:214)
	at
android.test.InstrumentationTestCase.runTest(InstrumentationTestCase.java:199)
	at org.chromium.base.test.BaseTestResult.run(BaseTestResult.java:129)
	at android.test.AndroidTestRunner.runTest(AndroidTestRunner.java:191)
	at android.test.AndroidTestRunner.runTest(AndroidTestRunner.java:176)
	at
android.test.InstrumentationTestRunner.onStart(InstrumentationTestRunner.java:554)
	at
android.app.Instrumentation$InstrumentationThread.run(Instrumentation.java:1701)

Original issue's description:
> android: Async setupConnection binder call
> 
> Make setupConnection oneway/async. Add a ICallbackInt interface to
> return the PID of the child process. The reply is then posted back to
> launcher thread. Connection setup is already mostly async so only
> involves moving related code to the reply callback.
> 
> One caveat is that getCallingPid no longer works and just returns 0
> for oneway calls. The pid check is not strictly necessary, so dropped
> the from setupConnection instead. 
> 
> BUG=689758, 690118
> 
> Review-Url: https://codereview.chromium.org/2810433002
> Cr-Commit-Position: refs/heads/master@{#463332}
> Committed:
https://chromium.googlesource.com/chromium/src/+/39c7bd71dc2f9c9f692dd346e4e4...

TBR=rsesek@chromium.org,jcivelli@chromium.org,torne@chromium.org,boliu@chromi...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=689758, 690118

Review-Url: https://codereview.chromium.org/2806343003
Cr-Commit-Position: refs/heads/master@{#463387}
Committed:
https://chromium.googlesource.com/chromium/src/+/2cc9e495637d506c65ae98024080...
==========

[commit-bot: I haz the power, 2017-04-10 20:53:33]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/2cc9e495637d506c65ae98024080...

[boliu, 2017-04-10 21:02:31]

On 2017/04/10 20:53:33, commit-bot: I haz the power wrote:
> Committed patchset #1 (id:1) as
>
https://chromium.googlesource.com/chromium/src/+/2cc9e495637d506c65ae98024080...

Ahh, already wrote fix waiting for review. I'll just re-open and reland then:
https://codereview.chromium.org/2810933002/