Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(373)

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: third_party/WebKit/Source/modules/payments/OnPaymentResponseTest.cpp

Issue 2748093003: PaymentRequest: Introduce PaymentDetailsInit and PaymentDetailsUpdate. (Closed)
Patch Set: PD Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« components/payments/content/payment_request.mojom ('K') | « third_party/WebKit/Source/modules/payments/CompleteTest.cpp ('k') | third_party/WebKit/Source/modules/payments/PaymentDetails.idl » ('j') | third_party/WebKit/Source/modules/payments/PaymentRequest.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/Source/modules/payments/OnPaymentResponseTest.cpp
diff --git a/third_party/WebKit/Source/modules/payments/OnPaymentResponseTest.cpp b/third_party/WebKit/Source/modules/payments/OnPaymentResponseTest.cpp
index 79fff77f2b1b84f878fd4169e885820c5500783d..99d1e1834b9d9f4ca3115d89e55ecdfc7e67cd4f 100644
--- a/third_party/WebKit/Source/modules/payments/OnPaymentResponseTest.cpp
+++ b/third_party/WebKit/Source/modules/payments/OnPaymentResponseTest.cpp
@@ -26,7 +26,7 @@ TEST(OnPaymentResponseTest, RejectMissingShippingOption) {
options.setRequestShipping(true);
PaymentRequest* request = PaymentRequest::create(
scope.getExecutionContext(), buildPaymentMethodDataForTest(),
- buildPaymentDetailsForTest(), options, scope.getExceptionState());
+ buildPaymentDetailsInitForTest(), options, scope.getExceptionState());
ASSERT_FALSE(scope.getExceptionState().hadException());
payments::mojom::blink::PaymentResponsePtr response =
buildPaymentResponseForTest();
@@ -52,7 +52,7 @@ TEST(OnPaymentResponseTest, RejectMissingAddress) {
options.setRequestShipping(true);
PaymentRequest* request = PaymentRequest::create(
scope.getExecutionContext(), buildPaymentMethodDataForTest(),
- buildPaymentDetailsForTest(), options, scope.getExceptionState());
+ buildPaymentDetailsInitForTest(), options, scope.getExceptionState());
ASSERT_FALSE(scope.getExceptionState().hadException());
payments::mojom::blink::PaymentResponsePtr response =
buildPaymentResponseForTest();
@@ -75,7 +75,7 @@ TEST(OnPaymentResponseTest, RejectMissingName) {
options.setRequestPayerName(true);
PaymentRequest* request = PaymentRequest::create(
scope.getExecutionContext(), buildPaymentMethodDataForTest(),
- buildPaymentDetailsForTest(), options, scope.getExceptionState());
+ buildPaymentDetailsInitForTest(), options, scope.getExceptionState());
EXPECT_FALSE(scope.getExceptionState().hadException());
payments::mojom::blink::PaymentResponsePtr response =
payments::mojom::blink::PaymentResponse::New();
@@ -97,7 +97,7 @@ TEST(OnPaymentResponseTest, RejectMissingEmail) {
options.setRequestPayerEmail(true);
PaymentRequest* request = PaymentRequest::create(
scope.getExecutionContext(), buildPaymentMethodDataForTest(),
- buildPaymentDetailsForTest(), options, scope.getExceptionState());
+ buildPaymentDetailsInitForTest(), options, scope.getExceptionState());
EXPECT_FALSE(scope.getExceptionState().hadException());
payments::mojom::blink::PaymentResponsePtr response =
payments::mojom::blink::PaymentResponse::New();
@@ -119,7 +119,7 @@ TEST(OnPaymentResponseTest, RejectMissingPhone) {
options.setRequestPayerPhone(true);
PaymentRequest* request = PaymentRequest::create(
scope.getExecutionContext(), buildPaymentMethodDataForTest(),
- buildPaymentDetailsForTest(), options, scope.getExceptionState());
+ buildPaymentDetailsInitForTest(), options, scope.getExceptionState());
EXPECT_FALSE(scope.getExceptionState().hadException());
payments::mojom::blink::PaymentResponsePtr response =
payments::mojom::blink::PaymentResponse::New();
@@ -141,7 +141,7 @@ TEST(OnPaymentResponseTest, RejectEmptyShippingOption) {
options.setRequestShipping(true);
PaymentRequest* request = PaymentRequest::create(
scope.getExecutionContext(), buildPaymentMethodDataForTest(),
- buildPaymentDetailsForTest(), options, scope.getExceptionState());
+ buildPaymentDetailsInitForTest(), options, scope.getExceptionState());
ASSERT_FALSE(scope.getExceptionState().hadException());
payments::mojom::blink::PaymentResponsePtr response =
buildPaymentResponseForTest();
@@ -168,7 +168,7 @@ TEST(OnPaymentResponseTest, RejectEmptyAddress) {
options.setRequestShipping(true);
PaymentRequest* request = PaymentRequest::create(
scope.getExecutionContext(), buildPaymentMethodDataForTest(),
- buildPaymentDetailsForTest(), options, scope.getExceptionState());
+ buildPaymentDetailsInitForTest(), options, scope.getExceptionState());
ASSERT_FALSE(scope.getExceptionState().hadException());
payments::mojom::blink::PaymentResponsePtr response =
buildPaymentResponseForTest();
@@ -192,7 +192,7 @@ TEST(OnPaymentResponseTest, RejectEmptyName) {
options.setRequestPayerName(true);
PaymentRequest* request = PaymentRequest::create(
scope.getExecutionContext(), buildPaymentMethodDataForTest(),
- buildPaymentDetailsForTest(), options, scope.getExceptionState());
+ buildPaymentDetailsInitForTest(), options, scope.getExceptionState());
EXPECT_FALSE(scope.getExceptionState().hadException());
payments::mojom::blink::PaymentResponsePtr response =
payments::mojom::blink::PaymentResponse::New();
@@ -215,7 +215,7 @@ TEST(OnPaymentResponseTest, RejectEmptyEmail) {
options.setRequestPayerEmail(true);
PaymentRequest* request = PaymentRequest::create(
scope.getExecutionContext(), buildPaymentMethodDataForTest(),
- buildPaymentDetailsForTest(), options, scope.getExceptionState());
+ buildPaymentDetailsInitForTest(), options, scope.getExceptionState());
EXPECT_FALSE(scope.getExceptionState().hadException());
payments::mojom::blink::PaymentResponsePtr response =
payments::mojom::blink::PaymentResponse::New();
@@ -238,7 +238,7 @@ TEST(OnPaymentResponseTest, RejectEmptyPhone) {
options.setRequestPayerPhone(true);
PaymentRequest* request = PaymentRequest::create(
scope.getExecutionContext(), buildPaymentMethodDataForTest(),
- buildPaymentDetailsForTest(), options, scope.getExceptionState());
+ buildPaymentDetailsInitForTest(), options, scope.getExceptionState());
EXPECT_FALSE(scope.getExceptionState().hadException());
payments::mojom::blink::PaymentResponsePtr response =
payments::mojom::blink::PaymentResponse::New();
@@ -261,7 +261,7 @@ TEST(OnPaymentResponseTest, RejectNotRequestedAddress) {
options.setRequestShipping(false);
PaymentRequest* request = PaymentRequest::create(
scope.getExecutionContext(), buildPaymentMethodDataForTest(),
- buildPaymentDetailsForTest(), options, scope.getExceptionState());
+ buildPaymentDetailsInitForTest(), options, scope.getExceptionState());
ASSERT_FALSE(scope.getExceptionState().hadException());
payments::mojom::blink::PaymentResponsePtr response =
payments::mojom::blink::PaymentResponse::New();
@@ -287,7 +287,7 @@ TEST(OnPaymentResponseTest, RejectNotRequestedShippingOption) {
options.setRequestShipping(false);
PaymentRequest* request = PaymentRequest::create(
scope.getExecutionContext(), buildPaymentMethodDataForTest(),
- buildPaymentDetailsForTest(), options, scope.getExceptionState());
+ buildPaymentDetailsInitForTest(), options, scope.getExceptionState());
ASSERT_FALSE(scope.getExceptionState().hadException());
payments::mojom::blink::PaymentResponsePtr response =
payments::mojom::blink::PaymentResponse::New();
@@ -310,7 +310,7 @@ TEST(OnPaymentResponseTest, RejectNotRequestedName) {
options.setRequestPayerName(false);
PaymentRequest* request = PaymentRequest::create(
scope.getExecutionContext(), buildPaymentMethodDataForTest(),
- buildPaymentDetailsForTest(), options, scope.getExceptionState());
+ buildPaymentDetailsInitForTest(), options, scope.getExceptionState());
EXPECT_FALSE(scope.getExceptionState().hadException());
payments::mojom::blink::PaymentResponsePtr response =
payments::mojom::blink::PaymentResponse::New();
@@ -333,7 +333,7 @@ TEST(OnPaymentResponseTest, RejectNotRequestedEmail) {
options.setRequestPayerEmail(false);
PaymentRequest* request = PaymentRequest::create(
scope.getExecutionContext(), buildPaymentMethodDataForTest(),
- buildPaymentDetailsForTest(), options, scope.getExceptionState());
+ buildPaymentDetailsInitForTest(), options, scope.getExceptionState());
EXPECT_FALSE(scope.getExceptionState().hadException());
payments::mojom::blink::PaymentResponsePtr response =
payments::mojom::blink::PaymentResponse::New();
@@ -356,7 +356,7 @@ TEST(OnPaymentResponseTest, RejectNotRequestedPhone) {
options.setRequestPayerPhone(false);
PaymentRequest* request = PaymentRequest::create(
scope.getExecutionContext(), buildPaymentMethodDataForTest(),
- buildPaymentDetailsForTest(), options, scope.getExceptionState());
+ buildPaymentDetailsInitForTest(), options, scope.getExceptionState());
EXPECT_FALSE(scope.getExceptionState().hadException());
payments::mojom::blink::PaymentResponsePtr response =
payments::mojom::blink::PaymentResponse::New();
@@ -379,7 +379,7 @@ TEST(OnPaymentResponseTest, RejectInvalidAddress) {
options.setRequestShipping(true);
PaymentRequest* request = PaymentRequest::create(
scope.getExecutionContext(), buildPaymentMethodDataForTest(),
- buildPaymentDetailsForTest(), options, scope.getExceptionState());
+ buildPaymentDetailsInitForTest(), options, scope.getExceptionState());
ASSERT_FALSE(scope.getExceptionState().hadException());
payments::mojom::blink::PaymentResponsePtr response =
buildPaymentResponseForTest();
@@ -428,7 +428,7 @@ TEST(OnPaymentResponseTest, CanRequestShippingInformation) {
options.setRequestShipping(true);
PaymentRequest* request = PaymentRequest::create(
scope.getExecutionContext(), buildPaymentMethodDataForTest(),
- buildPaymentDetailsForTest(), options, scope.getExceptionState());
+ buildPaymentDetailsInitForTest(), options, scope.getExceptionState());
ASSERT_FALSE(scope.getExceptionState().hadException());
payments::mojom::blink::PaymentResponsePtr response =
buildPaymentResponseForTest();
@@ -463,7 +463,7 @@ TEST(OnPaymentResponseTest, CanRequestName) {
options.setRequestPayerName(true);
PaymentRequest* request = PaymentRequest::create(
scope.getExecutionContext(), buildPaymentMethodDataForTest(),
- buildPaymentDetailsForTest(), options, scope.getExceptionState());
+ buildPaymentDetailsInitForTest(), options, scope.getExceptionState());
EXPECT_FALSE(scope.getExceptionState().hadException());
payments::mojom::blink::PaymentResponsePtr response =
payments::mojom::blink::PaymentResponse::New();
@@ -492,7 +492,7 @@ TEST(OnPaymentResponseTest, CanRequestEmail) {
options.setRequestPayerEmail(true);
PaymentRequest* request = PaymentRequest::create(
scope.getExecutionContext(), buildPaymentMethodDataForTest(),
- buildPaymentDetailsForTest(), options, scope.getExceptionState());
+ buildPaymentDetailsInitForTest(), options, scope.getExceptionState());
EXPECT_FALSE(scope.getExceptionState().hadException());
payments::mojom::blink::PaymentResponsePtr response =
payments::mojom::blink::PaymentResponse::New();
@@ -521,7 +521,7 @@ TEST(OnPaymentResponseTest, CanRequestPhone) {
options.setRequestPayerPhone(true);
PaymentRequest* request = PaymentRequest::create(
scope.getExecutionContext(), buildPaymentMethodDataForTest(),
- buildPaymentDetailsForTest(), options, scope.getExceptionState());
+ buildPaymentDetailsInitForTest(), options, scope.getExceptionState());
EXPECT_FALSE(scope.getExceptionState().hadException());
payments::mojom::blink::PaymentResponsePtr response =
payments::mojom::blink::PaymentResponse::New();
@@ -551,7 +551,7 @@ TEST(OnPaymentResponseTest, ShippingInformationNotRequired) {
options.setRequestShipping(false);
PaymentRequest* request = PaymentRequest::create(
scope.getExecutionContext(), buildPaymentMethodDataForTest(),
- buildPaymentDetailsForTest(), options, scope.getExceptionState());
+ buildPaymentDetailsInitForTest(), options, scope.getExceptionState());
ASSERT_FALSE(scope.getExceptionState().hadException());
ScriptValue outValue;
request->show(scope.getScriptState())
@@ -578,7 +578,7 @@ TEST(OnPaymentResponseTest, PhoneNotRequred) {
options.setRequestPayerPhone(false);
PaymentRequest* request = PaymentRequest::create(
scope.getExecutionContext(), buildPaymentMethodDataForTest(),
- buildPaymentDetailsForTest(), options, scope.getExceptionState());
+ buildPaymentDetailsInitForTest(), options, scope.getExceptionState());
EXPECT_FALSE(scope.getExceptionState().hadException());
payments::mojom::blink::PaymentResponsePtr response =
payments::mojom::blink::PaymentResponse::New();
@@ -607,7 +607,7 @@ TEST(OnPaymentResponseTest, NameNotRequired) {
options.setRequestPayerName(false);
PaymentRequest* request = PaymentRequest::create(
scope.getExecutionContext(), buildPaymentMethodDataForTest(),
- buildPaymentDetailsForTest(), options, scope.getExceptionState());
+ buildPaymentDetailsInitForTest(), options, scope.getExceptionState());
EXPECT_FALSE(scope.getExceptionState().hadException());
payments::mojom::blink::PaymentResponsePtr response =
payments::mojom::blink::PaymentResponse::New();
@@ -636,7 +636,7 @@ TEST(OnPaymentResponseTest, EmailNotRequired) {
options.setRequestPayerEmail(false);
PaymentRequest* request = PaymentRequest::create(
scope.getExecutionContext(), buildPaymentMethodDataForTest(),
- buildPaymentDetailsForTest(), options, scope.getExceptionState());
+ buildPaymentDetailsInitForTest(), options, scope.getExceptionState());
EXPECT_FALSE(scope.getExceptionState().hadException());
payments::mojom::blink::PaymentResponsePtr response =
payments::mojom::blink::PaymentResponse::New();
« components/payments/content/payment_request.mojom ('K') | « third_party/WebKit/Source/modules/payments/CompleteTest.cpp ('k') | third_party/WebKit/Source/modules/payments/PaymentDetails.idl » ('j') | third_party/WebKit/Source/modules/payments/PaymentRequest.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698