Add shippingOption to PaymentResponse

Add shippingOption to PaymentResponse

PaymentRequest lets web developers access the shipping option String
through the PaymentRequest object. As it is a part of information
necessary to complete a transaction, it makes sense to place this
information in PaymentResponse as well.

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

* All tests for PaymentRequest::OnPaymentResponse() have been moved into
  OnPaymentResponseTest.cpp.
* The shipping_option_id field in Mojo is renamed to shipping_option to
  match the JavaScript API.

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

[please use gerrit instead, 2016-06-15 23:00:32]

Patchset #1 (id:1) has been deleted

[please use gerrit instead, 2016-06-15 23:02:29]

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

[please use gerrit instead, 2016-06-15 23:02:30]

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

dfalcantara@, ptal android.

palmer@, ptal payment_request.mojom.

[haraken, 2016-06-16 00:44:09]

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

[Marijn Kruisselbrink, 2016-06-16 07:04:02]

lgtm

[gone, 2016-06-16 18:25:01]

android lgtm

[palmer, 2016-06-16 22:32:37]

lgtm

[please use gerrit instead, 2016-06-17 20:32:15]

The CQ bit was checked by rouslan@chromium.org

[please use gerrit instead, 2016-06-17 20:32:15]

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

[commit-bot: I haz the power, 2016-06-17 20:32:46]

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

[commit-bot: I haz the power, 2016-06-17 23:38:58]

Committed patchset #3 (id:60001)

[commit-bot: I haz the power, 2016-06-17 23:40:32]

Description was changed from

==========
Add shippingOption to PaymentResponse

PaymentRequest lets web developers access the shipping option String
through the PaymentRequest object. As it is a part of information
necessary to complete a transaction, it makes sense to place this
information in PaymentResponse as well.

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

* All tests for PaymentRequest::OnPaymentResponse() have been moved into
  OnPaymentResponseTest.cpp.
* The shipping_option_id field in Mojo is renamed to shipping_option to
  match the JavaScript API.

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

to

==========
Add shippingOption to PaymentResponse

PaymentRequest lets web developers access the shipping option String
through the PaymentRequest object. As it is a part of information
necessary to complete a transaction, it makes sense to place this
information in PaymentResponse as well.

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

* All tests for PaymentRequest::OnPaymentResponse() have been moved into
  OnPaymentResponseTest.cpp.
* The shipping_option_id field in Mojo is renamed to shipping_option to
  match the JavaScript API.

BUG=587995

Committed: https://crrev.com/1532d8d156be95a9bc272cc9fcbb96c00a67c286
Cr-Commit-Position: refs/heads/master@{#400545}
==========

[commit-bot: I haz the power, 2016-06-17 23:40:33]

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