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 462 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.isEmpty()) {

[palmer, 2016/06/16 21:23:34]

Nit: This code is unnecessarily repetitive. Maybe combine the various conditions
into 1 large condition, and only write the body once?

+        m_showResolver->reject(DOMException::create(SyntaxError));
+        clearResolversAndCloseMojoConnection();
+        return;
+    }
+
+    if (!m_options.requestPayerEmail() && !response->payer_email.isNull()) {
+        m_showResolver->reject(DOMException::create(SyntaxError));
+        clearResolversAndCloseMojoConnection();
+        return;
+    }
+
+    if (m_options.requestPayerPhone() && response->payer_phone.isEmpty()) {
+        m_showResolver->reject(DOMException::create(SyntaxError));
+        clearResolversAndCloseMojoConnection();
+        return;
+    }
+
+    if (!m_options.requestPayerPhone() && !response->payer_phone.isNull()) {
+        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 33 matching lines @@
 {
     m_completeResolver.clear();
     m_showResolver.clear();
     m_abortResolver.clear();
     if (m_clientBinding.is_bound())
         m_clientBinding.Close();
     m_paymentProvider.reset();
 }
 
 } // namespace blink