[Web Payments] Show a spinner after unmasking a card.

[Web Payments] Show a spinner after unmasking a card.

BUG=706511
Review-Url: https://codereview.chromium.org/2806533002
Cr-Commit-Position: refs/heads/master@{#462624}
Committed: https://chromium.googlesource.com/chromium/src/+/7e49f5ac85d0ac4d401349f39f07987475308e36

[anthonyvd, 2017-04-06 17:19:31]

The CQ bit was checked by anthonyvd@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-04-06 17:19:56]

Dry run: 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-06 18:14:11]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-04-06 18:14:12]

Dry run: This issue passed the CQ dry run.

[anthonyvd, 2017-04-06 18:37:02]

anthonyvd@chromium.org changed reviewers:
+ mathp@chromium.org

[anthonyvd, 2017-04-06 18:37:03]

Hi Math, PTAL!

[Mathieu, 2017-04-06 19:06:23]

lgtm!

https://codereview.chromium.org/2806533002/diff/20001/chrome/browser/ui/views...
File chrome/browser/ui/views/payments/payment_request_dialog_view.h (right):

https://codereview.chromium.org/2806533002/diff/20001/chrome/browser/ui/views...
chrome/browser/ui/views/payments/payment_request_dialog_view.h:125: // A full
dialog overlay that shows a spinner and the "processing" label. It's
nit: We could have this sort of documentation on ShowProcessingSpinner as well.

[anthonyvd, 2017-04-06 20:15:08]

https://codereview.chromium.org/2806533002/diff/20001/chrome/browser/ui/views...
File chrome/browser/ui/views/payments/payment_request_dialog_view.h (right):

https://codereview.chromium.org/2806533002/diff/20001/chrome/browser/ui/views...
chrome/browser/ui/views/payments/payment_request_dialog_view.h:125: // A full
dialog overlay that shows a spinner and the "processing" label. It's
On 2017/04/06 at 19:06:23, Mathieu wrote:
> nit: We could have this sort of documentation on ShowProcessingSpinner as
well.

Done.

[anthonyvd, 2017-04-06 20:15:11]

The CQ bit was checked by anthonyvd@chromium.org

[anthonyvd, 2017-04-06 20:15:12]

The patchset sent to the CQ was uploaded after l-g-t-m from mathp@chromium.org
Link to the patchset: https://codereview.chromium.org/2806533002/#ps40001
(title: "Add comment.")

[commit-bot: I haz the power, 2017-04-06 20:17:04]

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-06 21:19:58]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1491509711891730,
"parent_rev": "2fe329196d1596b0e516fda0727e8ebce5996152", "commit_rev":
"7e49f5ac85d0ac4d401349f39f07987475308e36"}

[commit-bot: I haz the power, 2017-04-06 21:21:05]

Description was changed from

==========
[Web Payments] Show a spinner after unmasking a card.

BUG=706511
==========

to

==========
[Web Payments] Show a spinner after unmasking a card.

BUG=706511

Review-Url: https://codereview.chromium.org/2806533002
Cr-Commit-Position: refs/heads/master@{#462624}
Committed:
https://chromium.googlesource.com/chromium/src/+/7e49f5ac85d0ac4d401349f39f07...
==========

[commit-bot: I haz the power, 2017-04-06 21:21:06]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/7e49f5ac85d0ac4d401349f39f07...