Adding support for phone and email.

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/JSONValuesForV8.h"
 #include "modules/payments/PaymentAddress.h"
 #include "modules/payments/PaymentCompleter.h"
 #include "wtf/Assertions.h"
 
 namespace blink {
 
 PaymentResponse::PaymentResponse(mojom::blink::PaymentResponsePtr response, PaymentCompleter* paymentCompleter)
     : m_methodName(response->method_name)
     , m_stringifiedDetails(response->stringified_details)
     , m_shippingAddress(response->shipping_address ? new PaymentAddress(std::move(response->shipping_address)) : nullptr)
+    , m_payerEmail(response->payer_email)

[zino, 2016/06/07 16:40:08]

The payerEmail can be null. (It has DOMString? type instead of DOMString.)
So, this should be m_payerEmail(response->payer_email.isEmpty() ? String() :
response->payer_email).

+    , m_payerPhone(response->payer_phone)

[zino, 2016/06/07 16:40:08]

ditto

     , m_paymentCompleter(paymentCompleter)
 {
     DCHECK(m_paymentCompleter);
 }
 
 PaymentResponse::~PaymentResponse()
 {
 }
 
 ScriptValue PaymentResponse::details(ScriptState* scriptState, ExceptionState& exceptionState) const
 {
     return ScriptValue(scriptState, fromJSONString(scriptState, m_stringifiedDetails, exceptionState));
 }
 
 ScriptPromise PaymentResponse::complete(ScriptState* scriptState, bool success)
 {
     return m_paymentCompleter->complete(scriptState, success);
 }
 
 DEFINE_TRACE(PaymentResponse)
 {
     visitor->trace(m_shippingAddress);
     visitor->trace(m_paymentCompleter);
 }
 
 } // namespace blink