[Payment Request] unit tests for the FullCardRequester

[Payment Request] unit tests for the FullCardRequester

BUG=602666
Review-Url: https://codereview.chromium.org/2959133002
Cr-Commit-Position: refs/heads/master@{#483085}
Committed: https://chromium.googlesource.com/chromium/src/+/64aa81bc065af2bc2863486d40f21cc9dc81ea3e

[Moe, 2017-06-28 00:20:18]

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

[commit-bot: I haz the power, 2017-06-28 00:20:46]

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-06-28 00:23:58]

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

[commit-bot: I haz the power, 2017-06-28 00:23:59]

Dry run: Try jobs failed on following builders:
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Moe, 2017-06-28 02:37:31]

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

[commit-bot: I haz the power, 2017-06-28 02:37:49]

Dry run: CQ is trying da patch.

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

[Moe, 2017-06-28 02:38:29]

Patchset #1 (id:1) has been deleted

[Moe, 2017-06-28 03:13:04]

Patchset #1 (id:20001) has been deleted

[Moe, 2017-06-28 03:13:37]

mahmadi@chromium.org changed reviewers:
+ jif@chromium.org

[Moe, 2017-06-28 03:13:38]

Hi JF, Please take a look.

[jif, 2017-06-28 16:30:15]

lgtm

https://codereview.chromium.org/2959133002/diff/40001/ios/chrome/browser/ui/p...
File ios/chrome/browser/ui/payments/payment_request_coordinator.mm (right):

https://codereview.chromium.org/2959133002/diff/40001/ios/chrome/browser/ui/p...
ios/chrome/browser/ui/payments/payment_request_coordinator.mm:126: #pragma mark
- FullCardRequesterConsumer
Thanks for adding those, I find them very useful.

[Moe, 2017-06-28 17:03:06]

On 2017/06/28 at 16:30:15, jif wrote:
> lgtm
> 
>
https://codereview.chromium.org/2959133002/diff/40001/ios/chrome/browser/ui/p...
> File ios/chrome/browser/ui/payments/payment_request_coordinator.mm (right):
> 
>
https://codereview.chromium.org/2959133002/diff/40001/ios/chrome/browser/ui/p...
> ios/chrome/browser/ui/payments/payment_request_coordinator.mm:126: #pragma
mark - FullCardRequesterConsumer
> Thanks for adding those, I find them very useful.

Thank you!

[Moe, 2017-06-28 17:03:11]

The CQ bit was checked by mahmadi@chromium.org

[commit-bot: I haz the power, 2017-06-28 17:03:49]

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-06-28 18:57:22]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1498669391582180,
"parent_rev": "519c16766d8f596da9b4855e2b3bde7a23ea0a0d", "commit_rev":
"64aa81bc065af2bc2863486d40f21cc9dc81ea3e"}

[commit-bot: I haz the power, 2017-06-28 18:57:39]

Description was changed from

==========
[Payment Request] unit tests for the FullCardRequester

BUG=602666
==========

to

==========
[Payment Request] unit tests for the FullCardRequester

BUG=602666

Review-Url: https://codereview.chromium.org/2959133002
Cr-Commit-Position: refs/heads/master@{#483085}
Committed:
https://chromium.googlesource.com/chromium/src/+/64aa81bc065af2bc2863486d40f2...
==========

[commit-bot: I haz the power, 2017-06-28 18:57:43]

Committed patchset #1 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/64aa81bc065af2bc2863486d40f2...