Add 'total' field to 'PaymentDetails'.

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 61 matching lines @@
         output->amount = CurrencyAmount::From(input.amount());
         return output;
     }
 };
 
 template <>
 struct TypeConverter<PaymentDetailsPtr, blink::PaymentDetails> {
     static PaymentDetailsPtr Convert(const blink::PaymentDetails& input)
     {
         PaymentDetailsPtr output = PaymentDetails::New();
-        output->display_items = mojo::WTFArray<PaymentItemPtr>::From(input.displayItems());
+        output->total = PaymentItem::From(input.total());
+
+        if (input.hasDisplayItems())
+            output->display_items = mojo::WTFArray<PaymentItemPtr>::From(input.displayItems());
+        else
+            output->display_items = mojo::WTFArray<PaymentItemPtr>::New(0);
+
         if (input.hasShippingOptions())
             output->shipping_options = mojo::WTFArray<ShippingOptionPtr>::From(input.shippingOptions());
         else
             output->shipping_options = mojo::WTFArray<ShippingOptionPtr>::New(0);
+
         return output;
     }
 };
 
 template <>
 struct TypeConverter<PaymentOptionsPtr, blink::PaymentOptions> {
     static PaymentOptionsPtr Convert(const blink::PaymentOptions& input)
     {
         PaymentOptionsPtr output = PaymentOptions::New();
         output->request_shipping = input.requestShipping();
         return output;
     }
 };
 
 } // namespace mojo
 
 namespace blink {
 namespace {
 
-// Validates ShippingOption and PaymentItem dictionaries, which happen to have identical fields,
+// Validates ShippingOption or PaymentItem, which happen to have identical fields,
 // except for "id", which is present only in ShippingOption.
 template <typename T>
-void validateShippingOptionsOrPaymentItems(HeapVector<T> items, ExceptionState& exceptionState)
+void validateShippingOptionOrPaymentItem(const T& item, ExceptionState& exceptionState)
 {
+    if (!item.hasLabel() || item.label().isEmpty()) {
+        exceptionState.throwTypeError("Item label required");
+        return;
+    }
+
+    if (!item.hasAmount()) {
+        exceptionState.throwTypeError("Currency amount required");
+        return;
+    }
+
+    if (!item.amount().hasCurrency()) {
+        exceptionState.throwTypeError("Currency code required");
+        return;
+    }
+
+    if (!item.amount().hasValue()) {
+        exceptionState.throwTypeError("Currency value required");
+        return;
+    }
+
     String errorMessage;
-    for (const auto& item : items) {
-        if (!item.hasLabel() || item.label().isEmpty()) {
-            exceptionState.throwTypeError("Item label required");
-            return;
-        }
+    if (!PaymentsValidators::isValidCurrencyCodeFormat(item.amount().currency(), &errorMessage)) {
+        exceptionState.throwTypeError(errorMessage);
+        return;
+    }
 
-        if (!item.hasAmount()) {
-            exceptionState.throwTypeError("Currency amount required");
-            return;
-        }
-
-        if (!item.amount().hasCurrency()) {
-            exceptionState.throwTypeError("Currency code required");
-            return;
-        }
-
-        if (!item.amount().hasValue()) {
-            exceptionState.throwTypeError("Currency value required");
-            return;
-        }
-
-        if (!PaymentsValidators::isValidCurrencyCodeFormat(item.amount().currency(), &errorMessage)) {
-            exceptionState.throwTypeError(errorMessage);
-            return;
-        }
-
-        if (!PaymentsValidators::isValidAmountFormat(item.amount().value(), &errorMessage)) {
-            exceptionState.throwTypeError(errorMessage);
-            return;
-        }
+    if (!PaymentsValidators::isValidAmountFormat(item.amount().value(), &errorMessage)) {
+        exceptionState.throwTypeError(errorMessage);
+        return;
     }
 }
 
-void validateShippingOptionsIds(HeapVector<ShippingOption> options, ExceptionState& exceptionState)
+void validateDisplayItems(const HeapVector<PaymentItem>& items, ExceptionState& exceptionState)
+{
+    for (const auto& item : items) {
+        validateShippingOptionOrPaymentItem(item, exceptionState);
+        if (exceptionState.hadException())
+            return;
+    }
+}
+
+void validateShippingOptions(const HeapVector<ShippingOption>& options, ExceptionState& exceptionState)
 {
     for (const auto& option : options) {
         if (!option.hasId() || option.id().isEmpty()) {
             exceptionState.throwTypeError("ShippingOption id required");
             return;
         }
+
+        validateShippingOptionOrPaymentItem(option, exceptionState);
+        if (exceptionState.hadException())
+            return;
     }
 }
 
 void validatePaymentDetails(const PaymentDetails& details, ExceptionState& exceptionState)
 {
-    if (!details.hasDisplayItems()) {
-        exceptionState.throwTypeError("Must specify display items");
+    if (!details.hasTotal()) {
+        exceptionState.throwTypeError("Must specify total");
         return;
     }
 
-    if (details.displayItems().isEmpty()) {
-        exceptionState.throwTypeError("Must specify at least one item");
+    validateShippingOptionOrPaymentItem(details.total(), exceptionState);
+    if (exceptionState.hadException())
+        return;
+
+    if (details.total().amount().value()[0] == '-') {
+        exceptionState.throwTypeError("Total amount value should be non-negative");
         return;
     }
 
-    validateShippingOptionsOrPaymentItems(details.displayItems(), exceptionState);
-    if (exceptionState.hadException())
-        return;
+    if (details.hasDisplayItems()) {
+        validateDisplayItems(details.displayItems(), exceptionState);
+        if (exceptionState.hadException())
+            return;
+    }
 
     if (details.hasShippingOptions()) {
-        validateShippingOptionsOrPaymentItems(details.shippingOptions(), exceptionState);
-        validateShippingOptionsIds(details.shippingOptions(), exceptionState);
+        validateShippingOptions(details.shippingOptions(), exceptionState);
     }
 }
 
 } // namespace
 
 PaymentRequest* PaymentRequest::create(ScriptState* scriptState, const Vector<String>& supportedMethods, const PaymentDetails& details, ExceptionState& exceptionState)
 {
     return new PaymentRequest(scriptState, supportedMethods, details, PaymentOptions(), ScriptValue(), exceptionState);
 }
 
@@ skipping 259 matching lines @@
 void PaymentRequest::clearResolversAndCloseMojoConnection()
 {
     m_completeResolver.clear();
     m_showResolver.clear();
     if (m_clientBinding.is_bound())
         m_clientBinding.Close();
     m_paymentProvider.reset();
 }
 
 } // namespace blink