Disconnect from the READY_TO_PAY service after query.

Disconnect from the READY_TO_PAY service after query.

BUG=700923
Review-Url: https://codereview.chromium.org/2746163002
Cr-Commit-Position: refs/heads/master@{#457580}
Committed: https://chromium.googlesource.com/chromium/src/+/6900db6107a70271dad8c6623464301dc1dbd5ed

[please use gerrit instead, 2017-03-13 15:07:40]

rouslan@chromium.org changed reviewers:
+ gogerald@chromium.org

[please use gerrit instead, 2017-03-13 15:07:40]

Ganggui, ptal.

[gogerald1, 2017-03-13 16:05:27]

https://codereview.chromium.org/2746163002/diff/20001/chrome/android/java/src...
File
chrome/android/java/src/org/chromium/chrome/browser/payments/AndroidPaymentApp.java
(right):

https://codereview.chromium.org/2746163002/diff/20001/chrome/android/java/src...
chrome/android/java/src/org/chromium/chrome/browser/payments/AndroidPaymentApp.java:161:
ContextUtils.getApplicationContext().bindService(
bindService returns a boolean to indicate whether the service is successfully
bound or not, might use it together with exceptions to decide
respondToGetInstrumentsQuery(null) or not,

https://codereview.chromium.org/2746163002/diff/20001/chrome/android/java/src...
chrome/android/java/src/org/chromium/chrome/browser/payments/AndroidPaymentApp.java:200:
mIsReadyToPayQueried = true;
Might return here if mInstrumentsCallback == null since it's not needed anymore

[please use gerrit instead, 2017-03-15 14:00:05]

https://codereview.chromium.org/2746163002/diff/20001/chrome/android/java/src...
File
chrome/android/java/src/org/chromium/chrome/browser/payments/AndroidPaymentApp.java
(right):

https://codereview.chromium.org/2746163002/diff/20001/chrome/android/java/src...
chrome/android/java/src/org/chromium/chrome/browser/payments/AndroidPaymentApp.java:161:
ContextUtils.getApplicationContext().bindService(
On 2017/03/13 16:05:27, gogerald1 wrote:
> bindService returns a boolean to indicate whether the service is successfully
> bound or not, might use it together with exceptions to decide
> respondToGetInstrumentsQuery(null) or not,
> 

Done.

https://codereview.chromium.org/2746163002/diff/20001/chrome/android/java/src...
chrome/android/java/src/org/chromium/chrome/browser/payments/AndroidPaymentApp.java:200:
mIsReadyToPayQueried = true;
On 2017/03/13 16:05:27, gogerald1 wrote:
> Might return here if mInstrumentsCallback == null since it's not needed
anymore

Done.

[please use gerrit instead, 2017-03-15 14:11:26]

Ganggui, ptal patch 3.

[gogerald1, 2017-03-15 14:13:11]

lgtm

[please use gerrit instead, 2017-03-16 20:51:37]

The CQ bit was checked by rouslan@chromium.org

[please use gerrit instead, 2017-03-16 20:51:37]

The patchset sent to the CQ was uploaded after l-g-t-m from
gogerald@chromium.org
Link to the patchset: https://codereview.chromium.org/2746163002/#ps60001
(title: "Rebase")

[commit-bot: I haz the power, 2017-03-16 20:52:34]

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-03-16 22:15:02]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1489697497507360,
"parent_rev": "ffd34d347dc1889ae692610c4d35cc2fc97ca9c0", "commit_rev":
"6900db6107a70271dad8c6623464301dc1dbd5ed"}

[commit-bot: I haz the power, 2017-03-16 22:15:49]

Description was changed from

==========
Disconnect from the READY_TO_PAY service after query.

BUG=700923
==========

to

==========
Disconnect from the READY_TO_PAY service after query.

BUG=700923

Review-Url: https://codereview.chromium.org/2746163002
Cr-Commit-Position: refs/heads/master@{#457580}
Committed:
https://chromium.googlesource.com/chromium/src/+/6900db6107a70271dad8c6623464...
==========

[commit-bot: I haz the power, 2017-03-16 22:15:50]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/chromium/src/+/6900db6107a70271dad8c6623464...