Adding support for phone and email.

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

Index: third_party/WebKit/Source/modules/payments/PaymentResponse.cpp
diff --git a/third_party/WebKit/Source/modules/payments/PaymentResponse.cpp b/third_party/WebKit/Source/modules/payments/PaymentResponse.cpp
index 66362d69ad24ad5e43b0193dec218e96b7c1646d..ea23f6e92a2ca1272e4fec892aaff766755ba745 100644
--- a/third_party/WebKit/Source/modules/payments/PaymentResponse.cpp
+++ b/third_party/WebKit/Source/modules/payments/PaymentResponse.cpp
@@ -15,6 +15,8 @@ PaymentResponse::PaymentResponse(mojom::blink::PaymentResponsePtr response, Paym
     : 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.isEmpty() ? String() : response->payer_email)

[dcheng, 2016/06/08 00:31:44]

I'm kind of surprised there's no better way to do this. +yzshen, +esprehn, do we
have best practices for Blink Onion Soup stuff like this?

[dcheng, 2016/06/08 00:37:53]

Actually, I just talked to esprehn@ and he confirmed that in the Blink variant,
this is already a WTF::String. Why do we need to do this check at all?

[please use gerrit instead, 2016/06/08 01:27:17]

I think we can be simple:

, m_payerEmail(response->payer_email)
, mPayerPhone(response->payer_phone)

[yzshen1, 2016/06/08 16:28:59]

(sorry for late response.) Yes, that is already WTF::String.

+    , m_payerPhone(response->payer_phone.isEmpty() ? String() : response->payer_phone)
     , m_paymentCompleter(paymentCompleter)
 {
     DCHECK(m_paymentCompleter);