Add data parameter to payment details modifier.

components/payments/payment_request.mojom

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 [JavaPackage="org.chromium.payments.mojom"]
 module payments.mojom;
 
 // The shipping address that the browser process provides to the renderer
 // process. Built either by the browser or a payment app.
 struct PaymentAddress {
@@ skipping 88 matching lines @@
   bool pending;
 };
 
 struct PaymentShippingOption {
   string id;
   string label;
   PaymentCurrencyAmount amount;
   bool selected;
 };
 
+struct PaymentMethodData {
+  array<string> supported_methods;
+
+  // A JSON string built by the renderer from a JavaScript object that the
+  // merchant website provides. The renderer uses
+  // blink::JSONObject::toJSONString() to generate this string. The browser does
+  // not parse the string and passes it as-is directly to payment apps. There's
+  // no one format for this object, so richer types cannot be used. A simple

[palmer, 2016/11/29 00:24:55]

Nit: s/richer/more specific/

[please use gerrit instead, 2016/11/29 16:19:52]

Done.

+  // example:
+  //
+  // {"gateway": "stripe"}
+  string stringified_data;
+
+  // Android Pay specific method data is parsed in the renderer.
+  // https://developers.google.com/web/fundamentals/getting-started/primers/payment-request/android-pay
+  // TODO(rouslan): Stop parsing Android Pay data. http://crbug.com/620173
+  AndroidPayEnvironment environment;
+  string? merchant_name;
+  string? merchant_id;
+  array<AndroidPayCardNetwork> allowed_card_networks;
+  AndroidPayTokenization tokenization_type;
+  array<AndroidPayTokenizationParameter> parameters;
+};
+
 struct PaymentDetailsModifier {
-  array<string> supported_methods;
   PaymentItem? total;
   array<PaymentItem> additional_display_items;
+  PaymentMethodData method_data;
 };
 
 struct PaymentDetails {
   PaymentItem total;
   array<PaymentItem> display_items;
   array<PaymentShippingOption> shipping_options;
   array<PaymentDetailsModifier> modifiers;
   string error;
 };
 
@@ skipping 27 matching lines @@
   UNSPECIFIED,
   GATEWAY_TOKEN,
   NETWORK_TOKEN
 };
 
 struct AndroidPayTokenizationParameter {
   string? key;
   string? value;
 };
 
-struct PaymentMethodData {
-  array<string> supported_methods;
-
-  // A JSON string built by the renderer from a JavaScript object that the
-  // merchant website provides. The renderer uses
-  // blink::JSONObject::toJSONString() to generate this string. The browser does
-  // not parse the string and passes it as-is directly to payment apps. There's
-  // no one format for this object, so richer types cannot be used. A simple
-  // example:
-  //
-  // {"gateway": "stripe"}
-  string stringified_data;
-
-  // Android Pay specific method data is parsed in the renderer.
-  // https://developers.google.com/web/fundamentals/getting-started/primers/payment-request/android-pay
-  // TODO(rouslan): Stop parsing Android Pay data. http://crbug.com/620173
-  AndroidPayEnvironment environment;
-  string? merchant_name;
-  string? merchant_id;
-  array<AndroidPayCardNetwork> allowed_card_networks;
-  AndroidPayTokenization tokenization_type;
-  array<AndroidPayTokenizationParameter> parameters;
-};
-
 enum PaymentComplete {
   SUCCESS,
   FAIL,
   UNKNOWN
 };
 
 interface PaymentRequest {
   Init(PaymentRequestClient client,
        array<PaymentMethodData> methodData,
        PaymentDetails details,
        PaymentOptions options);
   Show();
   UpdateWith(PaymentDetails details);
   Abort();
   Complete(PaymentComplete result);
   CanMakeActivePayment();
 };