PaymentRequest: Ignore shipping options if there are duplicated IDs.

third_party/WebKit/Source/modules/payments/PaymentRequestTest.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/JSONValuesForV8.h"
 #include "bindings/core/v8/V8BindingForTesting.h"
 #include "core/dom/Document.h"
 #include "core/dom/ExceptionCode.h"
@@ skipping 580 matching lines @@
       "\"value\": \"5.00\"}},"
       "\"error\": \"This is an error message.\"}";
 
   request->onUpdatePaymentDetails(ScriptValue::from(
       scope.getScriptState(),
       fromJSONString(scope.getScriptState(), detailWithErrorMsg,
                      scope.getExceptionState())));
   EXPECT_FALSE(scope.getExceptionState().hadException());
 }
 
+TEST(PaymentRequestTest,
+     ShouldResolveWithEmptyShippingOptionsIfIDsOfShippingOptionsAreDuplicated) {
+  V8TestingScope scope;
+  PaymentRequestMockFunctionScope funcs(scope.getScriptState());
+  makePaymentRequestOriginSecure(scope.document());
+  PaymentDetails details;
+  details.setTotal(buildPaymentItemForTest());
+  HeapVector<PaymentShippingOption> shippingOptions(2);
+  shippingOptions[0] = buildShippingOptionForTest(
+      PaymentTestDataId, PaymentTestOverwriteValue, "standard");
+  shippingOptions[0].setSelected(true);
+  shippingOptions[1] = buildShippingOptionForTest(
+      PaymentTestDataId, PaymentTestOverwriteValue, "standard");
+  details.setShippingOptions(shippingOptions);
+  PaymentOptions options;
+  options.setRequestShipping(true);
+  PaymentRequest* request = PaymentRequest::create(
+      scope.getScriptState(), buildPaymentMethodDataForTest(), details, options,
+      scope.getExceptionState());
+  EXPECT_FALSE(scope.getExceptionState().hadException());
+  EXPECT_TRUE(request->shippingOption().isNull());
+  request->show(scope.getScriptState())
+      .then(funcs.expectNoCall(), funcs.expectNoCall());
+  String detailWithShippingOptions =
+      "{\"total\": {\"label\": \"Total\", \"amount\": {\"currency\": \"USD\", "
+      "\"value\": \"5.00\"}},"
+      "\"shippingOptions\": [{\"id\": \"standardShippingOption\", \"label\": "
+      "\"Standard shipping\", \"amount\": {\"currency\": \"USD\", \"value\": "
+      "\"5.00\"}, \"selected\": true}, {\"id\": \"standardShippingOption\", "
+      "\"label\": \"Standard shipping\", \"amount\": {\"currency\": \"USD\", "
+      "\"value\": \"5.00\"}, \"selected\": true}]}";
+
+  request->onUpdatePaymentDetails(ScriptValue::from(
+      scope.getScriptState(),
+      fromJSONString(scope.getScriptState(), detailWithShippingOptions,
+                     scope.getExceptionState())));
+
+  EXPECT_FALSE(scope.getExceptionState().hadException());
+  EXPECT_TRUE(request->shippingOption().isNull());
+}
+
 }  // namespace
 }  // namespace blink