+Hwi

Here is what I've done so far, but I feel it's not a great UI to just have the X dissapear when we've sent the info to the merchant.

BTW I have added a 2seconds delay to both
- processing the info on our side (still cancelable)
- waiting for the merchant response (not cancelable)
so the change is more visible. For cards, the first part would probably not be visible at all but I think eventually Payment apps might take longer.

Deleted: before.mp4 2.4 MB

	before.mp4 2.4 MB View Download

Deleted: after.mp4 2.3 MB

	after.mp4 2.3 MB View Download

The "X" should disappear the moment you see "Loading" with a spinner. I understand that the data is not sent to the merchant immediately when "Loading" shows up, but that's an  implementation detail.

Oh much easier then. Thanks.

Here is the new UI, both in real speed and with a delay.

Deleted: real_speed.mp4 2.4 MB

	real_speed.mp4 2.4 MB View Download

Deleted: with_delay.mp4 1.9 MB

	with_delay.mp4 1.9 MB View Download

👍

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/01f556eda522112d177831dba13af274c2af4be2

commit 01f556eda522112d177831dba13af274c2af4be2
Author: sebsg <sebsg@chromium.org>
Date: Thu Oct 27 18:08:38 2016

[Payments] Remove the X in payments request processing UI.

BUG= 660028 

Review-Url: https://codereview.chromium.org/2461443002
Cr-Commit-Position: refs/heads/master@{#428078}

[modify] https://crrev.com/01f556eda522112d177831dba13af274c2af4be2/chrome/android/java/src/org/chromium/chrome/browser/payments/ui/PaymentRequestUI.java

Works as per expected behavior,"X" disappears the moment when we see "Loading" with a spinner .
Issue tested on Latest M56 Build.