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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/payments/ui/PaymentRequestUI.java

Issue 2571513004: [Payment Request] Update the logic when obtaining results from editors (Closed)
Patch Set: addressing test failures Created 4 years 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
Index: chrome/android/java/src/org/chromium/chrome/browser/payments/ui/PaymentRequestUI.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/payments/ui/PaymentRequestUI.java b/chrome/android/java/src/org/chromium/chrome/browser/payments/ui/PaymentRequestUI.java
index ccf8c2cabbc4d6fa6c35393ff45f86bb25a8bab2..95289b7696ef10afbf3504f4f3f0a62fb9cc5223 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/payments/ui/PaymentRequestUI.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/payments/ui/PaymentRequestUI.java
@@ -679,6 +679,7 @@ public class PaymentRequestUI implements DialogInterface.OnDismissListener, View
* @param section The shipping options.
*/
public void updateSection(@DataType int whichSection, SectionInformation section) {
+ mIsEditingPaymentItem = false;
if (whichSection == TYPE_SHIPPING_ADDRESSES) {
mShippingAddressSectionInformation = section;
mShippingAddressSection.update(section);
@@ -692,7 +693,6 @@ public class PaymentRequestUI implements DialogInterface.OnDismissListener, View
mPaymentMethodSectionInformation = section;
mPaymentMethodSection.update(section);
}
- mIsEditingPaymentItem = false;
updateSectionButtons();
updatePayButtonEnabled();
}
@@ -1111,7 +1111,7 @@ public class PaymentRequestUI implements DialogInterface.OnDismissListener, View
}
/**
- * Updates the enabled/disbled state of each section's edit button.
+ * Updates the enabled/disabled state of each section's edit button.
*
* Only the top-most button is enabled -- the others are disabled so the user is directed
* through the form from top to bottom.
@@ -1386,7 +1386,7 @@ public class PaymentRequestUI implements DialogInterface.OnDismissListener, View
}
@VisibleForTesting
- public ViewGroup getContactDetailsSectionForTest() {
+ public PaymentRequestSection getContactDetailsSectionForTest() {
return mContactDetailsSection;
}

Powered by Google App Engine
This is Rietveld 408576698