Add totalAmount to PaymentResponse.

Add totalAmount to PaymentResponse.

This patch adds the totalAmount to PaymentResponse. This reflects the
total amount that the user has confirmed they will pay. The change that
introduced totalAmount also introduced PaymentMethodModifier, which is
not in this patch.

https://w3c.github.io/browser-payment-api/#paymentresponse-interface
https://github.com/w3c/browser-payment-api/commit/67f4f25f4a0562e7b13c5139e1bb8f8b1d41f53a

BUG=618860
Committed: https://crrev.com/65f870288f6c830606d8eb9b0510cd0f156724b8
Cr-Commit-Position: refs/heads/master@{#399644}

[please use gerrit instead, 2016-06-11 00:42:11]

Patchset #1 (id:1) has been deleted

[please use gerrit instead, 2016-06-11 00:43:55]

rouslan@chromium.org changed reviewers:
+ cpalmer@chromium.org, dfalcantara@chromium.org, haraken@chromium.org,
mek@chromium.org

[please use gerrit instead, 2016-06-11 00:43:56]

(mek@ is ooo ? haraken@ : mek@), ptal WebKit.

cpalmer@, ptal payment_request.mojom.

dfalcantara@, ptal Java.

[please use gerrit instead, 2016-06-11 00:44:41]

rouslan@chromium.org changed reviewers:
+ palmer@chromium.org
- cpalmer@chromium.org

[please use gerrit instead, 2016-06-11 00:44:41]

-cpalmer@
+palmer@

[palmer, 2016-06-11 01:01:17]

lgtm

[haraken, 2016-06-11 05:26:49]

WebKit LGTM (but I can just review in terms of implementation.)

[please use gerrit instead, 2016-06-11 16:37:43]

On 2016/06/11 05:26:49, haraken wrote:
> (but I can just review in terms of implementation.)

Thank you, I think that's sufficient to land, because this patch is for a spec
change.

[gone, 2016-06-13 16:43:28]

java lgtm

[please use gerrit instead, 2016-06-13 16:56:03]

The CQ bit was checked by rouslan@chromium.org

[commit-bot: I haz the power, 2016-06-13 16:56:30]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2062583002/20001

[commit-bot: I haz the power, 2016-06-13 20:17:23]

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

[commit-bot: I haz the power, 2016-06-13 20:17:25]

Try jobs failed on following builders:
  win_chromium_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[please use gerrit instead, 2016-06-13 20:41:37]

The CQ bit was checked by rouslan@chromium.org

[commit-bot: I haz the power, 2016-06-13 20:42:16]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2062583002/20001

[commit-bot: I haz the power, 2016-06-13 22:45:20]

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

[commit-bot: I haz the power, 2016-06-13 22:45:22]

Failed to apply patch for
third_party/WebKit/public/platform/modules/payments/payment_request.mojom:
While running git apply --index -3 -p1;
  error: patch failed:
third_party/WebKit/public/platform/modules/payments/payment_request.mojom:57
  Falling back to three-way merge...
  Applied patch to
'third_party/WebKit/public/platform/modules/payments/payment_request.mojom' with
conflicts.
  U third_party/WebKit/public/platform/modules/payments/payment_request.mojom

Patch:      
third_party/WebKit/public/platform/modules/payments/payment_request.mojom
Index: third_party/WebKit/public/platform/modules/payments/payment_request.mojom
diff --git
a/third_party/WebKit/public/platform/modules/payments/payment_request.mojom
b/third_party/WebKit/public/platform/modules/payments/payment_request.mojom
index
1cbced4beb598794ccd26bf4e17c10047bcff868..0bbc5e4c03127f93f31ef1035a1c88a2a8dbefda
100644
--- a/third_party/WebKit/public/platform/modules/payments/payment_request.mojom
+++ b/third_party/WebKit/public/platform/modules/payments/payment_request.mojom
@@ -32,8 +32,21 @@ struct PaymentAddress {
   string phone;
 };
 
+// The currency amount that the renderer provides to the browser process. The
+// browser shows the amount in UI and forwards it on to the payment app, if it
+// requires the amount.
+struct CurrencyAmount {
+  // ISO 4217 currency code. Three upper case ASCII letters.
+  string currency_code;
+
+  // ISO 20022 CurrencyAnd30Amount. Up to 30 total digits. Up to 10 fraction
+  // digits. Separated by a dot.
+  string value;
+};
+
 struct PaymentResponse {
   string method_name;
+  CurrencyAmount total_amount;
 
   // Payment method specific JSON string that is built either by the browser or
   // a payment app, for example Android Pay. Browser ensures that the string
can
@@ -57,18 +70,6 @@ interface PaymentRequestClient {
   OnComplete();
 };
 
-// The currency amount that the renderer provides to the browser process. The
-// browser shows the amount in UI and forwards it on to the payment app, if it
-// requires the amount.
-struct CurrencyAmount {
-  // ISO 4217 currency code. Three upper case ASCII letters.
-  string currency_code;
-
-  // ISO 20022 CurrencyAnd30Amount. Up to 30 total digits. Up to 10 fraction
-  // digits. Separated by a dot.
-  string value;
-};
-
 struct PaymentItem {
   string label;
   CurrencyAmount amount;

[commit-bot: I haz the power, 2016-06-13 22:45:26]

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

[commit-bot: I haz the power, 2016-06-13 22:45:27]

CQ bit was unchecked

[please use gerrit instead, 2016-06-13 22:51:59]

The CQ bit was checked by rouslan@chromium.org

[please use gerrit instead, 2016-06-13 22:51:59]

The patchset sent to the CQ was uploaded after l-g-t-m from palmer@chromium.org,
haraken@chromium.org, dfalcantara@chromium.org
Link to the patchset: https://codereview.chromium.org/2062583002/#ps40001
(title: "Rebase")

[commit-bot: I haz the power, 2016-06-13 22:52:45]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2062583002/40001

[commit-bot: I haz the power, 2016-06-13 23:05:25]

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

[commit-bot: I haz the power, 2016-06-13 23:05:26]

Try jobs failed on following builders:
  chromeos_daisy_chromium_compile_only_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_daisy_...)
  chromeos_x86-generic_chromium_compile_only_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_x86-ge...)
  mac_chromium_compile_dbg_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_gn_rel on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_gn_r...)

