Implement request id in PaymentDetailsInit

third_party/WebKit/Source/modules/payments/PaymentResponse.cpp

 // 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.
 
 #include "modules/payments/PaymentResponse.h"
 
 #include "bindings/core/v8/ExceptionState.h"
 #include "bindings/core/v8/V8ObjectBuilder.h"
 #include "modules/payments/PaymentAddress.h"
 #include "modules/payments/PaymentCompleter.h"
 #include "wtf/Assertions.h"
 
 namespace blink {
 
 PaymentResponse::PaymentResponse(
     payments::mojom::blink::PaymentResponsePtr response,
     PaymentCompleter* paymentCompleter)
-    : m_methodName(response->method_name),
+    : m_requestId(response->requestId),

[please use gerrit instead, 2017/03/27 16:22:36]

Use the "id" from the PaymentRequest in the renderer instead.

[rwlbuis, 2017/03/27 20:45:29]

Done.

+      m_methodName(response->method_name),
       m_stringifiedDetails(response->stringified_details),
       m_shippingAddress(
           response->shipping_address
               ? new PaymentAddress(std::move(response->shipping_address))
               : nullptr),
       m_shippingOption(response->shipping_option),
       m_payerName(response->payer_name),
       m_payerEmail(response->payer_email),
       m_payerPhone(response->payer_phone),
       m_paymentCompleter(paymentCompleter) {
   DCHECK(m_paymentCompleter);
 }
 
 PaymentResponse::~PaymentResponse() {}
 
 ScriptValue PaymentResponse::toJSONForBinding(ScriptState* scriptState) const {
   V8ObjectBuilder result(scriptState);
+  result.addString("requestId", requestId());
   result.addString("methodName", methodName());
   result.add("details", details(scriptState, ASSERT_NO_EXCEPTION));
 
   if (shippingAddress())
     result.add("shippingAddress",
                shippingAddress()->toJSONForBinding(scriptState));
   else
     result.addNull("shippingAddress");
 
   result.addStringOrNull("shippingOption", shippingOption())
@@ skipping 20 matching lines @@
     convertedResult = PaymentCompleter::Fail;
   return m_paymentCompleter->complete(scriptState, convertedResult);
 }
 
 DEFINE_TRACE(PaymentResponse) {
   visitor->trace(m_shippingAddress);
   visitor->trace(m_paymentCompleter);
 }
 
 }  // namespace blink