Adding support for phone and email.

third_party/WebKit/Source/modules/payments/PaymentRequest.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/PaymentRequest.h"
 
 #include "bindings/core/v8/ExceptionState.h"
 #include "bindings/core/v8/JSONValuesForV8.h"
 #include "bindings/core/v8/ScriptPromiseResolver.h"
 #include "bindings/core/v8/ScriptState.h"
@@ skipping 433 matching lines @@
         m_shippingAddress = new PaymentAddress(response->shipping_address.Clone());
         m_shippingOption = response->shipping_option_id;
     } else {
         if (response->shipping_address) {
             m_showResolver->reject(DOMException::create(SyntaxError));
             clearResolversAndCloseMojoConnection();
             return;
         }
     }
 
+    if (m_options.requestPayerEmail() && (response->payer_email.isNull() || response->payer_email.isEmpty())) {

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

nit: Should be "if (m_options.requestPayerEmail() &&
response->payer_email.isEmpty()) {".

We don't need isNull check.. because isEmpty includes isNull check.

The isEmpty means "isNull || length == 0".

FYI:
https://cs.chromium.org/chromium/src/third_party/WebKit/Source/wtf/text/WTFSt...

+        m_showResolver->reject(DOMException::create(SyntaxError));
+        clearResolversAndCloseMojoConnection();
+        return;
+    }
+
+    if (m_options.requestPayerPhone() && (response->payer_phone.isNull() || response->payer_phone.isEmpty())) {

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

ditto

+        m_showResolver->reject(DOMException::create(SyntaxError));
+        clearResolversAndCloseMojoConnection();
+        return;
+    }
+
     m_showResolver->resolve(new PaymentResponse(std::move(response), this));
 
     // Do not close the mojo connection here. The merchant website should call
     // PaymentResponse::complete(boolean), which will be forwarded over the mojo
     // connection to display a success or failure message to the user.
     m_showResolver.clear();
 }
 
 void PaymentRequest::OnError()
 {
@@ skipping 14 matching lines @@
 void PaymentRequest::clearResolversAndCloseMojoConnection()
 {
     m_completeResolver.clear();
     m_showResolver.clear();
     if (m_clientBinding.is_bound())
         m_clientBinding.Close();
     m_paymentProvider.reset();
 }
 
 } // namespace blink