Add test to check that the payment response details field is a JSON-serializable object.

Add test to check that the payment response details field is a JSON-serializable object.

BUG=587995
Committed: https://crrev.com/c0452d6b6cf1ba95c7449f84df149edfc639f2b7
Cr-Commit-Position: refs/heads/master@{#407415}

[pals, 2016-07-22 09:04:12]

The CQ bit was checked by sanjoy.pal@samsung.com to run a CQ dry run

[pals, 2016-07-22 09:04:28]

sanjoy.pal@samsung.com changed reviewers:
+ rouslan@chromium.org

[pals, 2016-07-22 09:04:28]

PTAL.

[commit-bot: I haz the power, 2016-07-22 09:04:34]

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, 2016-07-22 10:05:08]

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

[commit-bot: I haz the power, 2016-07-22 10:05:09]

Dry run: This issue passed the CQ dry run.

[please use gerrit instead, 2016-07-22 16:58:55]

lgtm % nit

https://codereview.chromium.org/2171973005/diff/1/third_party/WebKit/Source/m...
File third_party/WebKit/Source/modules/payments/PaymentResponseTest.cpp (right):

https://codereview.chromium.org/2171973005/diff/1/third_party/WebKit/Source/m...
third_party/WebKit/Source/modules/payments/PaymentResponseTest.cpp:78: 
Remove this empty line. A typical test has three sections:

1) setup
2) action under test
3) verification of results

Lines 74-79 are all for setup, so there's no need to place an empty line in this
block.

[pals, 2016-07-25 03:35:15]

The CQ bit was checked by sanjoy.pal@samsung.com to run a CQ dry run

[commit-bot: I haz the power, 2016-07-25 03:35:27]

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, 2016-07-25 04:42:10]

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

[commit-bot: I haz the power, 2016-07-25 04:42:12]

Dry run: This issue passed the CQ dry run.

[pals, 2016-07-25 04:47:23]

The CQ bit was checked by sanjoy.pal@samsung.com

[pals, 2016-07-25 04:47:24]

The patchset sent to the CQ was uploaded after l-g-t-m from rouslan@chromium.org
Link to the patchset: https://codereview.chromium.org/2171973005/#ps20001
(title: "Fixed comments")

[commit-bot: I haz the power, 2016-07-25 04:47:36]

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, 2016-07-25 04:51:01]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-07-25 04:52:28]

Description was changed from

==========
Add test to check that the payment response details field is a JSON-serializable
object.

BUG=587995
==========

to

==========
Add test to check that the payment response details field is a JSON-serializable
object.

BUG=587995

Committed: https://crrev.com/c0452d6b6cf1ba95c7449f84df149edfc639f2b7
Cr-Commit-Position: refs/heads/master@{#407415}
==========

[commit-bot: I haz the power, 2016-07-25 04:52:29]

Patchset 2 (id:??) landed as
https://crrev.com/c0452d6b6cf1ba95c7449f84df149edfc639f2b7
Cr-Commit-Position: refs/heads/master@{#407415}