PaymentRequest: Introduce PaymentDetailsInit and PaymentDetailsUpdate.

third_party/WebKit/Source/modules/payments/PaymentTestHelper.cpp

Index: third_party/WebKit/Source/modules/payments/PaymentTestHelper.cpp
diff --git a/third_party/WebKit/Source/modules/payments/PaymentTestHelper.cpp b/third_party/WebKit/Source/modules/payments/PaymentTestHelper.cpp
index d4e326b88afb2b7e3883cc540c1dd7b8b77e1bda..bfddff4c5c2bac4b7f76cf3949e8e72026d1290f 100644
--- a/third_party/WebKit/Source/modules/payments/PaymentTestHelper.cpp
+++ b/third_party/WebKit/Source/modules/payments/PaymentTestHelper.cpp
@@ -55,6 +55,41 @@ void setValues(PaymentItemOrPaymentShippingOption& original,
   }
 }
 
+void buildPaymentDetailsBaseInternal(

[please use gerrit instead, 2017/03/21 13:42:21]

No need to use the "Internal" suffix. The "Base" word in the name sufficiently
distinguishes this function from its peers and describes its purpose well.
Please remove the "Internal" suffix.

[zino, 2017/03/22 16:43:45]

Done.

+    PaymentDetailsBase* details,

[please use gerrit instead, 2017/03/21 13:42:21]

The output parameter should be the last one.

[zino, 2017/03/22 16:43:45]

Done.

+    PaymentTestDetailToChange detail,
+    PaymentTestDataToChange data,
+    PaymentTestModificationType modificationType,
+    const String& valueToUse) {
+  PaymentItem item;
+  if (detail == PaymentTestDetailItem)
+    item = buildPaymentItemForTest(data, modificationType, valueToUse);
+  else
+    item = buildPaymentItemForTest();
+
+  PaymentShippingOption shippingOption;
+  if (detail == PaymentTestDetailShippingOption) {
+    shippingOption =
+        buildShippingOptionForTest(data, modificationType, valueToUse);
+  } else {
+    shippingOption = buildShippingOptionForTest();
+  }
+
+  PaymentDetailsModifier modifier;
+  if (detail == PaymentTestDetailModifierTotal ||
+      detail == PaymentTestDetailModifierItem) {
+    modifier = buildPaymentDetailsModifierForTest(detail, data,
+                                                  modificationType, valueToUse);
+  } else {
+    modifier = buildPaymentDetailsModifierForTest();
+  }
+
+  details->setDisplayItems(HeapVector<PaymentItem>(1, item));
+  details->setShippingOptions(
+      HeapVector<PaymentShippingOption>(1, shippingOption));
+  details->setModifiers(HeapVector<PaymentDetailsModifier>(1, modifier));
+}
+
 }  // namespace
 
 PaymentItem buildPaymentItemForTest(
@@ -106,54 +141,52 @@ PaymentDetailsModifier buildPaymentDetailsModifierForTest(
   return modifier;
 }
 
-PaymentDetails buildPaymentDetailsForTest(
+PaymentDetailsInit buildPaymentDetailsInitForTest(
     PaymentTestDetailToChange detail,
     PaymentTestDataToChange data,
     PaymentTestModificationType modificationType,
     const String& valueToUse) {
-  PaymentItem total;
-  if (detail == PaymentTestDetailTotal)
-    total = buildPaymentItemForTest(data, modificationType, valueToUse);
-  else
-    total = buildPaymentItemForTest();
+  PaymentDetailsInit details;
+  buildPaymentDetailsBaseInternal(&details, detail, data, modificationType,
+                                  valueToUse);
 
-  PaymentItem item;
-  if (detail == PaymentTestDetailItem)
-    item = buildPaymentItemForTest(data, modificationType, valueToUse);
-  else
-    item = buildPaymentItemForTest();
+  if (detail == PaymentTestDetailTotal) {
+    details.setTotal(
+        buildPaymentItemForTest(data, modificationType, valueToUse));
+  } else {
+    details.setTotal(buildPaymentItemForTest());
+  }
 
-  PaymentShippingOption shippingOption;
-  if (detail == PaymentTestDetailShippingOption)
-    shippingOption =
-        buildShippingOptionForTest(data, modificationType, valueToUse);
-  else
-    shippingOption = buildShippingOptionForTest();
+  return details;
+}
 
-  PaymentDetailsModifier modifier;
-  if (detail == PaymentTestDetailModifierTotal ||
-      detail == PaymentTestDetailModifierItem)
-    modifier = buildPaymentDetailsModifierForTest(detail, data,
-                                                  modificationType, valueToUse);
-  else
-    modifier = buildPaymentDetailsModifierForTest();
+PaymentDetailsUpdate buildPaymentDetailsUpdateForTest(
+    PaymentTestDetailToChange detail,
+    PaymentTestDataToChange data,
+    PaymentTestModificationType modificationType,
+    const String& valueToUse) {
+  PaymentDetailsUpdate details;
+  buildPaymentDetailsBaseInternal(&details, detail, data, modificationType,
+                                  valueToUse);
 
-  PaymentDetails result;
-  result.setTotal(total);
-  result.setDisplayItems(HeapVector<PaymentItem>(1, item));
-  result.setShippingOptions(
-      HeapVector<PaymentShippingOption>(1, shippingOption));
-  result.setModifiers(HeapVector<PaymentDetailsModifier>(1, modifier));
+  if (detail == PaymentTestDetailTotal) {
+    details.setTotal(
+        buildPaymentItemForTest(data, modificationType, valueToUse));
+  } else {
+    details.setTotal(buildPaymentItemForTest());
+  }
 
   if (detail == PaymentTestDetailError)
-    result.setError(valueToUse);
+    details.setError(valueToUse);
 
-  return result;
+  return details;
 }
 
-PaymentDetails buildPaymentDetailsErrorMsgForTest(const String& valueToUse) {
-  return buildPaymentDetailsForTest(PaymentTestDetailError, PaymentTestDataNone,
-                                    PaymentTestOverwriteValue, valueToUse);
+PaymentDetailsUpdate buildPaymentDetailsErrorMsgForTest(
+    const String& valueToUse) {
+  return buildPaymentDetailsUpdateForTest(
+      PaymentTestDetailError, PaymentTestDataNone, PaymentTestOverwriteValue,
+      valueToUse);
 }
 
 HeapVector<PaymentMethodData> buildPaymentMethodDataForTest() {