DO NOT SUBMIT This fixes CFI bot, but it's ugly

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/ExceptionState.h"
 #include "bindings/core/v8/ScriptState.h"
 #include "core/dom/ExceptionCode.h"
 #include "core/testing/DummyPageHolder.h"
@@ skipping 27 matching lines @@
         m_page->document().setSecurityOrigin(SecurityOrigin::create(KURL(KURL(), securityOrigin)));
     }
 
 private:
     OwnPtr<DummyPageHolder> m_page;
     TrackExceptionState m_exceptionState;
 };
 
 TEST_F(PaymentRequestTest, NoExceptionWithValidData)
 {
-    PaymentRequest::create(getScriptState(), Vector<String>(1, "foo"), buildPaymentDetailsForTest(), getExceptionState());
+    PaymentRequest::create(getScriptState(), Vector<String>(1, "foo"), buildPaymentDetailsForTest(true), getExceptionState());
 
     EXPECT_FALSE(getExceptionState().hadException());
 }
 
 TEST_F(PaymentRequestTest, SecureContextRequired)
 {
     setSecurityOrigin("http://www.example.com");
 
-    PaymentRequest::create(getScriptState(), Vector<String>(1, "foo"), buildPaymentDetailsForTest(), getExceptionState());
+    PaymentRequest::create(getScriptState(), Vector<String>(1, "foo"), buildPaymentDetailsForTest(true), getExceptionState());
 
     EXPECT_TRUE(getExceptionState().hadException());
     EXPECT_EQ(SecurityError, getExceptionState().code());
 }
 
 TEST_F(PaymentRequestTest, SupportedMethodListRequired)
 {
-    PaymentRequest::create(getScriptState(), Vector<String>(), buildPaymentDetailsForTest(), getExceptionState());
+    PaymentRequest::create(getScriptState(), Vector<String>(), buildPaymentDetailsForTest(true), getExceptionState());
 
     EXPECT_TRUE(getExceptionState().hadException());
     EXPECT_EQ(V8TypeError, getExceptionState().code());
 }
 
 TEST_F(PaymentRequestTest, ItemListRequired)
 {
     PaymentRequest::create(getScriptState(), Vector<String>(1, "foo"), PaymentDetails(), getExceptionState());
 
     EXPECT_TRUE(getExceptionState().hadException());
     EXPECT_EQ(V8TypeError, getExceptionState().code());
 }
 
 TEST_F(PaymentRequestTest, AtLeastOnePaymentDetailsItemRequired)
 {
     PaymentDetails emptyItems;
     emptyItems.setItems(HeapVector<PaymentItem>());
 
     PaymentRequest::create(getScriptState(), Vector<String>(1, "foo"), emptyItems, getExceptionState());
 
     EXPECT_TRUE(getExceptionState().hadException());
     EXPECT_EQ(V8TypeError, getExceptionState().code());
 }
 
 TEST_F(PaymentRequestTest, NullShippingOptionWhenNoOptionsAvailable)
 {
-    PaymentDetails details = buildPaymentDetailsForTest();
+    PaymentDetails details = buildPaymentDetailsForTest(false);
     details.setShippingOptions(HeapVector<ShippingOption>());
 
     PaymentRequest* request = PaymentRequest::create(getScriptState(), Vector<String>(1, "foo"), details, getExceptionState());
 
     EXPECT_TRUE(request->shippingOption().isNull());
 }
 
 TEST_F(PaymentRequestTest, NullShippingOptionWhenMultipleOptionsAvailable)
 {
-    PaymentDetails details = buildPaymentDetailsForTest();
+    PaymentDetails details = buildPaymentDetailsForTest(true);
     EXPECT_LE(2U, details.shippingOptions().size());
 
     PaymentRequest* request = PaymentRequest::create(getScriptState(), Vector<String>(1, "foo"), details, getExceptionState());
 
     EXPECT_TRUE(request->shippingOption().isNull());
 }
 
 TEST_F(PaymentRequestTest, SelectSingleAvailableShippingOption)
 {
     CurrencyAmount amount;
     amount.setCurrencyCode("USD");
     amount.setValue("10.00");
     ShippingOption option;
     option.setAmount(amount);
     option.setId("standard");
     option.setLabel("Standard shipping");
-    PaymentDetails details = buildPaymentDetailsForTest();
+    PaymentDetails details = buildPaymentDetailsForTest(true);
     details.setShippingOptions(HeapVector<ShippingOption>(1, option));
 
     PaymentRequest* request = PaymentRequest::create(getScriptState(), Vector<String>(1, "foo"), details, getExceptionState());
 
     EXPECT_EQ("standard", request->shippingOption());
 }
 
 TEST_F(PaymentRequestTest, AbortWithoutShowShouldThrow)
 {
-    PaymentRequest* request = PaymentRequest::create(getScriptState(), Vector<String>(1, "foo"), buildPaymentDetailsForTest(), getExceptionState());
+    PaymentRequest* request = PaymentRequest::create(getScriptState(), Vector<String>(1, "foo"), buildPaymentDetailsForTest(true), getExceptionState());
     EXPECT_FALSE(getExceptionState().hadException());
 
     request->abort(getExceptionState());
     EXPECT_TRUE(getExceptionState().hadException());
 }
 
 } // namespace
 } // namespace blink