[please use gerrit instead, 2016-06-14 01:03:14]

The CQ bit was checked by rouslan@chromium.org

[please use gerrit instead, 2016-06-14 01:03:14]

The patchset sent to the CQ was uploaded after l-g-t-m from palmer@chromium.org,
haraken@chromium.org, dfalcantara@chromium.org
Link to the patchset: https://codereview.chromium.org/2062583002/#ps60001
(title: "Rebase #2")

[commit-bot: I haz the power, 2016-06-14 01:03:40]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2062583002/60001

[commit-bot: I haz the power, 2016-06-14 02:35:36]

Committed patchset #3 (id:60001)

[commit-bot: I haz the power, 2016-06-14 03:40:20]

Description was changed from

==========
Add totalAmount to PaymentResponse.

This patch adds the totalAmount to PaymentResponse. This reflects the
total amount that the user has confirmed they will pay. The change that
introduced totalAmount also introduced PaymentMethodModifier, which is
not in this patch.

https://w3c.github.io/browser-payment-api/#paymentresponse-interface
https://github.com/w3c/browser-payment-api/commit/67f4f25f4a0562e7b13c5139e1b...

BUG=618860
==========

to

==========
Add totalAmount to PaymentResponse.

This patch adds the totalAmount to PaymentResponse. This reflects the
total amount that the user has confirmed they will pay. The change that
introduced totalAmount also introduced PaymentMethodModifier, which is
not in this patch.

https://w3c.github.io/browser-payment-api/#paymentresponse-interface
https://github.com/w3c/browser-payment-api/commit/67f4f25f4a0562e7b13c5139e1b...

BUG=618860

Committed: https://crrev.com/65f870288f6c830606d8eb9b0510cd0f156724b8
Cr-Commit-Position: refs/heads/master@{#399644}
==========

[commit-bot: I haz the power, 2016-06-14 03:40:22]

Patchset 3 (id:??) landed as
https://crrev.com/65f870288f6c830606d8eb9b0510cd0f156724b8
Cr-Commit-Position: refs/heads/master@{#399644}