Reduce redundant code in PaymentRequest::OnPaymentComplete().

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