tree 9ce2c905d1cc70149db3f8f18df52f3c66ffde2d
parent 6b49edd1adea9511645417b276cb8f9bdf280534
author Jordan Liu <jminjie@google.com> 1581373935 -0800
committer Jordan Liu <jminjie@google.com> 1581539787 +0000

Only unbind if bind() returned success

ServiceConnection may become disconnected asynchonously for various
reasons so calling unbind() on a disconnected SC is actually handled
by the framework. What we don't want to do is call unbind() on a
service which we never successfully called bindService() on, so in this
CL we will track the client-side binding instead of the connection
status through callbacks. This is less racy and handles the root cause
of the exception.

(this is a CP of aosp/1231873)

Bug: 145661922
Test: manual
Change-Id: I72512f4da276ff5fa1bee2254b42fb938d3a0b20
Merged-In: I72512f4da276ff5fa1bee2254b42fb938d3a0b20