PaymentRequest: Introduce PaymentMethodData.

third_party/WebKit/Source/modules/payments/PaymentRequestDetailsTest.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 59 matching lines @@
     void setSecurityOrigin(const String& securityOrigin)
     {
         m_page->document().setSecurityOrigin(SecurityOrigin::create(KURL(KURL(), securityOrigin)));
     }
 
 private:
     OwnPtr<DummyPageHolder> m_page;
     TrackExceptionState m_exceptionState;
 };
 
+HeapVector<PaymentMethodData> buildPaymentMethodDataForTest()

[please use gerrit instead, 2016/05/29 19:57:12]

Rename PaymentDetailsTestHelper into PaymentTestHelper and put this function in
there, so you can re-use it in multiple files.

[zino, 2016/06/01 17:43:46]

Done.

+{
+    HeapVector<PaymentMethodData> methodData(1, PaymentMethodData());
+    methodData[0].setSupportedMethods(Vector<String>(1, "food"));

[please use gerrit instead, 2016/05/29 19:57:12]

"foo" ?

[zino, 2016/06/01 17:43:46]

Done.

+    return methodData;
+}
+
 TEST_P(PaymentRequestDetailsTest, ValidatesDetails)
 {
-    PaymentRequest::create(getScriptState(), Vector<String>(1, "foo"), GetParam().buildDetails(), getExceptionState());
+    PaymentRequest::create(getScriptState(), buildPaymentMethodDataForTest(), GetParam().buildDetails(), getExceptionState());
 
     EXPECT_EQ(GetParam().expectException(), getExceptionState().hadException());
     if (GetParam().expectException())
         EXPECT_EQ(GetParam().getExpectedExceptionCode(), getExceptionState().code());
 }
 
 INSTANTIATE_TEST_CASE_P(MissingData,
     PaymentRequestDetailsTest,
     testing::Values(
         DetailsTestCase(PaymentTestDetailItem, PaymentTestDataAmount, PaymentTestRemoveKey, "", true, V8TypeError),
@@ skipping 88 matching lines @@
         DetailsTestCase(PaymentTestDetailShippingOption, PaymentTestDataValue, PaymentTestOverwriteValue, "10.", true, V8TypeError),
         DetailsTestCase(PaymentTestDetailShippingOption, PaymentTestDataValue, PaymentTestOverwriteValue, ".99", true, V8TypeError),
         DetailsTestCase(PaymentTestDetailShippingOption, PaymentTestDataValue, PaymentTestOverwriteValue, "-10.", true, V8TypeError),
         DetailsTestCase(PaymentTestDetailShippingOption, PaymentTestDataValue, PaymentTestOverwriteValue, "10-", true, V8TypeError),
         DetailsTestCase(PaymentTestDetailShippingOption, PaymentTestDataValue, PaymentTestOverwriteValue, "1-0", true, V8TypeError),
         DetailsTestCase(PaymentTestDetailShippingOption, PaymentTestDataValue, PaymentTestOverwriteValue, "1.0.0", true, V8TypeError),
         DetailsTestCase(PaymentTestDetailShippingOption, PaymentTestDataValue, PaymentTestOverwriteValue, "1/3", true, V8TypeError)));
 
 } // namespace
 } // namespace blink