PaymentHandler: Add a js test for PaymentInstruments get/set methods

PaymentHandler: Add a js test for PaymentInstruments get/set methods

Related Spec Link:
  https://w3c.github.io/webpayments-payment-apps-api/#payment-instruments

See the other CLs in this series:
  [1/3] http://crrev.com/2790013002 (blink side)
  [2/3] http://crrev.com/2806133002 (content side and unittests)
  [3/3] http://crrev.com/2811163002 (this patch)

BUG=661608
Review-Url: https://codereview.chromium.org/2811163002
Cr-Commit-Position: refs/heads/master@{#464953}
Committed: https://chromium.googlesource.com/chromium/src/+/bf32bc709c97515485b41f48c783dab5c5d0d30f

[zino, 2017-04-11 19:11:05]

Description was changed from

==========
PaymentHandler: Add a js test for PaymentInstruments get/set methods

Related Spec Link:
  https://w3c.github.io/webpayments-payment-apps-api/#payment-instruments

See the other CLs in this series:
  [1/3] http://crrev.com/2790013002 (blink side)
  [2/3] http://crrev.com/2806133002 (content side and unittests)
  [3/3] http://crrev.com/? (this patch)

BUG=661608
==========

to

==========
PaymentHandler: Add a js test for PaymentInstruments get/set methods

Related Spec Link:
  https://w3c.github.io/webpayments-payment-apps-api/#payment-instruments

See the other CLs in this series:
  [1/3] http://crrev.com/2790013002 (blink side)
  [2/3] http://crrev.com/2806133002 (content side and unittests)
  [3/3] http://crrev.com/2811163002 (this patch)

BUG=661608
==========

[zino, 2017-04-11 19:22:30]

jinho.bang@samsung.com changed reviewers:
+ rouslan@chromium.org

[zino, 2017-04-11 19:22:41]

jinho.bang@samsung.com changed reviewers:
+ tsepez@chromium.org

[zino, 2017-04-11 19:23:40]

PTAL

[please use gerrit instead, 2017-04-11 21:09:38]

lgtm % nit

https://codereview.chromium.org/2811163002/diff/20001/third_party/WebKit/Layo...
File third_party/WebKit/LayoutTests/http/tests/payments/payment-instruments.html
(right):

https://codereview.chromium.org/2811163002/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/http/tests/payments/payment-instruments.html:37:
.then (stored_instrument => {
No space between "then" and "(" to be consistent with the rest of the file.

[Tom Sepez, 2017-04-11 21:52:42]

lgtm

[zino, 2017-04-12 16:11:40]

Thank you for review.

I'll wait for other patches to be reviewed.

https://codereview.chromium.org/2811163002/diff/20001/third_party/WebKit/Layo...
File third_party/WebKit/LayoutTests/http/tests/payments/payment-instruments.html
(right):

https://codereview.chromium.org/2811163002/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/http/tests/payments/payment-instruments.html:37:
.then (stored_instrument => {
On 2017/04/11 21:09:38, ಠ_ಠ wrote:
> No space between "then" and "(" to be consistent with the rest of the file.

Done.

[zino, 2017-04-17 17:05:35]

The CQ bit was checked by jinho.bang@samsung.com

[zino, 2017-04-17 17:05:35]

The patchset sent to the CQ was uploaded after l-g-t-m from tsepez@chromium.org,
rouslan@chromium.org
Link to the patchset: https://codereview.chromium.org/2811163002/#ps60001
(title: "rebased")

[commit-bot: I haz the power, 2017-04-17 17:05:58]

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-17 18:24:12]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1492448735443350,
"parent_rev": "ebbffbf87e2017a9917c86b3a5066b3aacfafe98", "commit_rev":
"bf32bc709c97515485b41f48c783dab5c5d0d30f"}

[commit-bot: I haz the power, 2017-04-17 18:24:23]

Description was changed from

==========
PaymentHandler: Add a js test for PaymentInstruments get/set methods

Related Spec Link:
  https://w3c.github.io/webpayments-payment-apps-api/#payment-instruments

See the other CLs in this series:
  [1/3] http://crrev.com/2790013002 (blink side)
  [2/3] http://crrev.com/2806133002 (content side and unittests)
  [3/3] http://crrev.com/2811163002 (this patch)

BUG=661608
==========

to

==========
PaymentHandler: Add a js test for PaymentInstruments get/set methods

Related Spec Link:
  https://w3c.github.io/webpayments-payment-apps-api/#payment-instruments

See the other CLs in this series:
  [1/3] http://crrev.com/2790013002 (blink side)
  [2/3] http://crrev.com/2806133002 (content side and unittests)
  [3/3] http://crrev.com/2811163002 (this patch)

BUG=661608

Review-Url: https://codereview.chromium.org/2811163002
Cr-Commit-Position: refs/heads/master@{#464953}
Committed:
https://chromium.googlesource.com/chromium/src/+/bf32bc709c97515485b41f48c783...
==========

[commit-bot: I haz the power, 2017-04-17 18:24:23]

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